Code cleanup in an mbglib file

1 files changed, 8 insertions, 5 deletions

diff --git a/mbglib/common/mbg_xmr_util.c b/mbglib/common/mbg_xmr_util.c
index b31887f..94aab8f 100644
--- a/mbglib/common/mbg_xmr_util.c
+++ b/mbglib/common/mbg_xmr_util.c
@@ -1,7 +1,7 @@
 
 /**************************************************************************
  *
- *  $Id: mbg_xmr_util.c 1.2 2022/06/30 09:44:23 martin.burnicki REL_M $
+ *  $Id: mbg_xmr_util.c 1.3 2022/07/06 14:56:09 martin.burnicki REL_M $
  *
  *  Copyright (c) Meinberg Funkuhren, Bad Pyrmont, Germany
  *
@@ -9,6 +9,8 @@
  *
  * -----------------------------------------------------------------------
  *  $Log: mbg_xmr_util.c $
+ *  Revision 1.3  2022/07/06 14:56:09  martin.burnicki
+ *  Code cleanup.
  *  Revision 1.2  2022/06/30 09:44:23  martin.burnicki
  *  Added some doxygen comments and made a function static.
  *  Revision 1.1  2022/06/24 12:54:33  martin.burnicki
@@ -62,11 +64,12 @@ static /*HDR*/
  */
 int snprint_xmr_src_status( char *s, size_t max_len, unsigned xmr_status )
 {
-  s[0] = 0;  // Make string empty.
   int n = 0;
   bool seen = false;
   int i;
 
+  s[0] = 0;  // Make string empty.
+
   for ( i = 0; i < N_XMRS_BITS; i++ )
   {
     if ( xmr_status & 0x01 )
@@ -99,7 +102,7 @@ void print_all_xmr_status( ALL_XMULTI_REF_STATUS *p_axrs, int n_src,
                            const char *info, const char *indent_str,
                            int verbosity_level )
 {
-  int max_ref_name_len = 0;
+  size_t max_ref_name_len = 0;
   int i;
 
   // Determine the maximum length of the names of the
@@ -114,7 +117,7 @@ void print_all_xmr_status( ALL_XMULTI_REF_STATUS *p_axrs, int n_src,
       size_t l = strlen( multi_ref_names[ref_type] );
 
       if ( l > max_ref_name_len )
-        max_ref_name_len = (int) l;
+        max_ref_name_len = l;
     }
   }
 
@@ -139,7 +142,7 @@ void print_all_xmr_status( ALL_XMULTI_REF_STATUS *p_axrs, int n_src,
     n += snprintf_safe( &s[n], sz - n, " (%s):", ( ref_type < N_MULTI_REF ) ?
                         multi_ref_names[ref_type] : "unknown" );
 
-    n += snprintf_safe( &s[n], sz - n, "%*s", 19 + max_ref_name_len - n, "" );
+    n += snprintf_safe( &s[n], sz - n, "%*s", (int) ( 19 + max_ref_name_len - n ), "" );
 
     n += snprintf_safe( &s[n], sz - n, "offs: " );
     n += snprint_nano_time( &s[n], sz - n, &p_xrs->offset );