Skip to content
This repository has been archived by the owner on Nov 8, 2018. It is now read-only.

modified type from u16 to u32 #30

Merged
merged 1 commit into from
Jun 10, 2018
Merged

modified type from u16 to u32 #30

merged 1 commit into from
Jun 10, 2018

Conversation

ikegam
Copy link
Contributor

@ikegam ikegam commented Jun 10, 2018

I faced a type issue. Based on my JSON from google takeout, accuracy goes over u16 such as "74130". And then, I got following error message from yore.

thread 'main' panicked at 'called Result::unwrap() on an Err value: HistoryError(DeserializeError(ErrorImpl { code: Message("invalid value: integer 74130, expected u16"), line: 277226, column: 22 }))', libcore/result.rs:945:5

So I simply increases the size.

@coveralls
Copy link

Coverage Status

Coverage decreased (-9.07%) to 63.101% when pulling 49de397 on ikegam:type-u16 into 19bed3e on WrinklyNinja:master.

@Ortham
Copy link
Owner

Ortham commented Jun 10, 2018

Thanks, I didn't know anyone was interested in this! 😮

@Ortham Ortham merged commit 5341ff7 into Ortham:master Jun 10, 2018
@ikegam ikegam deleted the type-u16 branch June 11, 2018 11:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants