Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement max length behavior #4

Merged
merged 2 commits into from
Apr 23, 2019

Conversation

WilliamDenniss
Copy link
Collaborator

Implement behavior specified in google/open-location-code#190

@codecov-io
Copy link

codecov-io commented Apr 22, 2019

Codecov Report

Merging #4 into master will increase coverage by 0.79%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   86.87%   87.66%   +0.79%     
==========================================
  Files           6        6              
  Lines         655      681      +26     
==========================================
+ Hits          569      597      +28     
+ Misses         86       84       -2
Impacted Files Coverage Δ
...ionCodeTests/OpenLocationCodeAdditionalTests.swift 100% <100%> (ø) ⬆️
Source/OpenLocationCode.swift 93.83% <100%> (+0.62%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2e16b0d...6f8cb47. Read the comment docs.

@WilliamDenniss WilliamDenniss force-pushed the max-digits branch 4 times, most recently from f90ffae to c4c7669 Compare April 23, 2019 22:34
As documented in OLC issue number 190:
1. If you ask the library to encode a lat/long with more than 15 digits, the library will return a plus code with 15 digits.
2. If you ask the library to decode a plus code with more than 15 digits, the library will only take the first 15 digits into account with the returned code area.
3. Plus codes with more than 15 digits will be treated as valid if all the other validity constraints are satisfied. If the plus code has an invalid character after the 15th digit, it will be treated as invalid.
@WilliamDenniss WilliamDenniss changed the title Implement max digit behavior Implement max length behavior Apr 23, 2019
@WilliamDenniss WilliamDenniss merged commit f16dde2 into google:master Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants