Skip to content

fix: export fieldhash from Hash::Util - #1910

Merged
fglock merged 2 commits into
masterfrom
fix/hash-util-fieldhash-export
Oct 28, 2026
Merged

fglock merged 2 commits into
masterfrom
fix/hash-util-fieldhash-export

Conversation

@fglock

@fglock fglock commented Oct 28, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Export core fieldhash through the bundled Hash::Util module.
  • Extend the bundled Hash::Util::FieldHash tie to preserve ordinary scalar-key behavior as well as identity-based reference keys, including thread cloning.
  • Normalize tied-hash exists callback results to Perl's canonical 1 and '' values on both execution backends.
  • Add focused coverage and changelog entries.

Fixes #1909

Validation

  • nice -n 19 make passes locally.
  • Both new focused tests pass on system Perl, PerlOnJava JVM, and interpreter.
  • make check-links passes.
  • System Perl passes the unchanged Hash-ExtendedKeys 1.01 suite (7 files, 24 tests).
  • On PerlOnJava after these fixes, t/00-load.t and t/03-coverage.t pass. t/01-idea.t and t/02-tie.t stop at the transitive XS dependency Combine::Keys with Undefined subroutine &Combine::Keys::ckeys.

Expose the core fieldhash import through Hash::Util and preserve ordinary scalar-key behavior in the bundled fieldhash implementation. Add focused coverage for scalar and reference keys on both execution backends.

Fixes #1909

Generated with Codex (https://openai.com/codex)
Co-Authored-By: Codex <codex@openai.com>
Convert a tied hash EXISTS callback result to Perl's canonical boolean scalar in the shared runtime path. Cover undef, zero, and truthy results on both execution backends.

Fixes #1909

Generated with Codex (https://openai.com/codex)
Co-Authored-By: Codex <codex@openai.com>
@fglock
fglock marked this pull request as ready for review October 28, 2026 09:53
@fglock
fglock merged commit ed3fb02 into master Oct 28, 2026
2 checks passed
@fglock
fglock deleted the fix/hash-util-fieldhash-export branch October 28, 2026 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Export fieldhash from Hash::Util

1 participant