swiftlang/swift

[SR-4259] Request: fixit for import module casing

Open

#46,842 opened on Mar 16, 2017

View on GitHub
 (3 comments) (0 reactions) (0 assignees)Swift (69,989 stars) (10,719 forks)batch import
bugcompilerdiagnostics qualitygood first issue

Description
Previous ID SR-4259
Radar None
Original Reporter @NachoSoto
Type Bug

Swift version 3.0.2 (swiftlang-800.0.63 clang-800.0.42.1)
Votes 0
Component/s Compiler
Labels Bug, DiagnosticsQoI, StarterBug
Assignee None
Priority Medium

md5: 24348ceb701412a7d44fffc88ff4e8c8

relates to:

  • SR-9554 Improve diagnostic for naming a project after a system framework, case-insensitively

Issue Description:

`import UIKIt` (notice the capital I) fails with the error: "Cannot load module 'UIKit' as 'UIKIt'".

Expected behavior: it produces a fixit to correct this. I almost couldn't see what was wrong, since i and I are so similar.

Contributor guide

Tech stack
swift
Domain
developer experience
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
Swiftcompiler internals
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
50
Research direction
Investigate the diagnostic emitter for module imports in the Swift compiler. Look for the error message 'Cannot load module' and add a fixit suggestion to correct the casing. The related issue SR 9554 may provide insight. The code is likely in lib/Sema/ or lib/Parse/.