Skip to content

Commit

Permalink
close nim-lang#16646; since now works with bootstrap nim post csour…
Browse files Browse the repository at this point in the history
…ces_v1 (nim-lang#17895)

* revive nim-lang#16627 now that csources_v1 was merged

* use dedent in rst.nim, refs nim-lang#17257 (comment)

* fix test and improve rendering of a rst warning
  • Loading branch information
timotheecour authored and PMunch committed Mar 28, 2022
1 parent 09c5aa2 commit e282adf
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
4 changes: 2 additions & 2 deletions compiler/semtypes.nim
Original file line number Diff line number Diff line change
Expand Up @@ -256,8 +256,8 @@ proc semRangeAux(c: PContext, n: PNode, prev: PType): PType =
else:
result.n.add semConstExpr(c, range[i])

if (result.n[0].kind in {nkFloatLit..nkFloat64Lit} and classify(result.n[0].floatVal) == fcNan) or
(result.n[1].kind in {nkFloatLit..nkFloat64Lit} and classify(result.n[1].floatVal) == fcNan):
if (result.n[0].kind in {nkFloatLit..nkFloat64Lit} and result.n[0].floatVal.isNaN) or
(result.n[1].kind in {nkFloatLit..nkFloat64Lit} and result.n[1].floatVal.isNaN):
localError(c.config, n.info, "NaN is not a valid start or end for a range")

if weakLeValue(result.n[0], result.n[1]) == impNo:
Expand Down
5 changes: 2 additions & 3 deletions lib/packages/docutils/rst.nim
Original file line number Diff line number Diff line change
Expand Up @@ -2250,11 +2250,10 @@ proc parseEnumList(p: var RstParser): PRstNode =
let n = p.line + p.tok[j].line
let msg = "\n" & """
not enough indentation on line $2
(should be at column $3 if it's a continuation of enum. list),
(should be at column $3 if it's a continuation of enum. list),
or no blank line after line $1 (if it should be the next paragraph),
or no escaping \ at the beginning of line $1
(if lines $1..$2 are a normal paragraph, not enum. list)""".
unindent(8)
(if lines $1..$2 are a normal paragraph, not enum. list)""".dedent
let c = p.col + requiredIndent + ColRstOffset
rstMessage(p, mwRstStyle, msg % [$(n-1), $n, $c],
p.tok[j].line, p.tok[j].col)
Expand Down
2 changes: 1 addition & 1 deletion tests/stdlib/trstgen.nim
Original file line number Diff line number Diff line change
Expand Up @@ -849,7 +849,7 @@ Test1
let output8 = input8.toHtml(warnings = warnings8)
check(warnings8[].len == 1)
check("input(6, 1) Warning: RST style: \n" &
" not enough indentation on line 6" in warnings8[0])
"not enough indentation on line 6" in warnings8[0])
doAssert output8 == "Paragraph.<ol class=\"upperalpha simple\">" &
"<li>stringA</li>\n<li>stringB</li>\n</ol>\n<p>C. string1 string2 </p>\n"

Expand Down

0 comments on commit e282adf

Please sign in to comment.