5.8.3 (#658)

* add delay if interface not found

* adjust initialization behavior

* pushed for debugging reasons for concerned parties

* comment out simple recenter

* remove old style motion, some minor optimizations

* optimize default case check

* optimized default condition

* change manifest install behavior to trigger on desktop mode as well

* add desktop mode binding launch

* add tracker overlay option to utils

* commit to check status

* attempt to fix native LH issues with chaperone when exiting

* add desktop mode toggle and some cleanup

* remove adjust chaperone

* remove clang format from ci, increment version, cleanup

* update circle ci

* fix conversion warning

* add F13-19 to keyboard parser

* add additional logging around failure to initialize

* add unsaved changes

* update version 5.8.1

* fix some collisions in enum declaration

* add controller powers on steamvr, and no HMD to steamvr tab, increment version

* slightly alter drag movement, fix qml issue

* code cleanup

* fix logspam/attempt to delete non existant overlays

* increment version 5.8.3

Author: ykeara

.appveyor.yml

@@ -34,7 +34,7 @@ init:
 
 build_script:
   - python ./build_scripts/append_to_version_file.py %APPVEYOR_REPO_COMMIT:~0,8%
-  - python ./build_scripts/run-clang-format.py ./src -r --clang-format-executable "C:\Program Files\LLVM\bin\clang-format.exe" --color always
+#  - python ./build_scripts/run-clang-format.py ./src -r --clang-format-executable "C:\Program Files\LLVM\bin\clang-format.exe" --color always
   - call "build_windows.cmd"
 
 artifacts:

.circleci/config.yml

@@ -29,18 +29,18 @@ jobs:
                     sudo add-apt-repository 'deb http://apt.llvm.org/xenial/ llvm-toolchain-xenial-8 main'
                     sudo add-apt-repository 'deb http://apt.llvm.org/xenial/ llvm-toolchain-xenial-7 main'
                     sudo apt-get update -qq
-            - run:
-                name: Install LLVM-8 binaries
-                # clang is required due to a check by Qmake.
-                # clang-tidy-8 has new errors compared to clang-tidy-7, so we haven't upgraded.
-                command: |
-                    sudo apt-get install clang-format-8 -y
-                    sudo apt-get install clang-tidy-7 -y
-                    sudo apt-get install clang-8 -y
-                    sudo apt-get install clang -y
-            - run:
-                name: Run clang-format
-                command: python ./build_scripts/run-clang-format.py ./src -r --color always --clang-format-executable clang-format-8
+            #- run:
+            #    name: Install LLVM-8 binaries
+            #    # clang is required due to a check by Qmake.
+            #    # clang-tidy-8 has new errors compared to clang-tidy-7, so we haven't upgraded.
+            #    command: |
+            #        sudo apt-get install clang-format-8 -y
+            #        sudo apt-get install clang-tidy-7 -y
+            #        sudo apt-get install clang-8 -y
+            #        sudo apt-get install clang -y
+            #- run:
+            #    name: Run clang-format
+            #    command: python ./build_scripts/run-clang-format.py ./src -r --color always --clang-format-executable clang-format-8
             - run:
                 name: Install g++
                 command: |

.travis.yml

@@ -30,7 +30,7 @@ before_install:
   - sudo apt update -qq
   - sudo apt install clang-format-8
   - clang-format-8 --version
-  - python ./build_scripts/run-clang-format.py ./src -r --color always --clang-format-executable clang-format-8
+#  - python ./build_scripts/run-clang-format.py ./src -r --color always --clang-format-executable clang-format-8
   - sed -i 's/QMAKE_CXXFLAGS += -flto//g' ./build_scripts/qt/compilers/clang-gcc-common-switches.pri # CI version of clang can't handle LTO
   - sed -i 's/QMAKE_LFLAGS += -flto -fuse-ld=gold//g' ./build_scripts/qt/compilers/clang-gcc-common-switches.pri # CI version of clang can't handle LTO
 

build_scripts/compile_version_string.txt

@@ -1 +1 @@
-5.7.4-release
+5.8.3-release

build_scripts/qt/resources.pri

@@ -45,6 +45,7 @@ DISTFILES += \
     src/res/qml/utilities_page/media_keys/* \
     src/res/qml/utilities_page/keyboard_utils/* \
     src/res/qml/utilities_page/alarm_clock/* \
+    src/res/qml/utilities_page/misc/* \
     src/res/qml/audio_page/dialog_boxes/* \
     src/res/qml/audio_page/device_selector/* \
     src/res/qml/audio_page/proximity/* \

src/keyboard_input/input_parser.cpp

@@ -65,8 +65,20 @@ std::optional<Token> getFunctionNumberExtended( const char& character ) noexcept
         return Token::KEY_F11;
     case '2':
         return Token::KEY_F12;
-    // Theoretically we could extende out to f19, but as F12 is the only
-    // function key on Standard Keyboard.
+    case '3':
+        return Token::KEY_F13;
+    case '4':
+        return Token::KEY_F14;
+    case '5':
+        return Token::KEY_F15;
+    case '6':
+        return Token::KEY_F16;
+    case '7':
+        return Token::KEY_F17;
+    case '8':
+        return Token::KEY_F18;
+    case '9':
+        return Token::KEY_F19;
     default:
         LOG( INFO ) << "Unknown function key number: " << character;
         return std::nullopt;
@@ -477,6 +489,20 @@ bool isLiteral( const Token token ) noexcept
         return true;
     case Token::KEY_F12:
         return true;
+    case Token::KEY_F13:
+        return true;
+    case Token::KEY_F14:
+        return true;
+    case Token::KEY_F15:
+        return true;
+    case Token::KEY_F16:
+        return true;
+    case Token::KEY_F17:
+        return true;
+    case Token::KEY_F18:
+        return true;
+    case Token::KEY_F19:
+        return true;
 
     case Token::KEY_BACKSPACE:
         return true;

src/keyboard_input/input_parser.h

@@ -20,8 +20,15 @@ enum class Token
     KEY_F10,
     KEY_F11,
     KEY_F12,
+    KEY_F13,
+    KEY_F14,
+    KEY_F15,
+    KEY_F16,
+    KEY_F17,
+    KEY_F18,
+    KEY_F19,
 
-    KEY_BACKSPACE,
+    KEY_BACKSPACE = 150,
     KEY_SPACE,
     KEY_TAB,
     KEY_ESC,

src/keyboard_input/input_sender_X11.cpp

@@ -105,6 +105,20 @@ unsigned int tokenToKeySym( const Token token )
         return XK_F11;
     case Token::KEY_F12:
         return XK_F12;
+    case Token::KEY_F13:
+        return XK_F13;
+    case Token::KEY_F14:
+        return XK_F14;
+    case Token::KEY_F15:
+        return XK_F15;
+    case Token::KEY_F16:
+        return XK_F16;
+    case Token::KEY_F17:
+        return XK_F17;
+    case Token::KEY_F18:
+        return XK_F18;
+    case Token::KEY_F19:
+        return XK_F19;
 
     case Token::KEY_BACKSPACE:
         return XK_BackSpace;

src/keyboard_input/input_sender_win.cpp

@@ -36,6 +36,20 @@ WORD convertToVirtualKeycode( const Token token )
         return VK_F11;
     case Token::KEY_F12:
         return VK_F12;
+    case Token::KEY_F13:
+        return VK_F13;
+    case Token::KEY_F14:
+        return VK_F14;
+    case Token::KEY_F15:
+        return VK_F15;
+    case Token::KEY_F16:
+        return VK_F16;
+    case Token::KEY_F17:
+        return VK_F17;
+    case Token::KEY_F18:
+        return VK_F18;
+    case Token::KEY_F19:
+        return VK_F19;
 
     case Token::KEY_BACKSPACE:
         return VK_BACK;

src/main.cpp

@@ -4,8 +4,8 @@
 #ifdef _WIN64
 #    include <windows.h>
 extern "C" __declspec( dllexport ) DWORD NvOptimusEnablement = 0x00000001;
-extern "C" __declspec( dllexport ) DWORD
-    AmdPowerXpressRequestHighPerformance = 0x00000001;
+extern "C" __declspec( dllexport ) DWORD AmdPowerXpressRequestHighPerformance
+    = 0x00000001;
 #endif
 
 INITIALIZE_EASYLOGGINGPP
@@ -81,7 +81,7 @@ int main( int argc, char* argv[] )
                               application_strings::applicationKey );
 
         // Attempts to install the application manifest on all "regular" starts.
-        if ( !commandLineArgs.desktopMode && !commandLineArgs.forceNoManifest )
+        if ( !commandLineArgs.forceNoManifest )
         {
             try
             {
@@ -103,7 +103,9 @@ int main( int argc, char* argv[] )
             }
         }
 
-        if ( commandLineArgs.desktopMode )
+        if ( commandLineArgs.desktopMode
+             || settings::getSetting(
+                 settings::BoolSetting::APPLICATION_desktopModeToggle ) )
         {
             auto m_pWindow = new QQuickWindow();
             qobject_cast<QQuickItem*>( quickObj )