Skip to content

%jinx hint#910

Merged
pkova merged 18 commits intodevelopfrom
dozreg/jinx
Nov 24, 2025
Merged

%jinx hint#910
pkova merged 18 commits intodevelopfrom
dozreg/jinx

Conversation

@dozreg-toplud
Copy link
Copy Markdown
Contributor

@dozreg-toplud dozreg-toplud commented Nov 2, 2025

Implements UIP-0124, replaces #648, supersedes #894.

Moves Urbit time functions to manage.c and defines a per-road stack of absolute deadlines. Virtual timer is set or disarmed any time a new deadline is pushed onto the stack, popped off the stack or if a road is left with some deadlines on its stack.

Defines a dynamic hint %jinx. If the product of the hint-formula is 0 or a cell, nothing happens, otherwise new deadline is pushed before the hinted formula is executed and popped once it is executed.

Questions

I had to mess with noun/build.zig and jets/e/json_en.c to make Windows build work, otherwise I was getting build errors. Why was that happening? Was there another, saner way to fix that?

Warnings

When playing around in a fakeship's dojo I saw a leak report once when I ran |mass. I then tried to reproduce it, including with -Dmem-dbg option and -g flag, to no avail.

@dozreg-toplud dozreg-toplud requested a review from a team as a code owner November 2, 2025 11:11
@dozreg-toplud dozreg-toplud requested a review from joemfb November 2, 2025 13:30
Comment thread pkg/noun/build.zig Outdated
Comment thread pkg/noun/nock.c
joemfb
joemfb previously approved these changes Nov 19, 2025
@dozreg-toplud
Copy link
Copy Markdown
Contributor Author

@joemfb I made timer functions noop instead of the hint logic, I think it's nicer that way

@dozreg-toplud dozreg-toplud requested a review from joemfb November 19, 2025 22:55
@pkova pkova merged commit e9f8b31 into develop Nov 24, 2025
4 checks passed
@pkova pkova deleted the dozreg/jinx branch November 24, 2025 16:13
This was referenced Nov 24, 2025
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.

3 participants