Put @internal command in the right place

src/group_internal.h

@@ -1,5 +1,5 @@
-/// @file group_internal.h
 /// @internal
+/// @file group_internal.h
 /// @brief  Internal functions for representation of groups
 /// @license This file is under GNU General Public License 3.0 - see LICENSE.txt
 

src/internal_format.h

@@ -1,6 +1,6 @@
 /**
- * @file    internal_format.h
  * @internal
+ * @file    internal_format.h
  * @brief   internal format (FIXME: derived from filename)
  * @license GNU General Public License 3.0 - see LICENSE.txt
  */

src/message_api.c

@@ -4947,18 +4947,19 @@ static const char* process_key_claim(message* src,
     return sender_key;
 }
 
-// Rule for this function, since it is one of the three most complicated functions in this whole damned
-// business:
-//
-// If you calculate a status from something and expect it NOT to be fatal, once you are done USING that status,
-// you MUST set it back to "PEP_STATUS_OK".
-//
-// There are times when we don't want errors during calls to be fatal. Once any action is taken on that
-// status, if we are going to continue processing and not bail from the message, the status needs to be reset
-// to PEP_STATUS_OK, or, alternately, we need to be using a temp status variable.
-//
-// This internal function does *not* set the rating field of the message: that
-// part of the job is within decrypt_message.
+/// @internal
+/// Rule for this function, since it is one of the three most complicated functions in this whole damned
+/// business:
+///
+/// If you calculate a status from something and expect it NOT to be fatal, once you are done USING that status,
+/// you MUST set it back to "PEP_STATUS_OK".
+///
+/// There are times when we don't want errors during calls to be fatal. Once any action is taken on that
+/// status, if we are going to continue processing and not bail from the message, the status needs to be reset
+/// to PEP_STATUS_OK, or, alternately, we need to be using a temp status variable.
+///
+/// This internal function does *not* set the rating field of the message: that
+/// part of the job is within decrypt_message.
 static PEP_STATUS _decrypt_message(
         PEP_SESSION session,
         message *src,

src/message_api_internal.h

@@ -1,5 +1,5 @@
-/// @file message_api_internal.h
 /// @internal
+/// @file message_api_internal.h
 /// @author Created by Krista Bennett on 14.04.21.
 ///
 

src/pEpEngine_internal.h

@@ -1,6 +1,6 @@
 /**
- * @file    pEpEngine_internal.h
  * @internal
+ * @file    pEpEngine_internal.h
  * @brief   Exposed internal functions and structures.
  * @license GNU General Public License 3.0 - see LICENSE.txt
  */

src/pgp_netpgp.h

@@ -1,6 +1,6 @@
 /**
- * @file    src/pgp_netpgp.h
  * @internal
+ * @file    src/pgp_netpgp.h
  * 
  * @brief   NETPGP driver - implements required cryptotech 
  *          functions for the engine using netpgp with SSL 

src/platform_disable_attributes.h

@@ -2,8 +2,8 @@
 #define PLATFORM_DISABLE_ATTRIBUTES_H
 
 /**
- * @file platform_disable_attributes.h
  * @internal
+ * @file platform_disable_attributes.h
  *
  * @brief Disable GNU-style attributes for functions, variables, types and label
  *  (https://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html ), turning them

src/trans_auto.h

@@ -1,6 +1,6 @@
 /**
- * @file    src/trans_auto.h
  * @internal
+ * @file    src/trans_auto.h
  * @brief   transport auto functions? (FIXME: derived from filename)
  * @license GNU General Public License 3.0 - see LICENSE.txt
  */