Fix last CodeFactor Style issues (#202)

* fix malformed whitespace

* fix trailing spaces 


fix trailing spaces

.github/workflows/cmake-arm.yml

@@ -22,7 +22,7 @@ jobs:
 
     - name: Install bare metal ARM toolchain
       run: sudo apt-get install binutils-arm-none-eabi gcc-arm-none-eabi libnewlib-arm-none-eabi libstdc++-arm-none-eabi-newlib
-      
+
     - name: Show toolchain version
       run: arm-none-eabi-gcc --version
 
@@ -36,8 +36,8 @@ jobs:
       # access regardless of the host operating system
       shell: bash
       working-directory: ${{runner.workspace}}/build
-      # Note the current convention is to use the -S and -B options here to specify source 
+      # Note the current convention is to use the -S and -B options here to specify source
-      # and build directories, but this is only available with CMake 3.13 and higher.  
+      # and build directories, but this is only available with CMake 3.13 and higher.
       # The CMake binaries on the Github Actions machines are (as of this writing) 3.12
       run: cmake $GITHUB_WORKSPACE/examples/knx-cc1310 -DCMAKE_BUILD_TYPE=$BUILD_TYPE
 
@@ -50,6 +50,6 @@ jobs:
    # - name: Test
    #   working-directory: ${{runner.workspace}}/build
    #   shell: bash
-      # Execute tests defined by the CMake configuration.  
+      # Execute tests defined by the CMake configuration.
       # See https://cmake.org/cmake/help/latest/manual/ctest.1.html for more detail
    #   run: ctest -C $BUILD_TYPE

.github/workflows/cmake-native.yml

@@ -27,8 +27,8 @@ jobs:
       # access regardless of the host operating system
       shell: bash
       working-directory: ${{runner.workspace}}/build
-      # Note the current convention is to use the -S and -B options here to specify source 
+      # Note the current convention is to use the -S and -B options here to specify source
-      # and build directories, but this is only available with CMake 3.13 and higher.  
+      # and build directories, but this is only available with CMake 3.13 and higher.
       # The CMake binaries on the Github Actions machines are (as of this writing) 3.12
       run: cmake $GITHUB_WORKSPACE -DCMAKE_BUILD_TYPE=$BUILD_TYPE
 
@@ -41,6 +41,6 @@ jobs:
    # - name: Test
    #   working-directory: ${{runner.workspace}}/build
    #   shell: bash
-      # Execute tests defined by the CMake configuration.  
+      # Execute tests defined by the CMake configuration.
       # See https://cmake.org/cmake/help/latest/manual/ctest.1.html for more detail
    #   run: ctest -C $BUILD_TYPE

.github/workflows/codeql-analysis.yml

@@ -47,7 +47,7 @@ jobs:
       with:
         languages: ${{ matrix.language }}
         # If you wish to specify custom queries, you can do so here or in a config file.
-        # By default, queries listed here will override any specified in a config file. 
+        # By default, queries listed here will override any specified in a config file.
         # Prefix the list here with "+" to use these queries and those in the config file.
         # queries: ./path/to/local/query, your-org/your-repo/queries@main
 

src/knx/application_layer.cpp

@@ -347,7 +347,7 @@ void ApplicationLayer::dataConnectedConfirm(uint16_t tsap)
 
 void ApplicationLayer::dataConnectedConfirm(uint16_t tsap, const SecurityControl& secCtrl)
 {
-
+    //FIXME: implement dataConnectedConfirm DataSecurity
 }
 #pragma endregion
 void ApplicationLayer::groupValueReadRequest(AckType ack, uint16_t asap, Priority priority, HopCountType hopType, const SecurityControl& secCtrl)

src/knx/application_layer.h

@@ -157,7 +157,6 @@ class ApplicationLayer
 #pragma endregion
 
   protected:
-
 #pragma region hooks
     void dataGroupIndication(HopCountType hopType, Priority priority, uint16_t tsap, APDU& apdu, const SecurityControl &secCtrl);
     void dataGroupConfirm(AckType ack, HopCountType hopType, Priority priority, uint16_t tsap,

src/knx/bau_systemB_coupler.cpp

@@ -21,7 +21,6 @@ BauSystemBCoupler::BauSystemBCoupler(Platform& platform) :
 #ifdef USE_DATASECURE
     _memory.addSaveRestore(&_secIfObj);
 #endif
-
 }
 
 ApplicationLayer& BauSystemBCoupler::applicationLayer()

src/knx/device_object.cpp

@@ -91,7 +91,6 @@ DeviceObject::DeviceObject()
 #ifdef USE_RF
         new DataProperty(PID_RF_DOMAIN_ADDRESS_CEMI_SERVER, true, PDT_GENERIC_06, 1, ReadLv3 | WriteLv3),
 #endif
-
     };
     initializeProperties(sizeof(properties), properties);
 }

src/knx/dptconvert.cpp

