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.

Don't make it easy to iterate the state_t enum...except when we do. (#939)

Merge pull request #938 from tsteven4/makedoc

update makedoc perl code

stay on macos11 for github actions a bit longer.

macos-lastest is changing to macos12.

fix perlcritic issues with makedoc.

use strict on makedoc.

fix makedoc warnings.

and:
die on errors with file creation, creating autogen directory, or
running gpsbabel.

use dirname to get directory portion of $0.  previously
"./xmldoc/makedoc" would fail as $dir got set to "." instead of
"./xmldoc". "perl xmldoc/makedoc" would work.

clean up makedoc with perltidy.

among other issues the indentation was insconsistent, and sometimes
just flat out misleading.

This establishes our perl style as that provided by the defaults
with perltidy.

document generation improvements (#937)

* build docs on macos CI.

* add fop for macos ci

* block out of source doc builds, change WEB default.

* add gnu-sed for macos fixdoc

* set default logger for fop

with homebrew fop 2.7 the default logger is something else besides
SimpleLog.
The LOGLEVEL statement is specific to SimpleLog.

Retire qmake. (#935)

* remove qmake.

* fix fedora cmake builds in CI.

* update build instructions in doc.

* more qmake cleanup

* remove rc stringification nonsense

* tweak build instructions.

Qmaketocmake7 (#933)

* packaging improvements.

parse package_app arguments.
add fields to macos info.plist.
pass copyright year down.
make windows workflows match.

* fix windows rc copyright

* automate copyright date for gui in windows.

update workflows (#932)

* move github workflows towards cmake.

* update action versions.

* update actions

* add workflow dispatch to all workflows

* remove travis

fix skipping of kml validation with valgrind. (#930)

use cmake for ubuntu CI, CMakeFileLists tweaks. (#929)

refactor ubuntu workflow using a matrix.
create gpsbabelfe_lupdate, gpsbabelfe_lrelease targest on all platorms.
add USES_TERMINAL to check-vtesto so we can see test progress live.
update docker images for focal and jammy for Qt5 cmake issues.
update docker image for jammy adding Qt6.  This uses a PPA to work
around https://bugs.launchpad.net/ubuntu/+source/qt6-base/+bug/1970057.
Note the PPA isn't actually required to build, but it is required to
run the GUI, i.e. all users on jammy using Qt6 who want to run the GUI will
need the PPA, or need ubuntu to incorporate the upstream debian fix!

move CI sanitizier, clazy, coverage builds to cmake (#926)

* switch linux CI extra_tests and coverage builds to cmake.

* add ninja to linux coverage install list

* add qt serial for coverage build

* add webengine for coverage build

* actually fail when ubsan finds an error!

* avoid divide by zero error in kml writer.

* teach camke that the check target uses the console.

this lets the test output show up as the tests run instead of
delaying it all until the end.

simplify cmake builds (#925)

* fix some Qt version dependencies.

* disable strptime warning.

* simplify CMakeLists.txt.

* update a macos build to Xcode 13.2.1 (clang 13)

* Revert "update a macos build to Xcode 13.2.1 (clang 13)"

This reverts commit 2bd252713607249683db3b2dae7452c481041850.

* Revert "disable strptime warning."

This reverts commit b75ca3882185d7d9e3b4fd288f2b847bf2f56bfd.

fix Qt version dependencies (#922)

* fix some Qt version dependencies.

* disable strptime warning.

prepare for deprecation of qmake. (#920)

* prepare for deprecation of qmake.

build a release level package in CI with cmake on windows & macOS.
this can be compared with the release package build in CI with qmake.

support cmake multi-config generators on macOS (e.g. Xcode) as well
as well as single-config generators (.e.g Unix Makefiles, Ninja).

support packaging of out of source builds on macOS when using cmake.
note that Qt6 support for packaging still has many features in
preview, thus we still use our script based packaging.

move some Ubuntu CI jobs to jammy.  Unfortunately Qt6WebEngine is
unusable on jammy due to a bug in qt6-base.  So we are still using
Qt5 :(
https://bugs.launchpad.net/ubuntu/+source/qt6-base/+bug/1970057

* isolate qmake and cmake flows for windows.

add on on demand CI workflow for esoteric windows cases.

* use qt tools from cmake prefix, add gpsbabel translation support.

* fix coretool on windows.

add qt binary directory to environment so coretool can find the
Qt DLLs.

* catch coretool cmake up with #921.

* update packaging dependencies

coretool fixes and enhancements (#921)

* get coretool to work on macos under qmake.

* enhance coretool.

1. don't use stderr for coretool output.  This avoids comingling of
real error messages with the coretool output.
2. disable the similartext heuristic with coretool.  This heuristic,
which is enabled by default, was resulting in occasional mistranslations.

github runner updates (#919)

* update github action for macos, windows.

* fix macos action

* try harder to fix macos action

* try harder yet

* simplify mac cache keys, no compiler dependency

Merge pull request #917 from tsteven4/taborder

set tab order for filter widgets.

set tab order for filter widgets.

Merge pull request #916 from tsteven4/issue915

fix GUI handling of track filter start/stop times.

use radio buttons for exclusive choice of timespec

for track filter start and stop times in GUI.

fix GUI handling of track filter start/stop times.

This resolves #915

The timeSpec of the QDateTimeEdit objects is maintained in
accordance with the TZCheck QCheckBox.  This solves additional
issues with DST transitions and edits.

fix maps javascript api deprecation warning. (#914)

js: google.maps.event.addDomListener() is deprecated, use the standard addEventListener() method instead: https://developer.mozilla.org/docs/Web/API/EventTarget/addEventListener

QStringize vecs a bit more. (#913)

This fixes an ancient FIXME regarding the use of the option name
to indicate a option was given without a value.

This enhances the validity checking of default, minimum and maximum
values.

replace atoi, atol, atof (#912)

* eliminate usage of atoi

which has possible undefined behavior.

* eliminate a couple more atoi usages.

* eliminate atol usage.

* eliminate use of atof.

delete reference files not used by testo (or whitelisted) (#911)

* delete reference files not used by testo.

the script tools/obsolete_unused_reference_files does this,
altough it might be delicate.
note any unused files in the reference tree will be deleted!

* restore used alternate reference file

* restore and whitelist some reference files.

* restore and whitelist a reference readme file.

* Revert "restore and whitelist a reference readme file."

This reverts commit 496fbf58d9b083863ae5011db4f8e48e044af546.

* use 'sort -u' instead of 'sort | uniq'.

* whitelist some reference files that are referred to in comments.

delete obsolete and unused document sections. (#910)

remove remnants of zombie f90g_track. (#909)

This format was retired in #868 8b7a8613bca79cbbd62aae5378b6073ec511bb99

Add missing qstartz_bl_1000 header to build generators. (#908)