Skip to content

Commit afa457e

Browse files
committed
Rename signed in numeric tokens to has_sign.
1 parent 890cc7c commit afa457e

File tree

4 files changed

+13
-13
lines changed

4 files changed

+13
-13
lines changed

src/nth.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ fn parse_b(input: &mut Parser, a: i32) -> Result<(i32, i32), ()> {
5757
match input.next() {
5858
Ok(Token::Delim('+')) => parse_signless_b(input, a, 1),
5959
Ok(Token::Delim('-')) => parse_signless_b(input, a, -1),
60-
Ok(Token::Number(ref value)) if value.signed => {
60+
Ok(Token::Number(ref value)) if value.has_sign => {
6161
Ok((a, try!(value.int_value.ok_or(())) as i32))
6262
}
6363
_ => {
@@ -69,7 +69,7 @@ fn parse_b(input: &mut Parser, a: i32) -> Result<(i32, i32), ()> {
6969

7070
fn parse_signless_b(input: &mut Parser, a: i32, b_sign: i32) -> Result<(i32, i32), ()> {
7171
match try!(input.next()) {
72-
Token::Number(ref value) if !value.signed => {
72+
Token::Number(ref value) if !value.has_sign => {
7373
Ok((a, b_sign * (try!(value.int_value.ok_or(())) as i32)))
7474
}
7575
_ => Err(())

src/serializer.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ pub trait ToCss {
4949
fn write_numeric<W>(value: NumericValue, dest: &mut W) -> text_writer::Result
5050
where W: TextWriter {
5151
// `value.value >= 0` is true for negative 0.
52-
if value.signed && value.value.is_positive() {
52+
if value.has_sign && value.value.is_positive() {
5353
try!(dest.write_str("+"));
5454
}
5555

@@ -94,11 +94,11 @@ impl<'a> ToCss for Token<'a> {
9494
Token::Delim(value) => try!(dest.write_char(value)),
9595

9696
Token::Number(value) => try!(write_numeric(value, dest)),
97-
Token::Percentage(PercentageValue { unit_value, int_value, signed }) => {
97+
Token::Percentage(PercentageValue { unit_value, int_value, has_sign }) => {
9898
let value = NumericValue {
9999
value: unit_value * 100.,
100100
int_value: int_value,
101-
signed: signed,
101+
has_sign: has_sign,
102102
};
103103
try!(write_numeric(value, dest));
104104
try!(dest.write_char('%'));

src/tests.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -545,11 +545,11 @@ fn one_component_value_to_json(token: Token, input: &mut Parser) -> Json {
545545
Token::Delim(value) => value.to_string().to_json(),
546546

547547
Token::Number(value) => Json::Array(vec!["number".to_json()] + numeric(value).as_slice()),
548-
Token::Percentage(PercentageValue { unit_value, int_value, signed }) => Json::Array(
548+
Token::Percentage(PercentageValue { unit_value, int_value, has_sign }) => Json::Array(
549549
vec!["percentage".to_json()] + numeric(NumericValue {
550550
value: unit_value * 100.,
551551
int_value: int_value,
552-
signed: signed,
552+
has_sign: has_sign,
553553
}).as_slice()),
554554
Token::Dimension(value, unit) => Json::Array(
555555
vec!["dimension".to_json()] + numeric(value).as_slice() + [unit.to_json()].as_slice()),

src/tokenizer.rs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ pub struct NumericValue {
170170
/// Whether the number had a `+` or `-` sign.
171171
///
172172
/// This is used is some cases like the <An+B> micro syntax. (See the `parse_nth` function.)
173-
pub signed: bool,
173+
pub has_sign: bool,
174174
}
175175

176176

@@ -184,7 +184,7 @@ pub struct PercentageValue {
184184
pub int_value: Option<i32>,
185185

186186
/// Whether the number had a `+` or `-` sign.
187-
pub signed: bool,
187+
pub has_sign: bool,
188188
}
189189

190190

@@ -653,8 +653,8 @@ fn consume_numeric<'a>(tokenizer: &mut Tokenizer<'a>) -> Token<'a> {
653653
// But this is always called so that there is at least one digit in \d*(\.\d+)?
654654
let start_pos = tokenizer.position();
655655
let mut is_integer = true;
656-
let signed = matches!(tokenizer.next_char(), '-' | '+');
657-
if signed {
656+
let has_sign = matches!(tokenizer.next_char(), '-' | '+');
657+
if has_sign {
658658
tokenizer.advance(1);
659659
}
660660
consume_digits(tokenizer);
@@ -696,13 +696,13 @@ fn consume_numeric<'a>(tokenizer: &mut Tokenizer<'a>) -> Token<'a> {
696696
return Percentage(PercentageValue {
697697
unit_value: value / 100.,
698698
int_value: int_value,
699-
signed: signed,
699+
has_sign: has_sign,
700700
})
701701
}
702702
let value = NumericValue {
703703
value: value,
704704
int_value: int_value,
705-
signed: signed,
705+
has_sign: has_sign,
706706
};
707707
if is_ident_start(tokenizer) { Dimension(value, consume_name(tokenizer)) }
708708
else { Number(value) }

0 commit comments

Comments
 (0)