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.

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

disallow implicit return if result is used, don't use void context #24406

Closed
wants to merge 4 commits into from

completely consider void, giving discard check error again

46206d6
Select commit
Loading
Failed to load commit list.
Closed

disallow implicit return if result is used, don't use void context #24406

completely consider void, giving discard check error again
46206d6
Select commit
Loading
Failed to load commit list.
Azure Pipelines / nim-lang.Nim failed Nov 4, 2024 in 1h 10m 41s

Build #20241104.8 had test failures

Details

Tests

  • Failed: 39 (0.22%)
  • Passed: 8,480 (48.10%)
  • Other: 9,112 (51.68%)
  • Total: 17,631

Annotations

Check failure on line 5009 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / nim-lang.Nim

Build log #L5009

Bash exited with code '1'.

Check failure on line 7644 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / nim-lang.Nim

Build log #L7644

Bash exited with code '1'.

Check failure on line 4866 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / nim-lang.Nim

Build log #L4866

Bash exited with code '1'.

Check failure on line 48261 in Build log

See this annotation in the file changed.

@azure-pipelines azure-pipelines / nim-lang.Nim

Build log #L48261

Bash exited with code '1'.

Check failure on line 1 in tests/macros/t18203.nim c --hint:SuccessX:off --hint:Link:off --hint:Conf:off --hint:CC:off --hint:XDeclaredButNotUsed:on

See this annotation in the file changed.

@azure-pipelines azure-pipelines / nim-lang.Nim

tests/macros/t18203.nim c --hint:SuccessX:off --hint:Link:off --hint:Conf:off --hint:CC:off --hint:XDeclaredButNotUsed:on

Failure: reMsgsDiffer
Expected:


Gotten:
t18203.nim(12, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]

Check failure on line 1 in tests/arc/tmovebug.nim c

See this annotation in the file changed.

@azure-pipelines azure-pipelines / nim-lang.Nim

tests/arc/tmovebug.nim c

Failure: reNimcCrash
$ /home/vsts/work/1/s/bin/nim c --gc:arc tests/arc/tmovebug.nim
command line(1, 2) Warning: `gc:option` is deprecated; use `mm:option` instead [Deprecated]
Hint: used config file '/home/vsts/work/1/s/config/nim.cfg' [Conf]
Hint: used config file '/home/vsts/work/1/s/config/config.nims' [Conf]
Hint: used config file '/home/vsts/work/1/s/tests/config.nims' [Conf]
Hint: used config file '/home/vsts/work/1/s/tests/arc/nim.cfg' [Conf]
command line(1, 2) Warning: `gc:option` is deprecated; use `mm:option` instead [Deprecated]
tmovebug.nim(124, 1) Warning: A custom '=destroy' hook which takes a 'var T' parameter is deprecated; it should take a 'T' parameter [Deprecated]
tmovebug.nim(130, 1) Warning: Overriding `=` hook is deprecated; Override `=copy` hook instead [Deprecated]
tmovebug.nim(204, 9) Hint: 'x' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(205, 10) Hint: 'nm' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(205, 14) Hint: 'rest' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(211, 15) Hint: duplicate import of 'strutils'; previous import here: tmovebug.nim(200, 8) [DuplicateModuleImport]
tmovebug.nim(220, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(224, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(234, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(238, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(248, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(252, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(263, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
../../lib/system.nim(831, 3) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(267, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
../../lib/system.nim(709, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
../../lib/system.nim(667, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
../../lib/system.nim(918, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(349, 1) Warning: Overriding `=` hook is deprecated; Override `=copy` hook instead [Deprecated]
tmovebug.nim(451, 13) Warning: unreachable code after 'return' statement or '{.noReturn.}' proc [UnreachableCode]
tmovebug.nim(451, 42) Warning: unreachable code after 'return' statement or '{.noReturn.}' proc [UnreachableCode]
tmovebug.nim(508, 15) Hint: 'a' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(513, 15) Hint: 'a' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(528, 21) Hint: 'c' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(522, 15) Hint: 'a' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(524, 17) Hint: 'b' is declared but not used [XDeclaredButNotUsed]
tmovebug.nim(578, 3) Error: expression 'result' is of type 'ref Wrapper' and has to be used (or discarded); start of expression here: tmovebug.nim(577, 3)

Check failure on line 1 in tests/misc/taddr.nim c

See this annotation in the file changed.

@azure-pipelines azure-pipelines / nim-lang.Nim

tests/misc/taddr.nim c

Failure: reNimcCrash
$ /home/vsts/work/1/s/bin/nim c --hints:on -d:testing --nimblePath:build/deps/pkgs2   --nimCache:nimcache/tests/misc/taddr.nim_4a8a08f09d37b73795649038408b5f33  tests/misc/taddr.nim
Hint: used config file '/home/vsts/work/1/s/config/nim.cfg' [Conf]
Hint: used config file '/home/vsts/work/1/s/config/config.nims' [Conf]
Hint: used config file '/home/vsts/work/1/s/tests/config.nims' [Conf]
../../lib/system.nim(1799, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
../../lib/system.nim(1799, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
taddr.nim(90, 8) Hint: 'byPtr' is declared but not used [XDeclaredButNotUsed]
taddr.nim(144, 8) Error: expression 'a.x' is of type 'float' and has to be used (or discarded); start of expression here: taddr.nim(143, 7)

Check failure on line 1 in tests/misc/taddr.nim cpp

See this annotation in the file changed.

@azure-pipelines azure-pipelines / nim-lang.Nim

tests/misc/taddr.nim cpp

Failure: reNimcCrash
$ /home/vsts/work/1/s/bin/nim cpp --hints:on -d:testing --nimblePath:build/deps/pkgs2   --nimCache:nimcache/tests/misc/taddr.nim_2c5f2cba9af7f92986d3de1e753f3faa  tests/misc/taddr.nim
Hint: used config file '/home/vsts/work/1/s/config/nim.cfg' [Conf]
Hint: used config file '/home/vsts/work/1/s/config/config.nims' [Conf]
Hint: used config file '/home/vsts/work/1/s/tests/config.nims' [Conf]
../../lib/system.nim(1799, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
../../lib/system.nim(1799, 1) Hint: 'result' is declared but not used [XDeclaredButNotUsed]
taddr.nim(90, 8) Hint: 'byPtr' is declared but not used [XDeclaredButNotUsed]
taddr.nim(144, 8) Error: expression 'a.x' is of type 'float' and has to be used (or discarded); start of expression here: taddr.nim(143, 7)