more nits.

limit variable scope, found by LGTM.

Poor global variable name 'ops'. Prefer longer, descriptive names for globals (eg. kMyGlobalConstant, not foo).

Introduce resampling filter and nvectors. (#763)

* Introduce resampling filter and nvectors.

* use ellipsoid model with nvectors.

the code had been forcing a spherical earth model to match the
existing grtcirc.cc calculations.

improve string handling, fix 9 year old FIXME. (#769)

support generation of xcode project files with qmake. (#760)

update fit locations test with improved sample (#767)

* update fit test with improved sample.

* reference file mode correction.

Pick up fit locations test case (#766)

* Add support for Locations.fit files

* patch by tsteven4 to address review comments

* Add a test case for Locations.fit files from Garmin devices

Co-authored-by: Fabrizio Gennari <fabrizio.ge@tiscali.it>

add test for fit Locations(Lctns) files. (#765)

* add test for fit Locations(Lctns) files.

* fix new bug with change in Qt6 behavior for fit.

Add support for Locations.fit files (#762)

* Add support for Locations.fit files

* patch by tsteven4 to address review comments

Eliminiate redundant route_head data member (#764)

eliminate redundant route_head data member rte_waypt_ct, replacing it with a member function rte_waypt_ct() that gets the count from the waypoint_list container.

kill xmap, xmap2006. (#758)

* kill xmap, xmap2006.

* dprecate obsolete readme.

kill xmapwpt. (#757)

note is was wounded previously, see #754.

add fedora 35 to CI. (#756)

serial port names (#755)

* try qserialportinfo again.

* update docker images with qt serial port.

* enhance windows device text to match device manager on windows.

optimize speed and distance conversions. (#753)

* optimize speed and distance conversions.

structure constexpr conversions such that a minimal amount of
arithmetic is done at run time and avoiding run time division.

add tests for conversions.

* correct bug with display of accumulated path distance.

and add test case of same.

retire TopoMapPro Places File format (#711)

* retire TopoMapPro Places File format.

* remove topomap pro reference files.

* catch serialization references up with tmpro retirement.

update codacy badge.

update Qt 5.12 builds on macos and windows to 5.12.11 (#752)

* update windows, macos Qt 5.12 builds to 5.12.11

* fool around with macos qt install path

* more fooling with macos qt path

* try macos qt path fix yet again

Eliminate internal copy/free inside navilink. (#751)

* C++ improvements inside Vcard. Add test.
* Eternal impedance mismatch on C vs C++ pointer ownership.

C++ improvements in VCard. Add tests. (#750)

* C++ improvements inside Vcard. Add test.
* Eternal impedance mismatch on C vs C++ pointer ownership.
* VCF: Incorporate review feedback: auote ','. Match HTML tage regardless of case.
* Quote commas in ASCII source, not just UTH8-encoded text.

workaround github cache output issues. (#749)

a cache hit is reported only for an exact match.  But in a pull
request we can have an inexact match result in cache being fetched.

fix windows release with Qt6. (#748)

* fix windows release with Qt6.

windeployqt failed to deploy Qt6Core5Compat.dll because it was
not used by the gui and we didn't tell windeployqt there was
another executable to scan for dependencies.

* fiddle with github cache conditionals.

cache was restored, but we tried to install qt anyway.

* Revert "fiddle with github cache conditionals."

This reverts commit fece79e5e56de2f957960236bb851c6bc7682d7c.

* try to get github action conditionals working again.

* Revert "try to get github action conditionals working again."

This reverts commit 3152108e5267580f3836b9f6aadc7dfa2ecfe347.

* work around apparent bug in github action conditonal processing.

* the second half of the workaround

* fool around with bash

* more bash foolishness.

kill appveyor 5.9 build

update Qt6 builds to 6.2.1 (#747)

Add support for Qt6. (#739)

* Add support for Qt6.

* fix a few codacy complaints.

update fedora images dropping libusb versions before 1.0 (#746)

update included libusb for mac to 1.0.24 (#744)

replace the last uses of QRegExp. (#743)

also, correct the discard documentation.  The given example didn't
work in recent releases.

update packages names in doc. (#745)

update document. (#741)

update build chapter to reflect current builds.
update dtd to 4.5.
correct manual page document syntax and missing items.

obfuscate map key (#740)

* obfuscate map key.

* keep js debug turned off.

fix an out of bounds access to a QString in tpo. (#738)

This was caught with Qt6.2.0, which has an assertion to check for
out of bounds access.
ASSERT: "i >= 0 && i < size()" in file .../Qt-6.2.0/6.2.0/gcc_64/include/QtCore/qstring.h, line 1325

try harder to fix contious release tags. (#737)

fix sha on continous releases from github. (#736)

Merge pull request #735 from tsteven4/pr643fixestry2

tpo fixes for #643

use constexpr for debug code in tpo.

run astyle on tpo.cc

fix codacy flagged %d in format string mismatch.

%d in format string (no. 1) requires 'int' but the argument type is 'unsigned int'.

Update tpo.cc (#643)

Some .tpo file tracks were/are parsed such that wild jumps occur in lat/lon, creating invalid coords which were not caught by output routines. Some logic flaws and int size errors have been fixed, but some tracks still contain invalid points. Tracks not corrected by this patch now result in warning messages to STDOUT. Entirely valid tracks will also generate warnings if lat or lon changes by a full degree between adjacent track points (should never happen). Non-fatal warning allows manual removal of the few bad track points  in an output format like GPX. Damage is apparently limited to half a dozen incremental points before the next full point, but that can't be guaranteed.

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

don't set gpsbabel::XmlStreamWriter codec in mapfactor (#733)

We use utf-8 by default as of #671.

Merge pull request #732 from tsteven4/qtprep103

Qt 6 compatiblity changes

avoid std::min with QString::size

missed one in previous commit.

don't include QCharRef.

missed one in previous commit.

Merge pull request #731 from tsteven4/qt6prep102

compatible changes for Qt6

don't include QByteRef.

It is gone in Qt6.  It was not documented as a class in Qt5.
In Qt5 QByteRef just included qbytearray.h.

don't include QCharRef.

It is gone in Qt6.  It was a "helper class for QString".  It was not
documented as a class in Qt5.  In Qt5 QCharRef just included qstring.h.

don't include QStaticStringData.

missed one in the previous commit.

Merge pull request #730 from tsteven4/qt6prep101

compatible changes for Qt6

accomodate the deletion of QDateTime::toTime_t in Qt6.

don't include QStaticStringData.

It goes away in Qt6.  Besides, the documented include is QString,
which in Qt 5 will include qstringliteral.h just as QStaticStringData
does.

avoid std::min with QString::size, which

changes types in Qt6.

migrate from QRegExp to QRegularExpression. (#729)

compatible fixes for qt6 warnings and conflicts. (#728)

* quite qt6 yammering about foreach.

* avoid name collision with Qt6.

* fix warning: type qualifiers ignored on cast result type

Don't use Qt module names in include directives. (#727)

This simplifies migration to Qt6 where some classes moved.

This was done mostly by an ed script, although mkstyle.sh took some
hand holding.

1,$g/#include <QtCore\// s/<QtCore\/\([^>]*\)>/<\1>       /
1,$g/#include <QtGui\// s/<QtGui\/\([^>]*\)>/<\1>      /
1,$g/#include <QtNetwork\// s/<QtNetwork\/\([^>]*\)>/<\1>          /
1,$g/#include <QtWidgets\// s/<QtWidgets\/\([^>]*\)>/<\1>          /
1,$g/ *$/ s/ *$//
w
q

more qstringview work for qt6. (#726)

update gui/gpx.cc to use QStringView for Qt6. (#725)

pass QStringViews by value, not const reference. (#724)

As recommended in the QStringView documentation.

fixes for #722, Mechanically remove QStringRef use for QStringView to help with Qt6. (#723)

* Fix typo in maggeo from search-and-replace work earlier.

* Revert startOfDate() calls to keep floor at 5.12.x to keep builders happy. :-/

* Explictly get QString from StringView in gpx for compat with older Qt builds.

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

Mechanically remove QStringRef use for QStringView to help with Qt6. (#722)

* Make more string literals encoded at UTF-16 (bloaty-bytes) to
reduce conversions.
* Favor QStringView(Qt6-ism that mirrors C++ standard std::stringview)
over QStringRef (Qt5 hack).

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

don't limit simplify count maximum in GUI (#721)

Remove Bushnell and Bushnell_trl formats.
These were added in 2009 when Onix unit were on firesale at places like Woot
as Bushnell had just exited the handheld GPS space. There was a flurry of
activity by a low number of users, followed by years of silence beyond
maintenance burden.

fix map preview bug that dropped tracks with 2 pts. (#714)

add fedora 34 to CI (#713)

Minorfixes to "new" gtm_stringread() functions to eliminate leading and trailing spaces (#710)

* Trim more uses of C date and potentially unsafe memory practices.

* Add warning fixes for garmin_xt and exif

Merge pull request #664 from GPSBabel/reaper

Remove dead formats: alan, an1, netstumbler, naviguide.

restore lost format options when using positional parameters (#709)

* restore format options with positional parameters.

* fix positional test

Merge pull request #707 from GPSBabel/pvs

PVS-Studio fixes

Bring back some of the files I nuked and move them to deprecated/

fix some format mismatches found by PVS-Studio

V576 Incorrect format. Consider checking the second actual argument of the 'asprintf' function. The SIGNED integer type argument is expected.

QString::asprintf is not recommended for new code anyway.

fix erroneous Qt translation caught by PVS studio.

V646 Consider inspecting the application's logic. It's possible that 'else' keyword is missing.

This makes me wonder if this code is ever used.  If args == "OutputFromUnit"
it would throw a fatal error since b9cdbe4e8dc02cff07af9e0fa28adb2372337963 Oct 22, 2013

fix sequence related undefined behavior found by PVS Studio.

V567 The modification of the 's' variable is unsequenced relative to another operation on the same variable. This may lead to undefined behavior.

pvs fix

V668 There is no sense in testing the 'wpt_tmp' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error.

fix another pvs warning

V758 The 'pos' reference becomes invalid when temporary object returned by a function is destroyed.

fix ancient memory leak in filter_vecs. (#706)

another pvs studio find

↑ V547 Expression is always false.

PVS Studio kills an old bug.

V1064 The '360' operand of integer division is less than the '65535' one. The result will always be zero.

Also, we need to set the flag for course.

make some Qt containers const. (#705)

This avoids checks for detachment.

fix Qt6 deprecation warning in mapbar_track. (#703)

* fix Qt6 deprecation warning in mapbar_track.

"warning: 'fromUtf16' is deprecated: Use char16_t* overload."
However, it is recommended to use QString(const QChar *, int) or
QString(const QChar *) instead of
QString::fromUtf16(const ushort *unicode, int size = -1) and
QString::fromUtf16(const char16_t *str, int size = -1) anyway.

* fix Either the condition 'track==nullptr' is redundant or there is possible null pointer dereference.

as this isn't using a non-throwing allocation function memory allocation
errors will cause std::bad_alloc to be thrown anyway, we won't return
with track == nullptr.

* eliminate another non-functional memory check.

fix Wmissing-field-initializers. (#704)

These appear with Qt6 on macos which enables Wextra.

use default parameter instead of QString::KeepEmptyParts, (#702)

simplifying migration to Qt6.

Parse empty gx:coord elements in KML reader (#701)

Add test for empty gx:coord elements

modernize nmea reader (#700)

use object tree to cleanup UpgradeCheck, UpgradeCheck::manager_ and (#692)

UpgradeCheck::replyId.

Introduce RunMachine to manage gpsbabel process. (#699)

* Introduce RunMachine to manage gpsbabel process.

* fix comment

* guard runmachine init state

fix gui usage of duplicate filter. (#695)

the shortname and location options are not exclusive.

this resolves #686.

remove a few old kludges that are no longer needed. (#691)

* remove a few old kludges that are no longer needed.

* clean up upgrade a bit

"GPSBabel Version " doesn't need to be removed from currentVersionIn
in UpgradeCheck::checkForUpgrade,
that is taken care of in MainWindow::findBabelVersion

update readme.md with current continuous integration provider (#672)

* update README for new CI.

* use list for workflows

* appease codacy list item indent

* restore lost newline

* don't mention pre-releases in README.md

remove leftover includes from autotools (#689)

* remove leftovers from autotools

we no longer have config.h, HAVE_CONFIG_H is always false.

* more config.h deletions (from jeeps)

make src/core and jeeps includes relative to top cli directory (#688)

* make jeeps includes relative to cli diretory.

* make src/core includes all relative to top level src dir.

prefer implicitly-defined dtors instead of user dtors with empty bodies. (#685)

Not only is this simpler, it avoids clang-tidy complaining about
warning: class * defines a non-default destructor but does not define a copy
constructor, a copy assignment operator, a move constructor or a move
assignment operator [cppcoreguidelines-special-member-functions]

cleanup FormatLoad class header file. (#684)

clang-tidy modernize-use-default-member-init

This allows the use of the implicitly-defined default ctor.
Use implicitly-defined dtor as well.

further cleanups for Format class header. (#683)

manual fix for typedef found by clang-tidy modernize-use-using

clazy fixes for clazy-function-args-by-ref

clang-tidy fix for modernize-use-default-member-init

use new style connect, i.e. function pointers (#682)

* migrate from old style connects to the new style.

The changes using qOverload were done by hand, the rest were done
with clazy 1.9.

The ones using the TreeAction constructor were not found by clazy
and would require more manual intervention.

Also, I note uic generates old style connects.

* eliminate our TreeAction class in favor of QMenu::addAction method.

It is a mystery to me why we created TreeAction.  The QMenu method existed
in Qt 4.3.  Documentation before 4.3 is scarce.
QMenu::addAction(const QString &text, const QObject *receiver, const char *member, const QKeySequence &shortcut = 0)

* clazy updates gmapdlg connects.

* add TODOs for Qt6 related to qOverload.

drop support for QtWebKit. (#680)

Our QtWebKit build didn't work any longer on Ubuntu bionic or focal,
but went into an infinite loop if map preview was used.
I observed this behavior with builds of 1.7.0 as well.

Document exisiting gui configuration option disable-mappreview.

I note that latest fedora build spec falls back to QtWebKit on some
architectures where QtWebEngine is not available.  This change will
require them to fall back to disabling the map preview instead.
https://src.fedoraproject.org/rpms/gpsbabel/blob/rawhide/f/gpsbabel.spec

QtWebEngine appears to be coming to Qt6 with or after 6.2.
https://bugreports.qt.io/browse/QTBUG-63235

delete FAKE_LANGUAGE_MENU code. (#681)

This code hadn't compiled for a while, and is no longer necessary
to test translations.

replace obsolete and overloaded QProcess signal error(QProcess::ProcessError error) (#679)

with the recommended replacement, QProcess signal errorOccurred(QProcess::ProcessError error).
It seems this should have caused an issue in Qt6, but we hadn't noticed it yet.

fix deprecated-copy waring. (#675)

This fixes
"formatload.cc:127:40: warning: implicitly-declared
 ‘Format& Format::operator=(const Format&)’ is deprecated [-Wdeprecated-copy]"

Use default member initializers for FormatOption and Format classes.
This allows the use of the default constructor and simplifies the
creation of parameterized constructors.

Use implicit copy constructor for FormatOption and Format classes.
Note the previous copy constructor for the Format class was not
really a copy constructor, it re-initialized readUseCount and writeUseCount.
This was unnecessary.  The only place we want to use the copy constructor is
in FormatLoad::getFormats and the instances to be copied have just been
created with one of the Format constructors so the use counts will be zero.
There are plenty of other opportunities where Qt might detach formatList_
and use the copy constructor.  Most of these cases are read accesses and
improved const correctness could eliminate the possiblity, i.e. using
QList::at instead of QList::operator[].  A few of these are modifying an
item on the formatList_, and Qt will insist the copy constructor is available
at compile time even if it is never used at run time.

refactoring for cppcheck "The scope of the variable can be reduced" (#677)

* refactoring for cppcheck "The scope of the variable can be reduced"

The c++17 "If Statement with Initializer" often are useful in resovling
these.

* more scope reduction.

* and another.

Merge pull request #676 from tsteven4/moretidy

clang tidy fixes

qt6 compatibility changes. (#674)

QDateTime::toTime_t has been obsolete for a long time, it is gone
in Qt6.