@@ -135,7 +135,6 @@ int KNX_Decode_Value(uint8_t* payload, size_t payload_length, const Dpt& datatyp
 
 int KNX_Encode_Value(const KNXValue& value, uint8_t* payload, size_t payload_length, const Dpt& datatype)
 {
-
     if (datatype.mainGroup == 1 && datatype.subGroup >= 1 && datatype.subGroup <= 23 && datatype.subGroup != 20 && !datatype.index)
         return valueToBusValueBinary(value, payload, payload_length, datatype);
     // DPT 2.* - Binary Control
@@ -1553,7 +1552,6 @@ int valueToBusValueRGBW(const KNXValue& value, uint8_t* payload, size_t payload_
         case 1: // Mask bits
             unsigned8ToPayload(payload, payload_length, 5, (uint8_t)value, 0x0f);
             break;
-
     }
     return true;
 }

src/knx/interface_object.h

@@ -188,7 +188,6 @@ class InterfaceObject : public SaveRestore
     uint16_t saveSize() override;
 
   protected:
-
     /**
      * Intializes the Property-array the the supplied values.
      */

src/knx/knx_ip_frame.h

@@ -29,9 +29,6 @@ enum KnxIpServiceType
     TunnelingAck = 0x421,
     RoutingIndication = 0x530,
     RoutingLostMessage = 0x531,
-    
-    
-    
 };
 
 class KnxIpFrame

src/knx/knx_value.h

@@ -49,7 +49,6 @@ class KNXValue
     KNXValue& operator=(const float value);
 
   private:
-    
     bool boolValue() const;
     uint8_t ucharValue() const;
     uint16_t ushortValue() const;

src/knx/memory.cpp

@@ -162,7 +162,6 @@ void Memory::writeMemory()
         //save to size of the memoryblock for tableobject too, so that we can rebuild the usedList and freeList
         if (_tableObjects[i]->_data != nullptr)
         {
-
             MemoryBlock* block = findBlockInList(_usedList, _tableObjects[i]->_data);
             if (block == nullptr)
             {

src/knx/rf_physical_layer_cc1101.cpp

@@ -679,7 +679,6 @@ void RfPhysicalLayerCC1101::loop()
                         bytesLeft  -= (32 - 1);
                         pByteIndex += (32 - 1);
                     }
-
                 }
             }
 
@@ -719,7 +718,6 @@ void RfPhysicalLayerCC1101::loop()
                     packetStartTime = millis();
                     syncStart = true;
                 }
-                
             }
         }
         break;

src/knx/secure_application_layer.h

@@ -21,7 +21,6 @@ class BusAccessUnit;
  */
 class SecureApplicationLayer :  public ApplicationLayer
 {
-
   public:
     /**
      * The constructor.
@@ -56,8 +55,6 @@ class SecureApplicationLayer :  public ApplicationLayer
      void dataConnectedRequest(uint16_t tsap, Priority priority, APDU& apdu, const SecurityControl& secCtrl) override; // apdu must be valid until it was confirmed
 
   private:
-
-
     enum class AddrType : uint8_t
     {
         group,

src/knx/security_interface_object.cpp

@@ -490,7 +490,6 @@ uint16_t SecurityInterfaceObject::getNumberOfElements(PropertyID propId)
 
 uint64_t SecurityInterfaceObject::getLastValidSequenceNumber(uint16_t deviceAddr)
 {
-
     // Get number of entries for this property
     uint16_t numElements = getNumberOfElements(PID_SECURITY_INDIVIDUAL_ADDRESS_TABLE);
 

src/knx/tpuart_data_link_layer.cpp

@@ -616,7 +616,6 @@ bool TpUartDataLinkLayer::sendSingleFrameByte()
 
 void TpUartDataLinkLayer::addFrameTxQueue(CemiFrame& frame)
 {
-
     _tx_queue_frame_t* tx_frame = new _tx_queue_frame_t;
     tx_frame->length = frame.telegramLengthtTP();
     tx_frame->data = new uint8_t[tx_frame->length];

src/knx/transport_layer.cpp

@@ -343,7 +343,6 @@ void TransportLayer::dataIndividualConfirm(AckType ack, uint16_t destination, Ho
                 A5(destination);
                 break;
             }
-
         }
         break;
     case Disconnect:

src/knx/usb_tunnel_interface.cpp

@@ -63,7 +63,6 @@ void UsbTunnelInterface::loop()
 		handleHidReportRxQueue();
 		rxHaveCompletePacket = false;
 	}
-
 }
 
 /* USB TX */
@@ -136,7 +135,6 @@ void UsbTunnelInterface::loadNextTxFrame(uint8_t** sendBuffer, uint16_t* sendBuf
 	}
 	println("");
 #endif
-
 }
 
 void UsbTunnelInterface::sendKnxHidReport(ProtocolIdType protId, ServiceIdType servId, uint8_t* data, uint16_t length)