-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
websocket-test.el fails on 32 bit machine #18
Comments
I'm not sure I understand the gist. Those are my tests, but I don't see that my tests are doing anything with testing out powers of 2. For instance, in the first broken test, websocket-encode-frame, I just test out encoding strings of length 200 and 7000. But your test seems to be doing something else. Am I missing something? Also, the error with parsing 4294967296 seems plausible, at least when byte-compiling. I'll fix that somehow. |
I think you can see that some integer is overflowing in the In Emacs 23.1:
In Emacs 24.1:
BTW, these difference is maybe from this change in 23.2:
-- http://www.gnu.org/software/emacs/NEWS.23.2 (via http://stackoverflow.com/questions/2975653) |
That's really interesting, thanks for finding that issue. Yeah, I understand the issue, just not that gist. I'm working on replacing things with calc in appropriate places. It will slow some things down, but hopefully sending packets that are longer than ~4GB long is going to rare. |
Thanks! |
Should be fixed in cf885f0. Since I can't reproduce the issue, please re-open this bug if it does not fix the issue. |
...Amazing. I literally found emacs-ipython-notebook minutes before you fixed this bug. Your fix works for me. Arch linux 32-bit, Emacs 23.1. Before, With latest master, it works fine. Thanks. |
Wow, that's super fast! Unfortunately I can't access 32-bit machine now. I will tell the result when check it. @gcr, thanks for checking and comment! I guess then it should work on my machine. |
|
I tweaked test to run it in 32-bit system. Please check #19. It seems that websocket.el itself works properly. |
Here is the full tracebacks:
https://gist.github.com/3046935
I checked in Emacs 24.1.50.1 and 23.1.1. Some machine information:
In 64 bit machine, the test was fine. Note that websocket.el actually works in the 32 bit machine. I haven't notice the problem at all until now.
Furthermore, @epatters and @Boothead report that loading websocket.el fails with
load-with-code-conversion: Arithmetic overflow error: "4294967296"
while reading the number 4294967296. Note that I have no problem loading websocket.el even though most-positive-fixnum (=536870911) is smaller than 4294967296.The original report is here: tkf/emacs-ipython-notebook#8
The text was updated successfully, but these errors were encountered: