push pragma is silently ignored if position is in front of import statement

[StefanSalewski]

That can be very confusing.

(And I would really prefer SmallLshouldNotBeUsed being off by default :-)

cat t.nim
{.push warning[SmallLshouldNotBeUsed]: off.}

import strutils

var l = 1
echo l
stefan@nuc ~/NEd $ nimsuggest --v2 --stdin t.nim
Hint: used config file '/home/stefan/Nim/config/nim.cfg' [Conf]
Hint: used config file '/home/stefan/NEd/nim.cfg' [Conf]
usage: sug|con|def|use|dus|chk|highlight|outline file.nim[;dirtyfile.nim]:line:col
type 'quit' to quit
type 'debug' to toggle debug mode on/off
type 'terse' to toggle terse mode on/off
> chk t.nim:1:0
Hint: t [Processing]
t.nim(5, 5) Warning: 'l' should not be used as an identifier; may look like '1' (one) [SmallLshouldNotBeUsed]
t.nim(6, 6) Warning: 'l' should not be used as an identifier; may look like '1' (one) [SmallLshouldNotBeUsed]

> quit
stefan@nuc ~/NEd $ nim c t.nim 
Hint: used config file '/home/stefan/Nim/config/nim.cfg' [Conf]
Hint: used config file '/home/stefan/NEd/nim.cfg' [Conf]
Hint: system [Processing]
Hint: t [Processing]
Hint: strutils [Processing]
Hint: parseutils [Processing]
Hint: math [Processing]
Hint: algorithm [Processing]
t.nim(5, 5) Warning: 'l' should not be used as an identifier; may look like '1' (one) [SmallLshouldNotBeUsed]
t.nim(6, 6) Warning: 'l' should not be used as an identifier; may look like '1' (one) [SmallLshouldNotBeUsed]
CC: t
CC: stdlib_strutils
CC: stdlib_parseutils
CC: stdlib_math
CC: stdlib_algorithm
Hint:  [Link]
Hint: operation successful (14346 lines compiled; 0.178 sec total; 20.004MiB; Debug Build) [SuccessX]
stefan@nuc ~/NEd $ 

[mratsim]

Does it even apply on imported files?

I'm trying to remove the Warning: Number of spaces around '==' is not consistent [Spacing] spam on my test files.

[Araq]

Up to date example:

{.push warning[Spacing]: off.}

import strutils

var l = 1 ==  7
echo l

{.pop.}

vs

import strutils

{.push warning[Spacing]: off.}

var l = 1 ==  7
echo l

{.pop.}