Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Extended side effect error messages #18418

Merged
merged 11 commits into from
Jul 15, 2021
Merged

Extended side effect error messages #18418

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
5161ae5
Extended side effect error messages
quantimnot Jul 2, 2021
8be3665
Applied feedback:
quantimnot Jul 3, 2021
c24eda9
- Added hints of where for side effect call locations.
quantimnot Jul 4, 2021
0c660c5
fix tests
quantimnot Jul 4, 2021
0c75dbc
Applied PR review feedback:
quantimnot Jul 5, 2021
09a079f
Applied NEP1 formatting suggestion
quantimnot Jul 6, 2021
ff20342
fixed? message
quantimnot Jul 6, 2021
15d649c
Revert "fixed? message"
quantimnot Jul 6, 2021
2421bee
simpler message
quantimnot Jul 6, 2021
b358810
formatting long proc sig for NEP1
quantimnot Jul 6, 2021
0f2be02
fixed double string formatting bug
quantimnot Jul 7, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions compiler/sempass2.nim
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -270,7 +270,7 @@ proc listGcUnsafety(s: PSym; onlyWarning: bool; conf: ConfigRef) =
proc listSideEffects(result: var string; s: PSym; cycleCheck: var IntSet;
conf: ConfigRef; context: PContext; indentLevel: int) =
template addHint(msg; lineInfo; sym; level = indentLevel) =
result.addf "$# $# Hint: '$#' $#\n" % [repeat(">", level), conf $ lineInfo, sym, msg]
result.addf("$# $# Hint: '$#' $#\n", repeat(">", level), conf $ lineInfo, sym, msg)
if context.sideEffects.hasKey(s.id):
for (useLineInfo, u) in context.sideEffects[s.id]:
if u != nil and not cycleCheck.containsOrIncl(u.id):
Expand All @@ -289,7 +289,7 @@ proc listSideEffects(result: var string; s: PSym; cycleCheck: var IntSet;

proc listSideEffects(result: var string; s: PSym; conf: ConfigRef; context: PContext) =
var cycleCheck = initIntSet()
result.addf "'$#' can have side effects\n" % s.name.s
result.addf("'$#' can have side effects\n", s.name.s)
timotheecour marked this conversation as resolved.
Show resolved Hide resolved
listSideEffects(result, s, cycleCheck, conf, context, 1)

proc useVarNoInitCheck(a: PEffects; n: PNode; s: PSym) =
Expand Down