Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Suppress warnings on 32bit platforms.
On 32bit platforms, int is the same as i32, so 0xffffffff is "out of range." Annotating variables as u32 fixes the problems. Signed-off-by: OGINO Masanori <masanori.ogino@gmail.com>
- Loading branch information
7af5f64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
saw approval from alexcrichton
at https://github.com/omasanori/rust/commit/7af5f646f1884dcfbcf0d2fae117db5502cfb38e
7af5f64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
merging omasanori/rust/suppress-warnings = 7af5f64 into auto
7af5f64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
omasanori/rust/suppress-warnings = 7af5f64 merged ok, testing candidate = d990681
7af5f64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/5769
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/5768
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-c/builds/4863
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/4871
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/5867
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-c/builds/4955
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/4963
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/5869
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-c/builds/4954
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/4960
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android/builds/5022
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/2756
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/5862
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-c/builds/4958
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/4973
7af5f64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fast-forwarding master to auto = d990681