Closed tsteven4 closed 4 months ago
Lgtm
On Thu, May 23, 2024, 6:51 AM tsteven4 @.***> wrote:
- eliminate unused parameters
- pass gpsdevh* instead of gpsdevh** when function doesn't modify the original.
You can view, comment on, or merge this pull request online at:
https://github.com/GPSBabel/gpsbabel/pull/1283 Commit Summary
- 1272c32 https://github.com/GPSBabel/gpsbabel/pull/1283/commits/1272c32159eb8f269a24d7122fc1719056642e79 refactor interface to jeeps pvt related commands.
- eacd853 https://github.com/GPSBabel/gpsbabel/pull/1283/commits/eacd853b84b805d279c66e07bc1e12c9540a2f46 Merge branch 'master' into pvtinterface
File Changes
(5 files https://github.com/GPSBabel/gpsbabel/pull/1283/files)
- M garmin.cc https://github.com/GPSBabel/gpsbabel/pull/1283/files#diff-e56a68eebe1e26f2a24b2cd40ac885bb37c1f6368c1b040a3ad9be94e2229ada (2)
- M jeeps/gpsapp.cc https://github.com/GPSBabel/gpsbabel/pull/1283/files#diff-e192ae85322b3c91fef63752fd1b705f09e9fc0a0c6451ac13fa389dfb78ce26 (19)
- M jeeps/gpsapp.h https://github.com/GPSBabel/gpsbabel/pull/1283/files#diff-f53b978845836ae379e0abf80b3b22dacc7e80ad59ee164244f3ddf7b790fa73 (4)
- M jeeps/gpscom.cc https://github.com/GPSBabel/gpsbabel/pull/1283/files#diff-46a82d93690548f74659212096d75f85d12ff846927f4967bc43753a1ccf3b70 (11)
- M jeeps/gpscom.h https://github.com/GPSBabel/gpsbabel/pull/1283/files#diff-61d9c2137c39278af5dfd4382a75991059185b94cc410d87427567ee58f49123 (4)
Patch Links:
- https://github.com/GPSBabel/gpsbabel/pull/1283.patch
- https://github.com/GPSBabel/gpsbabel/pull/1283.diff
— Reply to this email directly, view it on GitHub https://github.com/GPSBabel/gpsbabel/pull/1283, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCSD34WDLDOTB2T37O2ZC3ZDXX6XAVCNFSM6AAAAABIFWUZ66VHI2DSMVQWIX3LMV43ASLTON2WKOZSGMYTEOJYHAZDOMQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>