Integer overflow bug in fastfilereader and offset parameter to stream a file#533
Integer overflow bug in fastfilereader and offset parameter to stream a file#533mayanks wants to merge 3 commits into
Conversation
mayanks
changed the title
|
Looks reasonable - please rebase the PR, and squash to 2 commits: one for the int/long fixes, the other for the new offset feature. |
|
Hi - I noticed the added commits, but I don't think it came out as intended. You might have rebased locally, then done a 'git pull' and merged origin/master back into your local master before you pushed? The trick with a rebase is that you must force-push once you have the commits on your local branch organized as you wish, so that the origin matches local. |
|
I should have commented that I messed this whole rebase business. I'll try and clean this up again. If I am not able to clean it, I'll submit a fresh pull request. I hope that should be fine. |
Fixed bug in reading file greater than 2GB in size converted INT to LONG to support >2GB file size converted to NUM to LONG sorry, my bad. used wrong macro yet another overflow when adding start and offset updated the function definition converted to long to unsigned long
|
Hi Aaron, after some hiccups and doing multiple reverts, I finally managed to clean the commit tree (and learned some new git tricks). Do let me know if this looks good. Thanks for your patience. |
2 participants
Nothing too fancy. Just had a requirement to stream a file from a certain offset (in case the file was not streamed completely last time). So went ahead and added this feature in the eventmachine.
Please accept this pull request if you feel it is a good feature to have.