Skip to content

fix: rename SimpleRng::next_u32 to next#64

Open
Lukasz-Juranek wants to merge 1 commit intoeclipse-score:mainfrom
Lukasz-Juranek:fix/rename-simple-rng-next-u32
Open

fix: rename SimpleRng::next_u32 to next#64
Lukasz-Juranek wants to merge 1 commit intoeclipse-score:mainfrom
Lukasz-Juranek:fix/rename-simple-rng-next-u32

Conversation

@Lukasz-Juranek
Copy link
Contributor

Summary

  • Renamed SimpleRng::next_u32 to SimpleRng::next in src/kyron/src/time/mod.rs (test-only code)
  • The function returns u64, not u32, making the previous name misleading
  • Updated all 3 call sites: gen_range() and 2 direct usages in fuzzy tests

Test plan

  • bazel test //src/kyron:tests — PASSED before and after fix
  • bazel test //... — all 8 test targets PASSED
  • bazel build //... — full build SUCCESS
  • Format checks (rustfmt, ruff, buildifier, yamlfmt) — all PASSED

Closes #13

🤖 Generated with Claude Code

@github-actions
Copy link

github-actions bot commented Feb 14, 2026

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: ac9b7ca6-a0da-4c51-89a8-f09a71642249
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'rules_python', the root module requires module version rules_python@1.4.1, but got rules_python@1.8.3 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'bazel_skylib', the root module requires module version bazel_skylib@1.7.1, but got bazel_skylib@1.8.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'rules_cc', the root module requires module version rules_cc@0.1.1, but got rules_cc@0.2.14 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'aspect_rules_lint', the root module requires module version aspect_rules_lint@1.0.3, but got aspect_rules_lint@1.5.3 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'buildifier_prebuilt', the root module requires module version buildifier_prebuilt@7.3.1, but got buildifier_prebuilt@8.2.0.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (38 packages loaded, 9 targets configured)

Analyzing: target //:license-check (71 packages loaded, 9 targets configured)

Analyzing: target //:license-check (85 packages loaded, 24 targets configured)

Analyzing: target //:license-check (130 packages loaded, 1329 targets configured)

Analyzing: target //:license-check (136 packages loaded, 2528 targets configured)

Analyzing: target //:license-check (141 packages loaded, 2577 targets configured)

Analyzing: target //:license-check (141 packages loaded, 2577 targets configured)

Analyzing: target //:license-check (142 packages loaded, 2701 targets configured)

INFO: Analyzed target //:license-check (146 packages loaded, 4716 targets configured).
[9 / 17] [Prepa] Expanding template external/score_tooling+/dash/tool/formatters/dash_format_converter [for tool]
INFO: From Generating Dash formatted dependency file ...:
INFO: Successfully converted 209 packages from Cargo.lock to bazel-out/k8-fastbuild/bin/formatted.txt
[15 / 17] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 27.198s, Critical Path: 2.50s
INFO: 17 processes: 12 internal, 4 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 17 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

The function returns u64, not u32, making the name misleading.
Rename to `next` to accurately reflect the return type.

Closes eclipse-score#13

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

The created documentation from the pull request is available at: docu-html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

Bug: SimpleRng::next_u32 name is invalid

1 participant