Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Remove a last println and use a static default parser
Heap profilers are fun
  • Loading branch information
bspeice committed Sep 18, 2018
1 parent 4d7c5dd commit 741afa3
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 9 deletions.
5 changes: 3 additions & 2 deletions build_pycompat.py 100644 → 100755
@@ -1,3 +1,4 @@
#!/usr/bin/python3
from dateutil.parser import parse
from dateutil.tz import tzutc
from datetime import datetime
Expand Down Expand Up @@ -232,7 +233,7 @@ def test_fuzzy_simple(i, s):
tzinfos: &HashMap<String, i32>,
) {
let mut parser = Parser::new(info);
let parser = Parser::new(info);
let rs_parsed = parser.parse(
s,
dayfirst,
Expand Down Expand Up @@ -282,7 +283,7 @@ def test_fuzzy_simple(i, s):
tzinfos: &HashMap<String, i32>,
) {
let mut parser = Parser::new(info);
let parser = Parser::new(info);
let rs_parsed = parser.parse(
s,
dayfirst,
Expand Down
1 change: 1 addition & 0 deletions build_pycompat_tokenizer.py 100644 → 100755
@@ -1,3 +1,4 @@
#!/usr/bin/python3
from dateutil.parser import _timelex

from build_pycompat import tests
Expand Down
8 changes: 4 additions & 4 deletions src/lib.rs
Expand Up @@ -108,6 +108,7 @@ lazy_static! {
static ref ONE: Decimal = Decimal::new(1, 0);
static ref TWENTY_FOUR: Decimal = Decimal::new(24, 0);
static ref SIXTY: Decimal = Decimal::new(60, 0);
static ref DEFAULT_PARSER: Parser = Parser::default();
}

impl From<DecimalError> for ParseError {
Expand Down Expand Up @@ -670,7 +671,7 @@ impl Parser {
/// order to be resolved.
#[cfg_attr(feature = "cargo-clippy", allow(too_many_arguments))] // Need to release a 2.0 for changing public API
pub fn parse(
&mut self,
&self,
timestr: &str,
dayfirst: Option<bool>,
yearfirst: Option<bool>,
Expand Down Expand Up @@ -699,7 +700,7 @@ impl Parser {

#[cfg_attr(feature = "cargo-clippy", allow(cyclomatic_complexity))] // Imitating Python API is priority
fn parse_with_tokens(
&mut self,
&self,
timestr: &str,
dayfirst: Option<bool>,
yearfirst: Option<bool>,
Expand Down Expand Up @@ -953,7 +954,6 @@ impl Parser {
) -> ParseResult<Option<FixedOffset>> {
// TODO: Actual timezone support
if let Some(offset) = res.tzoffset {
println!("offset={}", offset);
Ok(Some(FixedOffset::east(offset)))
} else if res.tzoffset == None
&& (res.tzname == Some(" ".to_owned()) || res.tzname == Some(".".to_owned())
Expand Down Expand Up @@ -1283,7 +1283,7 @@ fn ljust(s: &str, chars: usize, replace: char) -> String {
/// The default implementation assumes English values for names of months,
/// days of the week, etc. It is equivalent to Python's `dateutil.parser.parse()`
pub fn parse(timestr: &str) -> ParseResult<(NaiveDateTime, Option<FixedOffset>)> {
let res = Parser::default().parse(
let res = DEFAULT_PARSER.parse(
timestr,
None,
None,
Expand Down
2 changes: 1 addition & 1 deletion src/tests/fuzzing.rs
Expand Up @@ -16,7 +16,7 @@ fn test_fuzz() {
// OverflowError: Python int too large to convert to C long
// assert_eq!(parse("8888884444444888444444444881"), Err(ParseError::AmPmWithoutHour));
let default = NaiveDate::from_ymd(2016, 6, 29).and_hms(0, 0, 0);
let mut p = Parser::default();
let p = Parser::default();
let res = p.parse("\x0D\x31", None, None, false, false, Some(&default), false, &HashMap::new()).unwrap();
assert_eq!(res.0, default);

Expand Down
4 changes: 2 additions & 2 deletions src/tests/pycompat_parser.rs
Expand Up @@ -39,7 +39,7 @@ fn parse_and_assert(
tzinfos: &HashMap<String, i32>,
) {

let mut parser = Parser::new(info);
let parser = Parser::new(info);
let rs_parsed = parser.parse(
s,
dayfirst,
Expand Down Expand Up @@ -89,7 +89,7 @@ fn parse_fuzzy_and_assert(
tzinfos: &HashMap<String, i32>,
) {

let mut parser = Parser::new(info);
let parser = Parser::new(info);
let rs_parsed = parser.parse(
s,
dayfirst,
Expand Down

0 comments on commit 741afa3

Please sign in to comment.