Skip to content
This repository has been archived by the owner on Sep 6, 2023. It is now read-only.

machine.mem32[] returns signed 32bit integers #81

Open
oddstr13 opened this issue May 9, 2017 · 2 comments
Open

machine.mem32[] returns signed 32bit integers #81

oddstr13 opened this issue May 9, 2017 · 2 comments

Comments

@oddstr13
Copy link
Contributor

oddstr13 commented May 9, 2017

I would expect unsigned integers here.

@dpgeorge
Copy link
Member

dpgeorge commented May 9, 2017

Yes, it's possible that unsigned integers would be the more common use. But it's easy to get around the current behaviour: just do mem32[addr] & 0xffffffff.

It could be argued that there should be signed and unsigned accessors, like mem32 for signed and umem32 for unsigned.

@nickzoic
Copy link
Collaborator

nickzoic commented Aug 3, 2017

(yeah, I think we should fix that ... but that'd be all the ports, right?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants