Skip to content

Fix JSON encoding of 64-bit integers - #108

Merged
HaroldCindy merged 1 commit into
secondlife:mainfrom
wolfosbournesl:fix/lljson-large-integers-103
Nov 3, 2026
Merged

HaroldCindy merged 1 commit into
secondlife:mainfrom
wolfosbournesl:fix/lljson-large-integers-103

Conversation

@wolfosbournesl

Copy link
Copy Markdown
Contributor

Summary

  • Preserve all 64 bits when encoding integer values in standard JSON and SL-specific JSON.
  • Add boundary tests for signed 64-bit values and tagged round-trips.

Fixes #103.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 1, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-01T17:13:34.062572Z 7153435 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@HaroldCindy

Copy link
Copy Markdown
Contributor

Seems reasonable to me, thanks!

@HaroldCindy
HaroldCindy merged commit 1f88a5d into secondlife:main Nov 3, 2026
8 checks passed
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.

lljson.slencode cannot encode large integers

2 participants