diff options
| author | bors-servo <metajack+bors@gmail.com> | 2015-10-10 15:00:39 -0600 |
|---|---|---|
| committer | bors-servo <metajack+bors@gmail.com> | 2015-10-10 15:00:39 -0600 |
| commit | 02d889494580d2b387a42d3a94360e40dce96282 (patch) | |
| tree | 7bc10830b73f8deaadd325c5b0a2c7371d1f948e /components/script/dom/macros.rs | |
| parent | babeed998d297d2d7badc06ba9c8f90e42239b4f (diff) | |
| parent | 4dc8fd76ece28e8b2c351e6a3f4241baec1c0996 (diff) | |
| download | servo-02d889494580d2b387a42d3a94360e40dce96282.tar.gz servo-02d889494580d2b387a42d3a94360e40dce96282.zip | |
Auto merge of #7958 - frewsxcv:consolidate-magic-number, r=mbrubeck
Consolidate magic number representing max unsigned long
<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/7958)
<!-- Reviewable:end -->
Diffstat (limited to 'components/script/dom/macros.rs')
| -rw-r--r-- | components/script/dom/macros.rs | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/components/script/dom/macros.rs b/components/script/dom/macros.rs index 0583fc7edd7..5d843ce5e26 100644 --- a/components/script/dom/macros.rs +++ b/components/script/dom/macros.rs @@ -146,8 +146,9 @@ macro_rules! make_uint_setter( ($attr:ident, $htmlname:expr, $default:expr) => ( fn $attr(&self, value: u32) { use dom::bindings::codegen::InheritTypes::ElementCast; + use dom::values::UNSIGNED_LONG_MAX; use string_cache::Atom; - let value = if value > 2147483647 { + let value = if value > UNSIGNED_LONG_MAX { $default } else { value @@ -167,10 +168,11 @@ macro_rules! make_limited_uint_setter( ($attr:ident, $htmlname:expr, $default:expr) => ( fn $attr(&self, value: u32) -> $crate::dom::bindings::error::ErrorResult { use dom::bindings::codegen::InheritTypes::ElementCast; + use dom::values::UNSIGNED_LONG_MAX; use string_cache::Atom; let value = if value == 0 { return Err($crate::dom::bindings::error::Error::IndexSize); - } else if value > 2147483647 { + } else if value > UNSIGNED_LONG_MAX { $default } else { value |