Skip to content

Commit

Permalink
Fix and clean CI after Nim 2.2.0 update (#84)
Browse files Browse the repository at this point in the history
* Specify template's return type for correct variable declaration

* [modular_inverse] Reduce the number of tests to make CI readable again
  • Loading branch information
dlesnoff authored Oct 10, 2024
1 parent 4a59ed8 commit af31828
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion maths/modular_inverse.nim
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ when isMainModule:
check modularInverse(17, 17).is_none()

randomize()
const randomTestSize = 1000
const randomTestSize = 10
for testNum in 0..randomTestSize:
let a = rand(-10000000..10000000)
let modulus = rand(1..1000000)
Expand Down
2 changes: 1 addition & 1 deletion strings/check_anagram.nim
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ func toLowerUnchecked(c: char): char {.inline.} =
# 0b100_0001, so setting the sixth bit to 1 gives letter's lowercase pair.
char(uint8(c) or 0b0010_0000'u8)

template normalizeChar(c: char) =
template normalizeChar(c: char): char =

This comment has been minimized.

Copy link
@ZoomRmc

ZoomRmc Oct 14, 2024

Contributor

It's interesting how I missed that. Probably the fact that it somehow still compiled helped! :)

## Checks if the character is a letter and lowers its case.
##
## Raises a `ValueError` on other characters.
Expand Down

0 comments on commit af31828

Please sign in to comment.