fix kml cppcheck warnings. (#1077)

Operator '|' with one operand equal to zero is redundant.

fix some MISRA 20.7 warnings (#1076)

* fix some cppcheck warnings MISRA 20.7

and use QStringView in gdb writer routines.

* migrate gdb macros to static constexpr.

* name gdb constants per google style guidelines.

keep gdb route point names and waypoint names aligned. (#1075)

This used to work in 1.4.4, but stopped working with the conversion
to QString (NEW_STRINGS).

Add a test case to verify the names match.

introduce route_head::empty() (#1074)

exercise qt 6.5.0 on macos CI (#1073)

fix cppcheck constVariable warnings (#1070)

* fix cppcheck constVariable warnings.

* guard against qt container detachement.

use c++ style attribute syntax with gnu namespace. (#1069)

* use c++ style attribute syntax with gnu namespace.

cppcheck 2.10.3 seems to have difficulty combining c++ style
attributes like [[noreturn]] with gnu __attribute__ syntax.
this leads to lots of false positive [nullPointerRedundantCheck]
warings.

I have a feeling I did this before and it failed somewhere.  Perhaps
our compilers are all ready for it now.

* fix 7 real nullPointerRedundantCheck warnings.

and
7 nullPointerArithmeticRedundantCheck warnings
1 redundantInitialization warning
1 unreadVariable warning

* fix 1 nullPointerRedundantCheck.

A more involved fix would be to pass a reference to a non-const
parameter to mkshort_del_handle to be used as an in-out parameter.

* fix 1 nullPointerRedundantCheck

* workaround cppcheck bug with noretun.

de-duplicate format and filter vector display code. (#1068)

* de-duplicate format and filter vector display code.

* consolidate release of argument memory in vecs, filter_vecs.

add support for dynamic filters (#1008)

* support dynamic filters.

* delete static ResampleFilter.

* make some more filters dynamic.

* parrot Vecs::find_vec in FilterVecs::find_filter_vec.

---------

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

use constexpr if for trackfilter debug (#1067)

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

reference/paris.wpo was part of Holux-gm100 and no longer needed.

Remove holux gm-100 format. (#1065)

Following through with plan announced Feb 28, 2022.

This was a serial unit from the turn of the century. Holux is out of business.
We've done nothing but mechanical maintenance on this format for years.

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

Removing baroiq and cambridge formats. (#1066)

* Remove Cambridge and Baroiq formats as per Feb 28 RFC.

Make locale debugging a compile-time constexpr debug instead of #ifdef. (#1011)

* Make locale debugging a compile-time constexpr debug instead of #ifdef.

* Ignore *.a in git so a branch change work more effortlessly on MacOS. (Yeah, not part of this CL...)

Co-authored-by: GPSBabel <12013583+GPSBabelDeveloper@users.noreply.github.com>

Update interpolate-time.xml (#1021)

Important to explicitly show fractional seconds are OK.

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

Update interpolate-distance.xml (#1022)

#1021 we gave sub-second time examples. So we had better give sub-mile distance examples too!

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

Make track filter debugging an internal compile-time consant. (#1012)

Nobody says it the time is "12:3:3", they say 12:03:03 (#1044)

There are also additional bad / non-realistic examples on this page which I didn't correct, but you should.

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

#1062 Fix core dump when merging tracks with disparate IGC extensions (#1063)

* #1062 Fix core dump when merging tracks with disparate IGC extensions

* Reduce `value` variable scope when writing SimpleArrays

* Debug oopsie bugfix, fix core dump when merging IGC files with no extensions

* Collapse redundant if statement

* Fix brain fart

* Eliminate igc_fsdata::has_value()

Extract engine noise from IGC, add to KML multipath (#1054)

* Extract engine noise from IGC, add to KML multipath

* Add IGC tests and test files

* Fix kml test suite for extension data.

Note that reference file are stripped of creation times.

* Better way to figure out which extensions are present?

* New IgcMetaData and ExtensionDefinition classes

* Lots of new extensions

* This looks like a better way...

* Minor cleanup

* Dead code removal

* Update IGC->KML reference KML files

* Replace QHash with QList, eliminate evil if ladders

* Many small improvements and cleanups

* Many small improvements and cleanups

* Precision loss fixes, igc_fsdata now templated

* More precision fixes, less templating

* format with astylerc options.

* delete unused include

* convert KmlFormat::wp_field to scoped enum.

* add missed file with scoped enum refatoring changes.

---------

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

Disallow 'fields' when writing unicsv (#1059)

This closes https://github.com/GPSBabel/gpsbabel/issues/1042

At Qt6 build to Fedora CI (#1058)

* add Qt6 build to fedora 37

fedora finally provides Qt6 QWebEngine.

* and library for Qt6 find_package.

this prevents a cmake generation warning about xkb.

use 6 digits after the decimal point for osm_name_from_wpt. (#1053)

I beleive this restores the behavior from before commit 4673a25,
August 4, 2013.

This should resolve #1047.

support millisecond track moves (#1036)

* support millisecond track moves

* add another position to track move option.

avoid sed encoding issues with dop filter test. (#1037)

Qt 6.5.0 beta is generating warnigs due to
https://github.com/qt/qtbase/commit/fff9395c2899841bdca07f400392595963d9ff93

Detected system locale encoding (ANSI_X3.4-1968, locale "C") is not UTF-8.
Qt shall use a UTF-8 locale ("C.UTF-8") instead. If this causes problems,
reconfigure your locale. See the locale(1) manual for more information.

document spelling fixes. (#1035)

* document spelling fixes.

and deciphering of the arc filter project option interpolation.

* improve line break insertion in build table.

* reword some option documentation.

* whitespace

enhance exif subsecond and offset handling. (#1033)

* enhance exif subsecond and offset handling.

* don't warn for legal empty offset tags.

document fixes (#1034)

* fix lowrancusr document

with fop we got a bunch of warnings:
... exceed the available area in the inline-progression direction ...

spelling

* add USES_TERMINAL for docs so we get output as the doc generates.

Merge branch 'master' of https://github.com/GPSBabel/gpsbabel

Make single arg ctor for UrlLink explicit

don't create fictious course and speed values. (#1031)

We compute coures and speed in track_recompute, which is used
by the kml and gtrnctr writers.  We also compute these in the track
filter with the course speed options.

Both of these algorithums compute the course and speed for a point
based on the time and position of that point and a previous point.
Given that no information from the future is used, the course and
speed of the first point cannot be determined.  This PR introduces
a change so a fictious value for the first point is not generated.

use jfrog docker repo and ninja for codeql. (#1030)

remove some orphan gpssim reference files. (#1029)

* remove some orphan gpssim reference files.

* remove another orphan.

support subsecond resolution for trackfilter start/stop options. (#1026)

* support subsecond resolution for trackfilter start/stop options.

* de-ickify track filter start/stop implementation.

Remove gpssim formet per 02/28/2022 proposal. (#1028)

Clarify how to write feet in fmt_unicsv (#1018)

* Clarify how to write feet

For #1015.
By the way e.g., "height" oddly only mentions meters so I do the same.

* tweak unicsv keywords for feet.

---------

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

wrap long lines in verbatim elements, such as screen. (#1027)

The use of inline.boldmonoseq on verbatim elements such as screen
results in a lose of the line endings. This makes it impossible to
differentiate items in what might have better been a list.  However,
if the line in the screen element is too long it will be trucated.
Use the monospace.verbatim.property wrap-option to wrap any such lines.

An example that illustrates the problems is the list of keywords in the
unicsv format.

correct input validation for interpolate time option. (#1025)

* fix input validation on interpolate time option.

* update filter info for interpolate time option.

Make Navilink debugging runtime selectable. (#1013)

Work on grtcrc to be a little more functional and C++ style. (#1007)

add support for dynamic formats (#1005)

* de-duplicate format setup in find_vec

* create a class to pass format info pack and forth from vecs.

This simplifies re-initialization of a format, which main already
did in a few cases.

The operator overloads for the format info class make the transition
simplier.

* return empty braced init list from find_vecs when not found.

* dynamic format checkpoint.

* delete static GeoFormat instance.

* support dynamic xcsv format.

allow dynamic formats to use rd_init, rd_deinit, wr_init, wr_denint.
This makes conversion to dynamic formats easier as we can, but aren't
forced to move the old init/deinit code.
Often rd/wr_init is intertwined with rd/wr_posn_init.

use dynamic xcsv format.
fix undiscovered bug releated to option order. We assumed an order
of -i -f -o -F, but -i -o -f -F can cause errors if both formats
are style based.
test for above bug added to iblue747.test.

* add missing reference

* use template for format factories.

* correct vecs includes

* fix another filename parameter for positional args issue.

* make Vecs::prepare_format static.

* move prepare_format call to just before xxx_init

for both static and dynamic formats.

Remove remaining callers and definition of rtrim. (#1001)

Remove remaining callers and definition of rtrim.

* Minor CV-qualifier battle in jeeps for const correctness.
* Adds a (hacky) bash script to compile mkshort as a standalone binary for
development and testing for upcoming overhaul.

---------

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

Update gpsbabel_man.xml (#1003)

Fix critical mistake, overlooked for years.
(Sure hope I am editing the real source file.
There is no warning at the top of this file that it is machine generated.)

de-duplicate format setup in find_vec (#997)

* de-duplicate format setup in find_vec

* create a class to pass format info pack and forth from vecs.

This simplifies re-initialization of a format, which main already
did in a few cases.

The operator overloads for the format info class make the transition
simplier.

* return empty braced init list from find_vecs when not found.

Remove ltrtim(). No callers remain.

create wpts from kml placemark/multigeometry/points. (#999)

amazingly enough earth offered up a kml from a search and we
couldn't recover the point.

convert geo to Format class. (#998)

This also adds the encoding attribute to the xml declaration.
This also starts the loc element at the beginning of a line.
Previously we jumped through some hoops to avoid the above two
changes so we could exaclty match the original reference file.

We also pass the writer to the waypt_disp_all callback via a lambda.
This allows the output File and QXmlStreamWriter to be local which
simplified cleanup.

convert igc to Format class. (#996)

and fix a few bugs.

eliminate use of time_t etc in igc. (#995)

eliminate some time_t uses. (#994)

* eliminate some time_t uses.

* whittle down time_t

* move some gpi time handling to Qt.

* drop include comment on defs.h

use bool type (#992)

* use bools instead of ints.

* another bool parameter.

Enable the gpx reader to use the declared namespace prefixes for recognized namespace URIs. (#990)

* use the declared ns prefixes in the gpx reader.

* remap other used uris, even though we have never

seen a different prefix used with them.

delete GARMIN_RTEPT_EXT, which was never used and appears to have
never existed.

Update trackfilter.h (#987)

* Update trackfilter.h

Also mention exact matching timestamps. Proven by
https://github.com/GPSBabel/gpsbabel/issues/981#issuecomment-1380515460

* update reference files for changed help text.

and fiddle help text

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

macOS brew install fix (#986)

* optimize brew install time.

* try skipping brew update to speed up install

* is jing-trang really the problem?

* skip relink step

* avoid brew python link issues and dependency upgrades.

This cuts the brew install time down about 10x.

fix kml balloon trailing space (#984)

* kill trailing space in kml popup balloons.

* fix old FIXME with end document

that requires regeneration of references.

The change in LineStyles.kml is different handling of a vertical
tab.  This changed when we let Qt cleanup illegal xml characters
long ago.

Update kml-prec.xml (#979)

* Update kml-prec.xml

I am guessing, based on the same sentence in gpx-elevprec.xml, what you meant to say.

* correct description of kml prec option.

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

Update kml.xml (#980)

Fix typo.

eliminate some uses of xstrdup/xstrndup. (#977)

and fix another related memory leak!

remove some unused code. (#976)

mimic std::optional for flagged Waypoint fields. (#975)

* mimic std::optional for flagged Waypoint fields.

* manual adjustments to waypt access usage.

* split optional Waypoint flags, tune packing.

* bring op_flags into Waypoint.

* add comments about future Waypoint optional fields.

* sort Waypoint class almost normally.

private members refered to with decltype need to be defined
before they are referred to.

* remove extra parens around *has_value().

* remove extra parens around *value_or().

* remove more extra parens.

another use of potentially invalid waypt field. (#974)

* another use of potentially invalid waypt field.

* fix erroneous comparison of optional waypt values.

fix errors handling optional Waypoint fields. (#948)

* fix errors handling optional Waypoint fields.

1. Hide wp_flags from formats by using WAYPT_SET.
2. Don't print values corresponding to Waypoint fields that aren't
valid.
  a) if the field uses wp_flags, then the flag must be set.
  b) power, cadence heartrate must not be zero.

This is in anticipation of use of std::optional instead of wp_flags
to indicate an optional value is present.
The use of invalid power, cadence, and heartrate values was found
fortuitously.

Note this changes kml output, using empty gx:value elements to
correspond to missing data.  gx:value elements are of type
string, the empty string do not violate the schema (unlike empty
when elements).  However, it is unclear who is using this extra
SchemaData so we cannot check to make sure they acutally handle
empty string values.

* don't use WAYPT_HAS with wp_flags that aren't optionals.

* fix some bugs with our home grown optionals.

mostly not using WAYPT_SET resulting in the value being lost.

remove encoding, fixed encoding data from formats. (#973)

remove remaining shadows of cet (#971)

* wean garmin_gpi from global_opts.codec and cet.

* wean garmin from global_opts.codec and cet.

* remove cet

* don't include deprecated includes, no global codec.

* fix garmin includes

* fix up comment indicating garmin encoding issues.

* fiddle with encoding functions.

* change get_codec signature.

* trailling white space

enforce exclusivity among sort options (#970)

* enforce exclusive sort options.

* use default brace inits for sort members.

modernize units (#969)

* modernize units

fmt_* returns a std::pair of the value and the units.
use structured binding to unpack the pair.
use scoped enumeration for the units system.

* make units formatter a class.

* cleanup unitsformatter instance.

Make geocache a real class. (#968)

fix clang diagnostic cast-qual (#967)

* fix cast-qual clang diagnostic

* use const_cast to cheat

* decay an array with casts.

the assembly code for gpsusbwin and gpsusbcommon does match the master branch.

* explicitly use const with auto*

use enums for mutually exclusive options in simplify filter (#966)

* clean up simplify options

* tweak names

* fiddle with simplify test trying to be plaform neutral.

* use stable sort when testing simplify filter

for cross platform matching.

odds and ends (#964)

* odds and ends

support subseconds in exif GPSTimeStamp. While rare, some devices
provide subsecond information.

add missing include guards.

use cmake for mkicondoc, retiring the last qmake build.

provide default initializers for a couple of data members.

kill some empty statements.

* Dont' use case_ignore_strcmp for two QStrings.

Use QString::compare(QStringView, Qt::CaseSensitivity)  (#965)

* use QString::compare(QString, Qt::CaseSensitivity)

instead of case_ignore_strcmp with a QString and a character
literal.

* use QString::compare(QStringView, Qt::CaseSensitivity)

instead of QString::compare(QLatin1String, Qt::CaseSensitivity).

* compare QStrings to character literals instead of string literals that only contain one character.

move gbfile.h, zlib.h out of defs.h. (#963)

* move gbfile.h, zlib.b out of defs.h.

* fix comment

cleanup datum lookup & constants (#962)

* cleanup datum lookup

* use constexpr for datum contstants

Obsolete Magellan serial and SD formats (#961)

* Remove Magellan serial and SD formats. Two different polls (one on the list, one in groundspeak forums) have confirmed disuse.

[WIP] Modernize garmin_txt Take the few remaining good ideas from https://github.com/GPSBabel/gpsbabel/pull/442 and merge them manually into the trunk. (#959)

* Modernize garmin_txt
Take the few remaining good ideas from
https://github.com/GPSBabel/gpsbabel/pull/442
and merge them manually into the trunk.

move xml tag utilities to xml_tag.cc (#960)

* move xml tag utilities to xml_tag.cc

they are already declared in src/core/xmltag.h

* move xml_tag.cc to src/core, where it's h file is.

* move global xml_tag routines into class.

* make xml_next non-static.

* make xml_findnext non-static.

* make xml_findfirst non-static

* make xml_attribute non-static

* eliminate some shadowing

* rename XmlTag class.

* retire fs_xml_alloc.

* use QStringView with XmlTag.

* back off QStringView for xml_attribute for Qt 5.

Remove all uses of sprintf (#958)

Deprecate sprintf in favor of snprintf or QString::asprintf... This buys time until we have {fmt}

* remove unused bgr in tpo.cc
* more pruning of C-style constructs
* Remove unused lap code in Garmin
Unrelated, but required to build on Mac
* Apply Mac runner updates per active topics from runner images bugreports.
It's my belief there are transient problems in the GitHub-provided images (linked in this CL) but this bulletproofs our installation, so I'm rolling with it.

Drop incomplete html_encrypt option in kml writer (#956)

add zlib license (#954)

get dependabot.yaml in correct location (#953)

add dependabot for github actions. (#952)

update zlib to 1.2.13. (#951)

add power to kml track descriptions and balloons. (#950)

fix document generation on fedora (#949)

* try to find common docbook url for fedora/ubuntu/homebrew catalogs.

* fedora builds

jing is broken on f32, remove it so it doesn't kill our document build.

add f37, with Qt5 because fedora doesn't have Qt6 webengine yet.

add jing for fedora builds. (#942)

* add jing for fedora builds.

fedora f35 and on don't have a maintainer and thus the package
isn't available.  kludge in jing on f35.

* improve document validation.

* don't force jeng on f35

Merge pull request #947 from tsteven4/tweaks

minor improvements, mostly clang-tidy related

fix some tidy llvm-namespace-comment

manually.

fix rounding in holux.

and fiddle with the make short default name so we can do a
roundtrip test.

fix tidy bugprone-reserved-identifier

this was partially automatic and partially manual.

fix tidy readability-container-size-empty

automatic fix.

fix tidy performance-type-promotion-in-math-fn

automatic fix.

fix tidy performance-unnecessary-copy-initialization

manual fix.  also fixes an instance of modernize-loop-convert

fix tidy modernize-deprecated-headers

tidy bugprone-incorrect-roundings

manual fix, passing rounding off to Qt.  Qt may do something
smarter, or it may do just what si_round used to do.

use structured binding instead of std::tie.

update macos build system (#946)

relax time constraints on trackfilter move. (#945)

1. don't require all track points to have time, just leave timeless
track points alone.
2. don't require track points to be ordered by time for move.

This also results in the tracks not being sorted by time for move
only operations.  Instead the original order is preserved.

enhance faketime option to allow subsecond steps. (#944)

doc content fixes. (#941)

update documentation to DocBook 5.0 (#940)

* convert to docbook 5.0.

* get jing running.

* use uri to find docbook rng.

* brew install jing-trang.  use oasis for rngs.

* add insertion markers to autogen doc files.

* use db5-upgrade.xsl from docbook 5.1 release.

we are still converting to docbook 5.0.

this fixes issues with missing Appendix labels
(e.g. "Appendix A. Supported Datums" was "Suppored Datums")
and Glossary Indexes (e.g. "G" was omitted)
in the html output.

unfortunately it causes issues with translating ulinks inside term
elements.  We patch the dropped links back in.

* make sure makedoc is using utf8 for files.