Refactor BobinkClient and clean up comments

- Format BobinkAuth.cpp to match GNU style used elsewhere
- Replace brittle error name array with QMetaEnum in BobinkClient
- Move QOpcUaPkiConfiguration include from header to .cpp
- Replace m_discoveryInterval member with file-scope constant
- Add doc comments to SecurityMode and SecurityPolicy enums
- Comment endpoint selection strategy in handleEndpointsReceived
- Add missing QML ids to connectionPage and serverListView

1 files changed, 2 insertions, 2 deletions

diff --git a/src/BobinkClient.h b/src/BobinkClient.h
index 2b41e05..c566a67 100644
--- a/src/BobinkClient.h
+++ b/src/BobinkClient.h
@@ -15,7 +15,6 @@
 #include <QOpcUaClient>
 #include <QOpcUaEndpointDescription>
 #include <QOpcUaErrorState>
-#include <QOpcUaPkiConfiguration>
 #include <QOpcUaProvider>
 #include <QQmlEngine>
 #include <QTimer>
@@ -48,6 +47,7 @@ class BobinkClient : public QObject
       QString keyFile READ keyFile WRITE setKeyFile NOTIFY keyFileChanged)
 
 public:
+  /// OPC UA message security mode (values match MessageSecurityMode).
   enum SecurityMode
   {
     SignAndEncrypt = 3,
@@ -56,6 +56,7 @@ public:
   };
   Q_ENUM (SecurityMode)
 
+  /// OPC UA security policy for encryption algorithms.
   enum SecurityPolicy
   {
     Basic256Sha256,
@@ -175,7 +176,6 @@ private:
 
   /* -- Discovery -- */
   QString m_discoveryUrl;
-  qint32 m_discoveryInterval = 30000; // ms
   QTimer m_discoveryTimer;
   bool m_discovering = false;
   QList<QOpcUaApplicationDescription> m_discoveredServers;