refactor(packchk): clean up CodeQL style findings

tools/packchk/include/PackChk.h

@@ -22,7 +22,7 @@ class PackChk {
   PackChk();
   ~PackChk();
 
-  int Check(int argc, const char* argv[], const char* envp[]);
+  int Check(int argc, const char** argv, const char** envp);
 
   const RteGlobalModel& GetModel() { return m_rteModel; }
 

tools/packchk/include/ParseOptions.h

@@ -23,7 +23,7 @@ class ParseOptions {
     Error,
   };
 
-  Result Parse(int argc, const char* argv[]);
+  Result Parse(int argc, const char** argv);
 
 protected:
   bool SetWarnLevel(const std::string& warnLevel);

tools/packchk/src/PackChk.cpp

@@ -172,7 +172,7 @@ bool PackChk::CheckPackage()
  * @param envp command line argument (not used)
  * @return passed / failed
 */
-int PackChk::Check(int argc, const char* argv[], const char* envp[])
+int PackChk::Check(int argc, const char** argv, const char** envp)
 {
   const string header = m_packOptions.GetHeader();
   cout << header << endl;

tools/packchk/src/PackChkMain.cpp

@@ -15,7 +15,7 @@
  * @param envp command line argument (not used)
  * @return 0: ok, 1: error
 */
-int main(int argc, const char* argv [], const char* envp [])
+int main(int argc, const char** argv, const char** envp)
 {
   PackChk packChk;
 

tools/packchk/src/ParseOptions.cpp

@@ -191,7 +191,7 @@ bool ParseOptions::SetDisableValidation(bool bDisable)
  * @param argv command line
  * @return passed / failed
  */
-ParseOptions::Result ParseOptions::Parse(int argc, const char* argv[])
+ParseOptions::Result ParseOptions::Parse(int argc, const char** argv)
 {
   const string fileName = m_packOptions.GetProgramName();
   bool bOk = true;

tools/packchk/src/ValidateSyntax.cpp

@@ -1697,6 +1697,7 @@ bool ValidateSyntax::CheckBoards(RtePackage* pKg)
       LogMsg("M375", VAL("BOARD", boardName));
     }
 
+    // deviceIndex must be unique among mounted devices on this board.
     map<string, RteItem*> mountedDeviceIndex;
     for(auto mountedDevice : mountedDevices) {
       lineNo = mountedDevice->GetLineNumber();
@@ -1726,11 +1727,13 @@ bool ValidateSyntax::CheckBoards(RtePackage* pKg)
         continue;
       }
 
+      // Mounted devices require a concrete Dname; wildcard patterns are not allowed.
       if(dname.find_first_of("*?[]") != string::npos) {
         LogMsg("M607", TAG("Dname"), NAME(dname), lineNo);
         continue;
       }
 
+      // Resolve the reference against the loaded device model, including reference packs.
       list<RteDevice*> devices;
       GetModel().GetDevices(devices, dname, dvendor);
       if(devices.empty()) {
@@ -1742,6 +1745,7 @@ bool ValidateSyntax::CheckBoards(RtePackage* pKg)
         LogMsg("M100", lineNo);
       }
 
+      // Warn when a mounted device name still denotes a device with variants.
       for(const auto device : devices) {
         if(device->GetDeviceItemCount() > 0) {
           LogMsg("M611", VENDOR(dvendor), MCU(dname), lineNo);
@@ -1761,7 +1765,7 @@ bool ValidateSyntax::CheckBoards(RtePackage* pKg)
       LogMsg("M010");
     }
 
-    // ------------  compatible devices  ------------------
+    // Compatible devices describe alternatives and may use Dname wildcard patterns.
     Collection<RteItem*> compatibleDevices;
     board->GetCompatibleDevices(compatibleDevices);