introduce OptionString, (#1352)

* introduce OptionString,

a replacement for OptionCString. Formats and filters can be
manually converted, with the potential to eliminate c character
string usage.
OptionCString::get() usages can often use an implicit cast from
OptionString instead, although the usage of get() is still valid.

* more OptionCString -> OptionString conversion

* use custom conversion routines.

this introduces OptionString::toInt and OptionString::toDouble,
which enforce error checking on conversions.

* convert more formats to OptionString

* convert gdb to OptionString

* convert xcsv, unicsv to OptionString

* convert html, text to OptionString

* convert ozi to OptionString

* convert mtk to OptionString

* convert globalsat to OptionString

* convert exif to OptionString

* convert lowranceusr to OptionString

* convert igc to OptionString.

note the error checking on timeadj is now done by OptionString::toInt.

* prove igc timeadj works with ints.

* convert shape to OptionString

* convert garmin_gpi to OptionString

* consolidate integer/double parse routines.

* convert subrip to OptionString

* convert garmin_txt to OptionString

* enhance OptionString conversions.

overloads, as opposed to default parameters, allow tools to find differen usages.

improved OptionString error messages with module and argstring information.

* enhance argtype ...

to indicate integer base and if trailing data is allowed.
Use these fields to check and convert integer/doubles in Vecs, fataling on errors.

* allow trailng data with parse_distance, parse_speed.

and when necessary convert these from ARGTYPE_STRING to ARGTYPE_FLOAT.

* fix typo, update refs

* convert garmin to OptionString

* convert skytraq to OptionString

* Introduce OptionInt & OptionDouble.

These must be used with ARGTYPE_INT and ARGTYPE_FLOAT.
The data is checked and converted in Vecs, and the results may
be retrieved by the user.

* convert almost all ARGTYPE_INT to OptionInt.

* final conversion to OptionInt

* mark TODO done.

* correct include comments for greps sake.

* delete complete TODO comments.

* whitespace

* use consistent error messags regarding options.

i.e. "module(option): "

* delete unused Option method type

* use ARGTYPE_STRING if trailing data is allowed.

The GUI validator used with ARGTYPE_INT and ARGTYPE_FLOAT will
reject input with any trailing data.

The CLI, including Vecs::assign_option, keys off the Option subclass now,
not the ARGTYPE.

Note that OptionInt and OptionDouble still allow trailing data.  If
trailing data is allowed with these classes we just pair these with
ARGTYPE_STRING which controls validation in the GUI.

* drive arg validation from Option classes.

* simplify use of isEmpty

delete unused macro
delete extraneous file

* non member operator== with parameters const QString& not a candidate,

parameters are of class OptionString.

use correct compare in enhanced garmin_gpi test (#1360)

enhance subrip, garmin_gpi tests (#1359)

* add test for garmin gpi speed/proximity options.

* enhance subrip test

restore lowranceusr truncation in bytes (#1358)

as opposed to QChars.

Qtize garmin_tables (#1357)

Qtize rgbcolors. (#1356)

* Qtize rgbcolors.

* use big stick to validate hex color codes

delete unimplemented options (#1355)

* delete globalsat_sport track option

which was never implemented.

* delete ancient geo nuke_placer option

which hasn't worked since ~1.4.4

use consistent unit suffixes. (#1353)

* use consistent unit suffixes.

This is a user visible change. The suffixes for the distance
option of the arcdist, interpolate, radius and position filters
and the add option of the height filter are different.

* drop unused includes

* whitespace

* terminate fatal messages.

* change simplify to process distances in meters.

This simplifies option value parsing.
There is no user visible difference.

* define MYNAME in cc not h.

refactor option handling (#1351)

* rough in Option class

* testo passes

* rough in OptionBool

* more OptionBool use

* more OptionBool

* fix isEmpty

* store qstrings

* dont return const

* rename

* convert a few lost ARGTYPE_BOOLs to OptionBool.

* utilize avalable QString for options.

* obsolete argvalptr

* Option is abstract class

* eliminate special handling of ARGTYPE_BOOL without default in vecs.

* fix #982 ...

garmin, ozi, xcsv mkshort options whitespace_ok, must_unique broken since 2006.

* test to make sure OptionBool iff ARGTYPE_BOOL

* use OptionBool for all ARGTYPE_BOOL options.

* correct OptionBool type.

* delete unused parameterized Option ctors.

* format and comment option.h

* nodiscard for class Option.

* vecs nullptr checks

* have Vecs::validate_args check argval isn't nullptr.

* catch up includes

* clarify comment

* fix introduced bug caught by tidy.

Revert "fix an annoying Wstringop-truncation warning. (#1348)" (#1349)

This reverts commit a4dec0acfbeb6d93a9a73e3673da983bfdd7511b.

fix an annoying Wstringop-truncation warning. (#1348)

* fix an annoying Wstringop-truncation warning.

while the warning is fixed we can still index outside of the
gps_categories array :(

* solved stringop-truncation w/o format macro ...

constants, rely on integer promotion.
Also, guard against devices with more than 16 categories should
they exist.

further qstringification of osm (#1347)

* clean up code issues in osm.

including repeated char* -> QString conversions.

* add include

* qstringification

* remove cstring from osm

* fiddle with osm creator.

prepare for Qt 6.8 deprecations (#1346)

* fix Qt 6.8.0 deprecation warnings.

Also, the exif writer now prefers to establish a timestamp from
GPS_IFD_TAG_DATESTAMP/GPS_IFD_TAG_TIMESTAMP if they exist, as the
exif reader does.

* avoid installing *.debug_information modules

which appeared with 6.7.0

* centralize decision on lightweight time zones.

* correct sort order for includes, duplicates

* remove debug statement

* remove const from declaration.

* simplify lightweight timezone usage via

qt version depependent defined constants.

* fix for QTimeZone not a QLiteralType

* cleanup lightweight time constants.

* enhance exif offset test

* add missing reference

* fix test comment

* revert changes to exif writer regarding gps tags.

I beleive the original intent of the writer is to create the gps tags
from a waypoint.

* update qt modules

* qt 6.8 work

1. allow builds to succeed without webengine by default.  This is
needed by CI until aqt catches up.
2. macos minos change for 6.8
3. add macos 6.8.0 to CI, remove macos-12 build.

* avoid Wunused-const-variable

* add 6.8.0 to ci for windows

use containers for xml_init lists. (#1345)

use container for unicsv fields def. (#1344)

and cleanup an extraneous break from #1164.

catch gdb debug up with gbfile changes. (#1343)

* catch gdb debug up with gbfile changes.

* use "if constexpr()" for GDB_DEBUG to detect errors

even if not compiling for debug.

more QStrings in gbfile (#1342)

rewrite strip_nastyhtml, strip_html in Qt (#1341)

* rewrite strip_html with QString.

* rewrite strip_nasty_html in Qt.

and actually produce valid html:
1. the replacement for "<body>", "<!   >", is invalid.
2. leaving an html tag in causes the html format output to be invalid.

* cleanup comment xstrdup

* use regex for strip_html

* strip_html deletes other tags

* fix strip_html img tag handling

* Revert "fix strip_html img tag handling"

This reverts commit b0440f7e3729909fd51ecbbacdd7a7e4aa5d9198.

* Revert "strip_html deletes other tags"

This reverts commit 40fe2ef0153d8c2e6d53432b9184f8acffbab2ef.

* Revert "use regex for strip_html"

This reverts commit 677da95a77b335bfa686a35fc33e862f8b50f053.

* implement strip_html using QRegularExpressionMatchIterator.

* a little cleanup

* remove obsolete include

* take care to distinguish tags with common roots

like p, param, pre.

* suppress InvalidReads in qhash.

These are known to occur per comment in qhash.cpp.

* supress qhash false positive with libqt6core6/jammy-updates,now 6.2.4+dfsg-2ubuntu1.1 amd64

* suppress vg warnings on noble (intermittant).

* install qt core dbgsyms for valgrind suppression.

* kill space preceding newline when stripping html.

* use modernize-raw-string-literal

* valgrind suppressions for f40

* add symbols for fedora valgrind suppression.

remove obsolete mkshort test. (#1340)

The macro TEST_MKSHORT hasn't existed for some time.

fixes for pr1238. (#1339)

shapelib is handled in shapelib.cmake.
CSVFMTS_ENABLED is now unreferenced.

Remove XCSV_ENABLED, MAXIMAL_ENABLED, MINIMAL_ENABLED (#1238)

Remove XCSV_ENABLED, MAXIMAL_ENABLED, MINIMAL_ENABLED
---------

Co-authored-by: Robert Lipe <robertlipe@gpsbabel.org>

Update gpsbabel_man.xml (#1169)

1. add dash, else looks like old tar(1) command.
2. Fix grammar.

Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

bring back See You support (#1280)

I think someone was waiting for me to say no. Sometimes, "good enough" (for users with no representation here...) seems to be.
Co-authored-by: Robert Lipe <robertlipe@users.noreply.github.com>

Various modernizations in humminbird. (#1100)

* Various modernizations in humminbird.

- Make humminbird_icons[] a QStringList.
- I don't think the icon finder ever actually worked the way it was described in the
comments.
- Comment fix in exif
- Kill xstrndup (yay)

modernize some dynamic memory usage. (#1334)

* modernize some dynamic memory usage.

* refactor convert_human_[date|time]_format

* convert convert_human_*_format to Qt.

* fix include comment

* use back instead of *rbegin to reference last QChar in QString

use persistant docker container for GUI (#1338)

update packages for jammy, noble. (#1337)

create docker images with historical releases. (#1336)

* create docker images with historical releases.

* add scripts to automate image creation

* enhance test script to avoid cwd dependencies.

* move 1.7.0 base from focal to jammy

* make sure to clean up container

* make sure container is cleaned up.

* fix varaible use in arithmetic context

* delete unused archive from images.

* don't confuse our zlib and system zlib

* use system zlib shapelib for 1.8.0, 1.9.0

* add script to run GUI

* fix up 1.5.x plugins.

these were probematic for 1.5.4, previous releases ran ok with webkit.
translations don't work for 1.5.x

* shellcheck

* enhance GUI run script to forward LANG

* add bleeding edge image.

Merge pull request #1335 from GPSBabel/tidymore

clang tidy fixes

fix tidy modernize-use-nullptr

fix tidy readability-duplicate-include

fix tidy modernize-raw-string-literal

fix tidy readability-avoid-const-params-in-decls

optimize [QDate|QTime|QDateTime]::fromString format literals. (#1278)

* optimize [QDate|QTime|QDateTime]::fromString format literals.

These accept QStringViews in Qt6 which can be very efficiently
passed with UTF-16 character literals.

* optimize QDateTime::toString format literals.

This isn't new to Qt6, but we had a few we hadn't taken care of
yet.

* use QString::replace(QChar, QChar) instead of (QString, QString)

kill a position normilization clone caught by codacy. (#1333)

normalize waypoint position. (#1331)

prevent assertion failures in nmea reader (#1332)

* replace all empty nmea fields with 0.

including adjacent empty fileds, i.e. ",,," -> ",0,0,".

* defend against indexing past end of arrays.

prevents assertion failures in qstring.h operator[].

also consistently default longitude direction.

Humminbirdhash (#1327)

* refactor humminbird handling of wpt numbers.

also:
convert some macros to constexpr.
consistenly use strncpy to move strings to humminbird char arrays.`

* fix clang-diagnostic-missing-field-initializers

clang-tidy-18 fixes performance-unnecessary-copy-initialization (#1328)

Zap repeated examples (#1084)

On https://www.gpsbabel.org/htmldoc-development/fmt_globalsat.html
we observe these two examples are repeated further down, in context too. So they shouldn't also
appear higher up.

I am guessing that I have correctly zapped the higher up pair.

Co-authored-by: Robert Lipe

simplify osm_tag option processing. (#1325)

I suspect the check of "tagnd" was used in developement before
opt_tagnd was added.  The only thing the check did was ignore
opt_tag values that started with tagnd, they weren't used elsewhere.

add fix & test for humminbird route writer. (#1324)

The missing initialization was introduced in 824e01d.

update shapelib to 1.6.1 (#1323)

Modernize Humminbird internals (#1322)

* Modernize Humminbirds internals

---------

Co-authored-by: Robert Lipe <robertlipe@gpsbabel.org>

use implicit conversions for lat,lon degs<->rads in grtcirc interface (#1321)

* use implict conversions for lat,lon degs<->rads.

* use Position* return values.

* convert grtcirc i/f to PositionDeg, PositionRad.

* fix mac compile, denote converting ctor.

* inline PositionDeg/Rad converting ctors.

* respect PosotionX encapsulation.

* differentiate member names for PositionDeg, PositiionRad.  This can catch unintended usages where the type is not what was expected.

* clean up unneccesary usage of Waypoint::position.

* delete obsolete declaration.

* spell

* update garmin include

* consistenly unpack Positions to consts in grtcirc.

* provide default initializers for PositionX.

fix bend filter usage of linepart. (#1320)

* fix bend filter usage of linepart.

* fix new bend filter test.

* make new reference files valid kml.

* debug macos fails

* workaround rounding errors in problematic kml lookat range.

cleanup great circle. (#1318)

* cleanup great circle.

* fix whitespace

correct simplify filter with maximum allowable error option and the metric being overall route length or relative error (#1316)

* add test for simplify w/ relative+error opts.

* fix simplify filter error+length operation.

* fix simplify relative+error

* remove debug statement

* simplify error unit tests and documentation.

* format parse, update includes.

* prune redundant test case

have jeeps use std::endian (#1315)

* have jeeps use std::endian

* detect unsupported endian platforms.

move floor to C++20 (#1206)

* move floor to c++20

drop Visual Studio 2017.
use default member intializers for bit-fields.

* use more bit-field default initializers

* try Xcod 14.3.1 on macOS 13.

* echo clang version

which can be useful with homebrew fails_with

* use std::numbers

* finish eradication of M_PI.

* fix some whitespace

use constexpr for constants, match parms in declaration and definition. (#1314)

* use constexpr for constants, match parms in declaration and definition.

* increase precision of GPS_PI.

Use Helmert transform for conversions between OSGB36 and WSG84 (#1311)

* use Helmert transform for OSBG36 <-> WGS84.

* adjust unicsv_grids reference for Helmert xform.

* update copyright.

* use constexpr to select helmert inversion method.

* tweak helmert.

spell Datum (#1313)

unify access to ellipsoid parameters (#1312)

* define constants for often used spherioid parms.

* add function to calculate semi-minor axis.

* use semi major axis function more.

* have semi_minor_axis consume an ellipse

* compute semi minor axis in GPS_Ellipse class.

add constants for other ellipses that are used directly.

correct WGS84 semi minor axis value. (#1310)

It appears the GSR80 value was used instead of the WGS84 value, which
is a very small error.
However, there was a typo in the value in GPS_Math_XYZ_To_WGS84LatLonH
that resulted in an order of magnitude error!
This could lead to GPS_Math_XYZ_To_WGS84LatLonH never converging.
This may be why skytraq created their own function to convert ECEF
to WGS84.

cleanup datetime.h tidy and g++ warnings. (#1309)

Wsign-conversion
readability-implicit-bool-conversion
readability-else-after-return
cppcoreguidelines-avoid-magic-numbers,readability-magic-numbers

enhance validate filter to report on track segments. (#1308)

use a signed type for waypt_counts and route counts. (#1307)

* use a signed type for waypt_counts and route counts.

Although larger in Qt6, the underlying type is signed.

* fix -Wformat-signedness warnings introduced by this PR.

* fix new -Wsign-conversion warnings.

teach lowranceusr about track segments (#1306)

* teach lowranceusr about track segments.

lowranceusr versions 2 & 3 have a continuous flag used to
indicate dicontinuities in a trail.
The lowranceusr version 2 & 3 reader now flags trail discontinuities
as new track segements.
The lowranceusr version 2 & 3 writer now flags new track segments as
discontinuities.

Also fix two subtle bugs:
1) when reading a version 2 or 3 trail with the break option we would
fail to break the trail if the discontinuity was at the start of a
section other than the first section.
2) when writing a version 2 or 3 trail with the merge option if the
number of points exceeded MAX_TRAIL_POINTS we would write a header
that indicated MAX_TRAIL_POINTS but actually write MAX_TRAIL_POINTS + 1.

* add lowranceusr test for track segments.

Update Russian translation (#1304)

* Update Russian translation

* Update Russian translation

correct garmin real time altitude, add geoid height. (#1303)

correct computation of timestamp with garmin real time positioning. (#1302)

* correct computation of timestamp with garmin real time positioning.

* tweak garmin pvt conversion

fix gpsapp copy_char_array that could leave part of a packet uninitialized. (#1301)

* fix gpsapp copy_char_array that could leave part of a packet uninitialized.

* tidy up a bit.

* delete empty statement

fix memory leaks writing garmin routes. (#1299)

enhance garmin route write for devices using 16 member icon set. (#1298)

enhance garmin route writes for devices using D201_Rte_Hdr_Type. (#1297)

which use GPS_PWay->rte_cmnt for the route name.

don't send links with GPS_A200_Send. (#1295)

fix faulty assertion in garmin. (#1290)

the '-' character is legal for some devices, but was
excluded in the assertions as it can be a metacharacter.

remove qtci installer code (#1289)

* remove qtci which hasn't been used for a long time.

* remove another obsolete qtci file.

* remove another qtci reference.

* delete obsolete travis build code.

* rename osx install script

* catch ci up with osx travis install script renames.

* remove obsolete debug code from qtio installer.

exclude qt .ts files from codacy analysis. (#1288)

they confuse eslint which thinks they are TypeScript.

Fix unicsv reader delimiter detection (#1287)

* Fix unicsv reader delimiter detection.

Any delimiter sequence that is enclosed has never been considered
a delimiter when splitting a line into fields.  However, previously
an enclosed potential delimiter could be detected when scanning the
first line, and then erroneously used as the delimiter.

* tweak auto

* add unicsv delimiter detection test

delete obsolete jeeps functions that were never used (#1285)

since introduction in 2002.

update macos release to Qt 6.5.3 (#1284)

refactor interface to jeeps pvt related commands. (#1283)

fix xcsv RECORD_DELIMITER CR. (#1282)

add Qt's webengine translations to package. (#1277)

tidy readability-convert-member-functions-to-static (#1276)

clang-tidy modernize-loop-convert (#1275)

clang-tidy readability-isolate-declaration (#1274)

Move floor to Qt 6.2 (#1272)

* move floor from Qt 5.15 to 6.2.

* fix some CI failures

* try harder to fix coverage build

* try to fix coverage again

* try agin

* move up msvc tools minimum for Qt6

* clean up clazy warnings with Qt6.

* drop redundnant CI job

* revert translation related unintentionl changes

* simplify find qt packages

* assume Qt6 in textstream

* move CI tidy to noble

* assume Qt6 in textstream.h

* remove obsolete makesetup.bat

* remove Qt5 support from coretool

* strip Qt5 from noble image

* update default qt versions.

* update minimum cmake version to match Qt 6.2.4 QT_SUPPORTED_MIN_CMAKE_VERSION_FOR_USING_QT

* assume QHash result is type size_t

* remove unnecessary qOverloads (in Qt6).

add Ubuntu noble to CI (#1273)

* add ubuntu noble docker support

* update qt6 packages for noble.

noble has some new qt6 packages that alleviate the need to
directly install some libraries.
also, the dependencies of some qt6 packages have been improved
allowing us to specify what we use more directly.
this results in the same set of packges being installed.

* remove dependent qt5 package

* add noble to CI

add fedora 40 to CI (#1270)

support builds with shapelib (as documented). (#1268)

update libusb to 1.0.27 (#1259)

work around homebrew python linking issues (#1267)

* debug python brew issues

* dbg2

* dbg3

* dbg4

* insulate workflow from homebrew location.

Add test coverage for ozi color setting options. (#1266)

* Unit test ozi color commandline options.

Improves test coverage.

Co-authored-by: Robert Lipe

update macos github workflow (#1265)

* update github macos runners.

macos-11 runner is now deprecated.

* update xcode versions for github

* update XML_CATALOG_FILES for macos-14 runner

* work around macos-14 runner brew locations

* find homebrew gsed on apple silicon

* generalize brew location detection in macos workflow

* update python version for arm availability

* push xcode versions to latest minor levels.

clang tidy 18 fixes readability-redundant-casting (#1262)

update zlib to 1.3.1 (#1258)

* update zlib to 1.3.1

* delete extra zlib files

update shapelib to 1.6.0 (#1257)

Bump actions/cache from 3 to 4 (#1256)

Bumps [actions/cache](https://github.com/actions/cache) from 3 to 4.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

update codacy badge (#1251)

update codeql workflow template (#1250)

Bump actions/upload-artifact from 3 to 4 (#1249)

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3 to 4.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](https://github.com/actions/upload-artifact/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bump github/codeql-action from 2 to 3 (#1248)

Bumps [github/codeql-action](https://github.com/github/codeql-action) from 2 to 3.
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](https://github.com/github/codeql-action/compare/v2...v3)

---
updated-dependencies:
- dependency-name: github/codeql-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

modernize google maps javascript (#1247)

to avoid:
"js: Google Maps JavaScript API has been loaded directly without a callback. This is not supported and can lead to race conditions and suboptimal performance. For supported loading patterns please see https://goo.gle/js-api-loading"

remove CMakeSettings.json (#1246)

which had machine specific windows configuration values.

Bump actions/setup-python from 4 to 5 (#1245)

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

make xmlgeneric a class (#1244)

* make xmlgeneric a class.

* encapsulate more of xmlgeneric

* use scoped enum

* encapsulate a bit more.

* retire xg_string type

* use scoped enums

* rule of zero applied to xmlgeneric class.

* remove test code

fix errors in gpi writer related to bitmaps. (#1243)

Erroneous warnings "garmin_gpi: Code error in load_bitmap_from_file,
expected output size 1060, actual output -28."

Corrupted colors when input bitmap has 24 bits per pixel.