aleleba/lightcraft
1
0
Fork 0
mirror of https://github.com/storytold/lightcraft.git synced 2026-11-01 05:57:09 -06:00
Code Issues Packages Projects Releases Wiki Activity

LR-META-LOCATION: check prefix GPS split ambiguity

Browse Source
This commit is contained in:
ronibalthazar
2026-10-24 18:19:28 +01:00
parent 64712276bc
commit 96d7a38b1f
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/engine/src/cmd/library.rs
+2 -2
View File
@@ -1837,11 +1837,12 @@ pub(crate) fn parse_gps(t: &str) -> Option<(f64, f64)> {
// A suffix N/S marks the end of latitude; a prefix N/S uses the next hemisphere prefix as
// the boundary. If only longitude has a hemisphere, the whitespace pass below handles it.
let mut parsed: Option<(f64, f64)> = None;
let mut latitude_markers = t.char_indices().filter(|(_, c)| matches!(c, 'N' | 'S' | 'n' | 's'));
if let Some((first, marker)) = latitude_markers.next() {
if first == 0 {
if let Some((second, _)) = t.char_indices().skip(1).find(|(_, c)| is_gps_hemisphere(*c)) {
return parse_gps_pair(t.get(..second)?, t.get(second..)?);
parsed = Some(parse_gps_pair(t.get(..second)?, t.get(second..)?)?);
}
} else {
let end = first.checked_add(marker.len_utf8())?;
@@ -1852,7 +1853,6 @@ pub(crate) fn parse_gps(t: &str) -> Option<(f64, f64)> {
// Without hemisphere letters, a comma-free pair is separated by whitespace. Trying each
// boundary also permits spaces between DMS components without letting unconsumed text vanish.
// If multiple boundaries parse, they must agree so ambiguous text cannot be reinterpreted.
let mut parsed: Option<(f64, f64)> = None;
for (at, separator) in t.char_indices().filter(|(_, c)| c.is_whitespace()) {
let after = at.checked_add(separator.len_utf8())?;
if let Some(pair) = parse_gps_pair(t.get(..at)?, t.get(after..)?) {