search

felixc / rexiv2

Rust library for read/write access to media-file metadata (Exif, XMP, and IPTC)
GNU General Public License v3.0
79 stars 17 forks source link

Make altitude optional in GPS info #44

Closed jonas-hagen closed 1 year ago

jonas-hagen commented 3 years ago

This implies: For setting GPS info, the three components need to be supplied explicitly and separate. This is not ideal because of the asymmetry in get / set GPS info. Since gexiv2 does not export a method to set the coordinates separately, this is the best option if altitude shall be optional in GPS Info.

See discussion in #42 (alternative solution in #43 ).

felixc commented 1 year ago

Thank you for taking the time to put together this option for fixing the bug. I've merged https://github.com/felixc/rexiv2/pull/69 which takes a similar approach, so I think this is now redundant. However if you feel this version has a better API or anything like that, please feel free to discuss on #42 and we can figure it out!