Skip to content

add hal_extend_int function#3936

Open
rmu75 wants to merge 1 commit intoLinuxCNC:masterfrom
rmu75:rs/hal_extend_int
Open

add hal_extend_int function#3936
rmu75 wants to merge 1 commit intoLinuxCNC:masterfrom
rmu75:rs/hal_extend_int

Conversation

@rmu75
Copy link
Copy Markdown
Collaborator

@rmu75 rmu75 commented Apr 14, 2026

helper function to deal with wrap around and extension of lower-width counters to 64bit ints.

Call with current 64bit counter value to be updated, new
lower-bit counter value and number of bits of counter.
@returns new counter value.
Code by Jeff Epler. */
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the license?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no idea. is this copyrightable in the first place?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes it is.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then @andypugh needs to get perm from @jepler.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi! You're free to use this code under the GPL v2.0 or newer license. I wrote it with the intent that it would be contributed to LinuxCNC.

helper function to deal with wrap around and extension of lower-width counters to 64bit ints.
@rmu75 rmu75 force-pushed the rs/hal_extend_int branch from e30194a to bf3c828 Compare April 14, 2026 18:06
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