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)

remove remnants of zombie mapbar. (#907)

this format was removed in #870 dd320d39acfc7e3ebb57df04f3a52e0350cc6bac

retire html_entitize, xml_entitize. (#906)

* retire html_entitize, xml_entitize.

in favor of QString::toHtmlEscaped() and XmlStreamWriter.

* delete obsolete entity struct.

convert html writer to html 5. (#905)

this also fixes a bug where we could fail to entitize wpt->description.
this also fixes a bug where we could create illegal fragments and id
attributes.

convert html writer to textstream. (#904)

update text writer to use a textstream. (#903)

* convert text writer to textstream.

* tweak text writer

deleted obsolete osm declaration. (#902)

use xmlstreamwriter with osm format (#901)

* convert osm writer use xmlstreamwriter

* use xmlstreamwriter for osm.

* add missing reference files for osm test.

* fix time zone test issues with osm writer test.

* use writeEmptyElement in osm writer

change return type of strip_html, strip_nastyhtml to QString. (#900)

change return type of convert_human_[date|time]_format to QString. (#899)

change return type get_cache_icon, gs_get_cachetype, gs_get_container to QString. (#898)

return QString from html_entitize, xml_entitize. (#897)

change return type of pretty_deg_format to QString. (#896)

this fixes memory leak in garmin_gpi, and adds a test.
enhance testing of pretty_deg_format to get coverage of dms.

use QStringView with namepart parameter in add wpt functions. (#891)

* use QStringView with namepart parameter in add wpt functions.

* update includes

Merge pull request #892 from tsteven4/tidyup

various clang tidy fixes

clang tidy modernize-raw-string-literal

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

clang tidy readability-convert-member-functions-to-static

manually removed now unnecessary Functors.

clang tidy readability-duplicate-include

Merge pull request #889 from tsteven4/qstringliterals

string optimizations

use utf-16 string literals with [QDateTime,QDate,QTime]::toString format parameters.

use QStringLiteral with constant QStrings and QString::arg(*).

retire qsort in favor of std::sort, std::stable_sort (#888)

* replace qsort with std::sort or std::stable_sort

* add test for duplicate filter using gc_data exported dates.

* fix reference file permissions.

* fix smplrout sort bug that shows up on macos with std::sort.

the compare function has always generated incorrect values when
both objects are HUGEVAL. The historic function worked with qsort
everywhere, worked with std::sort on linux and windows, but failed
with std::sort on macos.

Remove stray spaces from reference files, code that put them there. (#885)

* Remove spaces from generated help messages, reference files.

Final deprecation of: cup dmtlog dna igc itracku navitel nima raymarine s_and_t tef_xml tef_xml teletype teletype tomtom tomtom_asc tomtom_itn tomtom_itn_places wintec_tes wintec_tes xol (#884)

Second wave of Feb 28 gpsbabel-misc discussion, "RFC: Removing little used..."

add option to transform filter to delete creation times (#882)

* add option to transform filter to delete creation times

* document grammar corrections.