aboutsummaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)Author
2018-05-24mobile: application variable unused in mainGravatar jan Iversen
Simplify starting application. instead of declaring var. application, and the setting unused, simple do the "new QApplication" without a variable. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-24Cleanup: remove code related to picture-storage in git repositoriesGravatar Berthold Stoeger
Saving of pictures to git repositories was disabled. Finally remove this code and the corresponding load code. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
2018-05-24subsurface: update .gitignore with extra dirs.Gravatar jan Iversen
make sure "git status" does not report know dirs. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-24build-system: integrate get-dep-lib in mobilecomponentsGravatar jan Iversen
use get-dep-lib in mobilecomponents to do git clone of kirigami and breeze-icons. Remove -no-pull option Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-24build-system: get-dep-lib add support for kirigami and breeze-iconsGravatar jan Iversen
mobilecomponents.sh do git clone of kirigami and breeze-icons. should be centralized with get-dep-lib Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-23Fix includes for building against Qt-5.11.0Gravatar Gaetan Bisson
Signed-off-by: Gaetan Bisson <bisson@archlinux.org>
2018-05-21mobile: removing print mode from mobile.Gravatar jan Iversen
print mode was used to limit the functionality of the profile, when used in ssrf-mobile. The effect is that DC events are displayed, but not selectable Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21profile-widget: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21desktop-widgets/statistics: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21desktop-widgets/.../facebook: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21desktop-widgets: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21subsurface: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21qt-models: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21core/subsurface-qt: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21core: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21mobile-widgets: Change Q_UNUSED to no parameter nameGravatar jan Iversen
C++ permits use of parameters without name, which signals unused Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21Dive pictures: don't repopulate DivePictureModel on deletionGravatar Berthold Stoeger
On deletion of a single or multiple pictures, the whole DivePictureModel was repopulated, which was clearly visible in the UI, owing to the reconstructing of all images in the profile plot. To avoid this vexing behavior, implement proper deletion routines in DivePictureModel and ProfileWidget2. Since this needs sensible erase() semantics the QList<PictureEntry> member of DivePictureModel was replaced by a QVector. A QVector should be the default anyway, unless there are very specific reasons to use a QList (which actually is a deque, not a classical linked list). Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
2018-05-21Dive pictures: change removePicture() interfaceGravatar Berthold Stoeger
The function removePicture() had a flag "last", which would indicate that the called had finished removing pictures. Only then would the model be recalculated. This is a strange interface and, matter of fact, the caller was buggy: if the last picture to be removed didn't have a proper url, removePicture() was never called with "last" being set. Change the interface to take a list of pictures to be deleted. This will allow us to make picture deletion smarter in follow-up commits. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
2018-05-21Dive pictures: use modelReset signal when reseting picture listGravatar Berthold Stoeger
In the old code a combination of removeRows()/insertRows() was used to signal a model reset. Replace this by a single, modelReset signal. This saves a call to plotPictures() and will allow us to be smarter in the future, when removing pictures. Reset-model and remove-items are semantically different. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
2018-05-21iOS: copy libdivecomputer.log to clipboardGravatar jan Iversen
Read libdivecomputer.log file and append to clipboard Remark, subsurface_open is not available in iOS so using QFile instead. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21iOS: updated text in About to reflect logs not logGravatar jan Iversen
Both subsurface.log and libdivecomputer.log are on the clipboard. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21iOS: store libdivecomputer.log in documents (same dir as app log)Gravatar jan Iversen
Make location of subsurface.log and libdivecomputer.log consistent Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21mobile: removed setting for libdivecomputer.logGravatar jan Iversen
Users need a simple way to report download problems. Asking them to go into settings, activate logging, and repeat the download is too much. libdivecomputer.log will always be generated, but overwritten, with every download, so the latest log is ready. The download is very slow due to BT, so there are no impact of having the log active, neither in terms of size (a concern on small mobiles) or write time. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21iOS: qt-ble.cpp remove unsupported event.Gravatar jan Iversen
QEvent::DeferredDelete is not supported in iOS Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21profile-widget: divepixmapitem.cpp, silence unused parameterGravatar jan Iversen
remove parameter name for not used parameter, to silence unused warning. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21profile-widget: profilewidget2.cpp, silence unused parameterGravatar jan Iversen
Add Q_UNUSED for not used parameter Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-21build-system: iOS only turn off specific warningsGravatar jan Iversen
Turning off all warnings (-w) was too radical. Only turning off specific warnings. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-20iOS: get rid of ios/Subsurface-mobileGravatar jan Iversen
Having a split between ios and ios/Subsurface-mobile is just confusing and without a practical reason Move files from ios/Subsurface-mobile to ios and update .gitignore Signed-off-by: Jan Iversen <jani@apache.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-20iOS: remove duplicate of deployment.priGravatar jan Iversen
one deployment.pri is enough Signed-off-by: Jan Iversen <jani@apache.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-20QML UI: allow changing the connection IDGravatar jan Iversen
When downloading with bluetooth, changing connection did not work. The new selection was not passed to the download thread. Fixes #1274 Signed-off-by: Jan Iversen <jani@apache.org> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-20Travis: don't fail Linux build when .zsync is missingGravatar Dirk Hohndel
It seems that sometimes the AppImage tooling gets confused and doesn't create a .zsync file. That should cause Travis to fail. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-20build-system: libgit2 depends on libcurl and libssh2Gravatar jan Iversen
When building with -builddeps on a "virgin" mac, configure of libgit2 could not find libcurl and libssh2 moved building of libcurl and libssh2 in front of libgit2 Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-20build-system: corrected -build-deps problemGravatar jan Iversen
Missed a "cd $SRC" so when building with -build-deps the script broke Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-20core: marked unused parameterGravatar jan Iversen
Add Q_UNUSED to unused parameter Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-19Mac: don't try to copy libraries we got from HomebrewGravatar Dirk Hohndel
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-19build-system: change the logic when to build whatGravatar Dirk Hohndel
Specifically, don't conflate needing libgit2 with the Mac -builddep argument, and when determining if we need to build libgit2 on Linux, make sure to also check for a version that we may have built in a previous run of the build.sh script. This commit is much easier to understand with git show -w as it contains quite a bit of simple indentation change. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-19Travis: add secondary Linux build targetGravatar Dirk Hohndel
The goal is to reduce the overall wall clock time of our test builds. This secondary Linux target uses the no-sudo container setup with a Qt 5.10.1 backport and tests the build without BT support as well as the mobile build (but not the full desktop build and doesn't create an AppImage as the pre-built Qt5.10.1 is missing QtWebKit). In exchange we remove the no-bt and mobile build from the existing linux target. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-19build-system: don't always build qtserialbluetooth.cppGravatar Dirk Hohndel
This should only be built if we have BT support enabled. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-19build-system: updated OpenSSL and libusb for AndroidGravatar Dirk Hohndel
These are the latest, supported versions for OpenSSL. I have not tested if Android would work with OpenSSL 1.1. The lower versio for libzip is still required as libzip 1.2.0 doesn't compile for Android. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-19Android: set target API level to 26Gravatar Dirk Hohndel
This doesn't affect the minimum we support, but a target level of at least 26 will be required starting in August in order to be able to upload to the Google app store. This is equivalent to targeting Android 8.0. Google plans to bump this target API level every year. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-19Mac: support running Subsurface-mobileGravatar Dirk Hohndel
Copy the required QML modules into the app bundle. Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
2018-05-19iOS, supress warnings.Gravatar jan Iversen
Changed .pro file to supress warnings clang does not allow all of the issued warnings to be supressed with -Wno-foo, so instead doing -w Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-19iOS, remove linker warnings (version mismatch).Gravatar jan Iversen
The Qt libraries are compiled as iOS 10.0, we link as iOS 8.0, this is a problem waiting to happen. First time we use a Qt function that uses a iOS 10.0 specific function the app will crash. Bumping iOS minimum version from 8.0 to 10.0 Removing the 76x icon, which are not valid from 10.0 and forward Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-19Profile: minor coding style issues concerning axis-ticksGravatar Berthold Stoeger
1) Fix the English of a comment. 2) Remove a number of int-to-double compares: Make "steps" an integer variable (the number of steps). Rename the old double "steps" variable to "stepsInRange". This gives a non-integer number of steps and is necessary to calculate the correct step size 3) Replace a "x = x/y" by a "x /= y" construct. 4) Remove an unnecessary if clause. Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
2018-05-19Profile: Fix leak in animationGravatar Berthold Stoeger
If animDelete() was called with prefs.animation_speed == 0, the object would not be marked for deletion, as opposed to calling with prefs.animation_speed != 0. This would leak the objects. Therefore delete the objects if called with prefs.animation_speed == 0. The caller doesn't keep a reference to the objects. Therefore, a plain delete is fine, as opposed to a deleteLater(). While touching this function, use the function-pointer version of connect(). Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
2018-05-18build-system: added get-dep-lib to android/build.shGravatar jan Iversen
Integrated get-dep-lib in android/build.sh Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-18build-system: update get-dep-lib with Android fix.Gravatar jan Iversen
Android build uses wget and not curl. Currently android uses older versions of some libs, due to that a new switch is added temporary. Old version tested for curl and if not found set the different versions, however this is not a stable method for obvious reasons Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-18mobile: add "Copy log to clipboard" buttonGravatar jan Iversen
on iOS it is practically impossible to copy the App log to e.g. a mail! in iOS 11 the log file is stored within the subsurface container and you first need to copy (actually using the clipboard) out from there to the "normal" document shared space, before it can be used. At least iOS users (and I believe Android users) are not really used to work with files, so the process is not easy to document in an understandable way. The alternative is to provide a button, which simply puts the log on the general clipboard, allowing it to be pasted in a multitud of applications. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-18core: solved PictureEntry defined as class and struct.Gravatar jan Iversen
PictureEntry was defined as class in imagedownloader.h and as struct in divepicturemodel.h A class has a vptr in front, so the difference is real at least for the clang compiler. Signed-off-by: Jan Iversen <jani@apache.org>
2018-05-18core: removed clang warnings from dive.hGravatar jan Iversen
interpolate, rel_mbar_to_depth, gas_mod and gas_mnd returns int but uses a function that returns long, causing clang to warn about conversion loss due to implicit conversion. Adding a cast, shows that it is correct. Signed-off-by: Jan Iversen <jani@apache.org>