Skip to content
This repository was archived by the owner on Oct 8, 2020. It is now read-only.

fix 32bit machine bug#23

Open
sloanyyc wants to merge 1 commit intom8rge:masterfrom
sloanyyc:patch-1
Open

fix 32bit machine bug#23
sloanyyc wants to merge 1 commit intom8rge:masterfrom
sloanyyc:patch-1

Conversation

@sloanyyc
Copy link
Copy Markdown

@sloanyyc sloanyyc commented Mar 9, 2017

(((n - 1 - i) >> 31) & '=') | ...
----->
(((n - 1 - i) >> 23) & '=') | ...
32bit core -1 >> 31 0x00000001
64bit core -1 >> 31 0x00000001ffffffff

32bit core -1 >> 23 0x000001ff
64bit core -1 >> 23 0x000001ffffffffff
base64 output limit to 2^23

 (((n - 1 - i) >> 31) & '=') | ...
----->
 (((n - 1 - i) >> 23) & '=') | ...
32bit core     -1 >> 31      0x00000001
64bit core     -1 >> 31      0x00000001ffffffff

32bit core     -1 >> 23      0x000001ff
64bit core     -1 >> 23      0x000001ffffffffff
base64 output limit to 2^23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant