/** @file report_util.c
 *
 * Functions for creating messages in a standardized format, with flexible
 * indentation.
 *
 * The functions in this source file are thread safe.
 *
 * TODO: describe
 * - indentation depth
 *     - indentation stack
 * - destination stack
 */

// Copyright (C) 2014-2023 Sanford Rockowitz <rockowitz@minsoft.com>
// SPDX-License-Identifier: GPL-2.0-or-later

/** \cond */
#include <assert.h>
#include <glib-2.0/glib.h>
#include <stdarg.h>
#include <stdbool.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
/** \endcond */

#include "coredefs_base.h"
#include "file_util_base.h"
#include "string_util.h"

#include "report_util.h"


#define DEFAULT_INDENT_SPACES_PER_DEPTH 3
#define INDENT_SPACES_STACK_SIZE  16
#define OUTPUT_DEST_STACK_SIZE 8

#ifdef OLD
static int indent_spaces_stack[INDENT_SPACES_STACK_SIZE];
static int indent_spaces_stack_pos = -1;

static FILE* output_dest_stack[OUTPUT_DEST_STACK_SIZE];
static int   output_dest_stack_pos = -1;

// work around for fact that can't initialize the initial stack entry to stdout
static FILE* alt_initial_output_dest = NULL;
static bool  initial_output_dest_changed = false;
#endif

static FILE* default_output_dest;

/** Sets the initial report output destination for newly created threads.
 *
 *  Note this does not change the report output destination for existing threads.
 *
 *  @param output_dest  default output destination
 */
void rpt_set_default_output_dest(FILE* output_dest) {
   default_output_dest = output_dest;
}


//* Thread specific state */
typedef struct {
   uint8_t indent_spaces_stack[INDENT_SPACES_STACK_SIZE];
   int indent_spaces_stack_pos;    // initial  -1;

   FILE* output_dest_stack[OUTPUT_DEST_STACK_SIZE];
   int   output_dest_stack_pos;   // initial  -1;

   // work around for fact that can't initialize the initial stack entry to stdout
   FILE* alt_initial_output_dest;     // initial NULL;
   bool  initial_output_dest_changed; // initial false;
} Per_Thread_Settings;


/** Returns a struct for maintaining thread-specific settings
 *  @return thread-specific struct of global settings
 */
static Per_Thread_Settings *  get_thread_settings() {
   static GPrivate per_thread_settings_key = G_PRIVATE_INIT(g_free);

   Per_Thread_Settings* settings = g_private_get(&per_thread_settings_key);

   // GThread * this_thread = g_thread_self();
   // printf("(%s) this_thread=%p, settings=%p\n", __func__, this_thread, settings);

   if (!settings) {
      settings = g_new0(Per_Thread_Settings, 1);
      settings->indent_spaces_stack_pos = -1;
      settings->output_dest_stack_pos   = -1;

      if (default_output_dest)
         settings->output_dest_stack[++settings->output_dest_stack_pos] = default_output_dest;

      g_private_set(&per_thread_settings_key, settings);
   }

   // printf("(%s) Returning: %p\n", __func__, settings);
   return settings;
}


//
// Indentation
//

// Functions that allow for temporarily changing the number of indentation
// spaces per logical indentation depth on the current thread.
// 10/16/15: not currently used

/** Sets the spaces-per-indentation-depth to be used for report functions.
 *  The current spaces-per-depth is saved on the thread-specific spaces-per-depth stack.
 *
 *  @param new_dest  new output destination
 */
void rpt_push_indent(int new_spaces_per_depth) {
   Per_Thread_Settings * settings = get_thread_settings();
   assert(settings->indent_spaces_stack_pos < INDENT_SPACES_STACK_SIZE-1);
   settings->indent_spaces_stack[++settings->indent_spaces_stack_pos] = new_spaces_per_depth;
}


/** Pops the space-per-indentation-depth stack.
 *
 *  @remark
 *  No effect if stack is empty.
 */
void rpt_pop_indent() {
   Per_Thread_Settings * settings = get_thread_settings();
   if (settings->indent_spaces_stack_pos >= 0)
      settings->indent_spaces_stack_pos--;
}


/** Empties the space-per-indentation-depth stack.
 *
 *  The effect is to reset the per-thread spaces-per-indentation-depth
 *  to its default value.
 */
void rpt_reset_indent_stack() {
   Per_Thread_Settings * settings = get_thread_settings();
   settings->indent_spaces_stack_pos = -1;
}


/** Given a logical indentation depth, returns the number of spaces
 *  of indentation to be used.
 *
 *  @param depth logical indentation depth, if < 0 perform no indentation
 *  @return number of indentation spaces
 */
int rpt_get_indent(int depth) {
   if (depth < 0)
      depth = 0;
   Per_Thread_Settings * settings = get_thread_settings();
   int spaces_ct = DEFAULT_INDENT_SPACES_PER_DEPTH;
   if (settings->indent_spaces_stack_pos >= 0)
      spaces_ct = settings->indent_spaces_stack[settings->indent_spaces_stack_pos];
   return depth * spaces_ct;
}


// Functions that allow for temporarily changing the output destination
// on the current thread.

/** Sets the output destination to be used for report functions.
 *  The current output destination is saved on the output destination stack.
 *
 *  @param new_dest  new output destination
 */
void rpt_push_output_dest(FILE* new_dest) {
   Per_Thread_Settings * settings = get_thread_settings();
   assert(settings->output_dest_stack_pos < OUTPUT_DEST_STACK_SIZE-1);
   settings->output_dest_stack[++settings->output_dest_stack_pos] = new_dest;
}


/** Pops the output destination stack, and sets the output destination
 *  to be used for report functions to the new top of the stack.
 */
void rpt_pop_output_dest() {
   Per_Thread_Settings * settings = get_thread_settings();
   if (settings->output_dest_stack_pos >= 0)
      settings->output_dest_stack_pos--;
}


/** Clears the output destination stack.
 * The output destination to be used for report functions to the default (stdout).
 */
void rpt_reset_output_dest_stack() {
   Per_Thread_Settings * settings = get_thread_settings();
   settings->output_dest_stack_pos = -1;
}


/** Gets the current output destination.
 *
 * @return current output destination
 */
FILE * rpt_cur_output_dest() {
   Per_Thread_Settings * settings = get_thread_settings();
   // special handling for unpushed case because can't statically initialize
   // output_dest_stack[0] to stdout
   FILE * result = NULL;
   if (settings->output_dest_stack_pos < 0)
      result = (settings->initial_output_dest_changed) ? settings->alt_initial_output_dest : stdout;
   else
      result = settings->output_dest_stack[settings->output_dest_stack_pos];
   return result;
}


/** Debugging function to show output destination.
 */
void rpt_debug_output_dest() {
    Per_Thread_Settings * settings = get_thread_settings();
    FILE * dest = rpt_cur_output_dest();
    char * addl = (dest == stdout) ? " (stdout)" : "";
    printf("(%s) output_dest_stack[%d] = %p %s\n",
          __func__, settings->output_dest_stack_pos, (void*)dest, addl);
}


/** Changes the current output destination, without saving
 * the current output destination on the destination stack.
 *
 * @param new_dest new output destination
 *
 * @remark Needed for set_fout() in core.c
 */
void rpt_change_output_dest(FILE* new_dest) {
   Per_Thread_Settings * settings = get_thread_settings();
   if (settings->output_dest_stack_pos >= 0)
      settings->output_dest_stack[settings->output_dest_stack_pos] = new_dest;
   else {
      settings->initial_output_dest_changed = true;
      settings->alt_initial_output_dest = new_dest;
   }
}


// should not be needed, for diagnosing a problem
void rpt_flush() {
   fflush(rpt_cur_output_dest());
}


/** Writes a newline to the current output destination.
 */
void rpt_nl() {
   f0printf(rpt_cur_output_dest(), "\n");
}


/** Writes a constant string to the current output destination, or adds
 *  the string to a specified GPtrArray.
 *
 * A newline is appended to the string specified.
 *
 * The output is indented per the specified indentation depth.
 *
 * @param title     string to write
 * @param collector if non-NULL, add string to this GPtrArray instead of
 *                  writing it to the current output destination
 * @param depth     logical indentation depth.
 *
 * @remark This is the core function through which all output is funneled.
 */
void rpt_title_collect(const char * title, GPtrArray * collector, int depth) {
   bool debug = false;
   if (debug)
      printf("(%s) Writing to %p\n", __func__, (void*)rpt_cur_output_dest());

   if (collector) {
      g_ptr_array_add(collector, g_strdup_printf("%*s%s\n", rpt_get_indent(depth), "", title));
   }
   else {
      f0printf(rpt_cur_output_dest(), "%*s%s\n", rpt_get_indent(depth), "", title);
   }
}


/** Writes a constant string to the current output destination.
 *
 * A newline is appended to the string specified.
 *
 * The output is indented per the specified indentation depth.
 *
 * @param title string to write
 * @param depth logical indentation depth.
 */
void rpt_title(const char * title, int depth) {
   rpt_title_collect(title, NULL, depth);
}


/** Writes a constant string to the current output destination, or
 *  adds it to a GPtrArray if one is given.
 *
 * A newline is appended to the string specified.
 *
 * The output is indented per the specified indentation depth.
 *
 * @param depth     logical indentation depth.
 * @param collector if non-NULL, append string to this GPtrArray
 *                  instead of writing it to the current output device
 * @param title     string to write
 */
void rpt_label_collect(int depth, GPtrArray * collector, const char * text) {
   rpt_title_collect(text, collector, depth);
}


/** Writes a constant string to the current output destination.
 *
 * A newline is appended to the string specified.
 *
 * The output is indented per the specified indentation depth.
 *
 * @param depth logical indentation depth.
 * @param title string to write
 *
 * @remark
 * This function is logically equivalent to #rpt_title(), except that
 * the **depth** parameter is first, not last.
 * Experience wih the API has shown that #rpt_title() tends not to be
 * used along with #rpt_vstring() because the different position of the
 * **depth** parameter makes the code harder to read.
 */
void rpt_label(int depth, const char * text) {
   rpt_title(text, depth);
}


/** Writes a formatted string to the current output destination.
 *
 * A newline is appended to the string specified
 *
 * @param  depth    logical indentation depth
 * @param  format   format string (normal printf)
 * @param  ...      arguments
 *
 * @remark Note that the depth parm is first on this function because of variable args
 */
void rpt_vstring(int depth, char * format, ...) {
   int buffer_size = 200;
   char buffer[buffer_size];
   char * buf = buffer;
   va_list(args);
   va_start(args, format);
   int reqd_size = vsnprintf(buffer, buffer_size, format, args);
   // if buffer wasn't sufficiently large, allocate a temporary buffer
   if (reqd_size >= buffer_size) {
      // printf("(%s) Allocating temp buffer, reqd_size=%d\n", __func__, reqd_size);
      buf = malloc(reqd_size+1);
      va_start(args, format);
      vsnprintf(buf, reqd_size+1, format, args);
   }
   va_end(args);

   rpt_title(buf, depth);

   if (buf != buffer)
      free(buf);
}


/** Writes a formatted string to the current output destination, or
 *  adds the string to a collector array.
 *
 * A newline is appended to the string specified
 *
 * @param  depth     logical indentation depth
 * @param  collector if non-NULL, add string to GPtrArray instead of
 *                   writing to current output destination
 * @param  format    format string (normal printf argument)
 * @param  ap        va_list array of arguments
 *
 * @remark Note that the depth parm is first on this function because of variable args
 */
void vrpt_vstring_collect(int depth, GPtrArray * collector, char * format, va_list ap) {
   char * s = g_strdup_vprintf(format, ap);
   rpt_title_collect(s, collector, depth);
   free(s);
}


/** Writes a formatted string to the current output destination.
 *
 * A newline is appended to the string specified
 *
 * @param  depth     logical indentation depth
 * @param  collector if non-NULL, add string to GPtrArray instead of
 *                   writing to current output destination
 * @param  format    format string (normal printf argument)
 * @param  ...       arguments
 *
 * @remark Note that the depth parm is first on this function because of variable args
 */
void rpt_vstring_collect(int depth, GPtrArray* collector, char * format, ...) {
   va_list(args);
   va_start(args, format);
   vrpt_vstring_collect(depth, collector, format, args);
   va_end(args);
}



/** Convenience function that writes multiple constant strings.
 *
 *  @param depth    logical indentation depth
 *  @param ...      pointers to constant strings,
 *                  last pointer is NULL to terminate list
 */
void rpt_multiline(int depth, ...) {
   va_list args;
   va_start(args, depth);
   char * s = NULL;
   while( (s = va_arg(args, char *)) != NULL) {
      rpt_title(s, depth);
   }
   va_end(args);
}


/** Writes all strings in a GPtrArray to the current report destination
 *
 * @param  depth   logical indentation depth
 * @param  strings pointer to GPtrArray of strings
 */
void rpt_g_ptr_array(int depth, GPtrArray * strings) {
   for (int ndx = 0; ndx < strings->len; ndx++) {
      char * s = g_ptr_array_index(strings, ndx);
      rpt_title(s, depth);
   }
}


/** Writes a hex dump with indentation.
 *  Output is written to the current report destination
 *
 *  @param data  start of bytes to dump
 *  @param size  number of bytes to dump
 *  @param depth logical indentation depth
 */
void rpt_hex_dump(const Byte * data, int size, int depth) {
   fhex_dump_indented(rpt_cur_output_dest(), data, size, rpt_get_indent(depth));
}


/** Writes a Null_Terminated_String_Array with indentation.
 *  Output is written to the current report destination.
 *
 *  @param ntsa  array to report
 *  @param depth logical indentation depth
 */
void rpt_ntsa(Null_Terminated_String_Array ntsa, int depth) {
   assert(ntsa);
   for (int ndx=0; ntsa[ndx]; ndx++) {
      rpt_vstring(depth, "%s", ntsa[ndx]);
   }
}


/** Writes a string to the current output destination, describing a pointer
 *  to a named data structure.
 *
 *  The output is indented per the specified indentation depth.
 *
 *  @param name  struct name
 *  @param ptr   pointer to struc
 *  @param depth logical indentation depth
 */
void rpt_structure_loc(const char * name, const void * ptr, int depth) {
   // fprintf(rpt_cur_output_dest(), "%*s%s at: %p\n", rpt_indent(depth), "", name, ptr);
   rpt_vstring(depth, "%s at: %p", name, ptr);
}


/** Writes a pair of strings to the current output destination.
 *
 *  If offset_absolute is true, then the s2 value will start in the same column,
 *  irrespective of the line indentation.   This may make some reports easier to read.
 *
 *  @param s1 first string
 *  @param s2 second string
 *  @param col2offset  offset from start of line where s2 starts
 *  @param offset_absolute  if true,  col2offset is relative to the start of the line, before indentation
 *                          if false, col2offset is relative to the indented start of s1
 *  @param depth logical indentation depth
 */
void rpt_2col(char * s1,  char * s2,  int col2offset, bool offset_absolute, int depth) {
   int col1sz = col2offset;
   int indentct = rpt_get_indent(depth);
   if (offset_absolute)
      col1sz = col1sz - indentct;
   rpt_vstring(depth, "%-*s%s", col1sz, s1, s2 );
}


/** Reports the contents of a file.
 *
 *  @param   fn        name of file
 *  @param   verbose   if true, emit message if error reading file
 *  @param   depth     logical indentation depth
 *  @retval >=0        number of lines read
 *  @retval <0         -errno from fopen() or getline()
 */
int rpt_file_contents(const char * fn, bool verbose, int depth) {
   GPtrArray * line_array = g_ptr_array_new();
   g_ptr_array_set_free_func(line_array, g_free);
   int rc = file_getlines(fn, line_array, false);
   if (rc < 0) {
      if (verbose)
         rpt_vstring(depth, "Error reading file %s: %s", fn, strerror(-rc));
   }
   else if (rc == 0) {
      if (verbose)
         rpt_vstring(depth, "Empty file: %s", fn);
   }
   else if (rc > 0) {
      int ndx = 0;
      for (; ndx < line_array->len; ndx++) {
         char * curline = g_ptr_array_index(line_array, ndx);
         // trim_in_place(curline);     // strip trailing newline - now done in file_getlines()
         rpt_title(curline, depth);
      }
   }
   g_ptr_array_free(line_array, true);
   return rc;
}


/* The remaining rpt_ functions various data types share a common formatting so that they can
 * be use together.  All channel their output through rpt_str().
 *
 * Depending on whether the info parm is null, output takes one of the following forms:
 *    name       (info) : value
 *    name              : value
 */

/** Writes a string to the current output destination describing a named character string value.
 *
 * The output is indented per the specified indentation depth.
 *
 * Optionally, a description string can be specified along with the name.
 * The description string will be surrounded by parentheses.
 *
 * The string value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param val   string value
 * @param depth logical indentation depth
 */
void rpt_str(const char * name, char * info, const char * val, int depth) {
   bool debug = false;
   if (debug)
      printf("(%s) Writing to %p\n", __func__, (void*)rpt_cur_output_dest());

   char infobuf[100];
   if (info)
      snprintf(infobuf, 99, "(%s)", info);
   else
      infobuf[0] = '\0';
   rpt_vstring(depth, "%-25s %30s : %s", name, infobuf, val);
}


/** Writes a string to the current output destination describing a boolean value.
 *
 * The value is displayed as "true" or "false".
 *
 * The output is indented per the specified indentation depth.
 * The description string will be surrounded by parentheses.
 *
 * The value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param val   value to show
 * @param depth logical indentation depth
 *
 * The value is formatted as "true" or "false".
 */
void rpt_bool(char * name, char * info, bool val, int depth) {
   char * valName = (val) ? "true" : "false";
   rpt_str(name, info, valName, depth);
}


/** Writes a string to the current output destination, describing a named integer value.
 *
 * The output is indented per the specified indentation depth.
 *
 * Optionally, a description string can be specified along with the name.
 * The description string will be surrounded by parentheses.
 *
 * The value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param val   integer value
 * @param depth logical indentation depth
 */
void rpt_int(char * name, char * info, int val, int depth) {
   char buf[10];
   snprintf(buf, 9, "%d", val);
   rpt_str(name, info, buf, depth);
}

/** Writes a string to the current output destination, describing a named unsigned integer value.
 *
 * The output is indented per the specified indentation depth.
 *
 * Optionally, a description string can be specified along with the name.
 * The description string will be surrounded by parentheses.
 *
 * The value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param val   unsigned integer value
 * @param depth logical indentation depth
 */
void rpt_unsigned(char * name, char * info, int val, int depth) {
   char buf[10];
   snprintf(buf, 9, "%u", val);
   rpt_str(name, info, buf, depth);
}



/** Writes a string to the current output destination describing a 4 byte integer value,
 * indented per the specified indentation depth.
 *
 * The integer value is formatted as printable hex.
 *
 * Optionally, a description string can be specified along with the name.
 * The description string will be surrounded by parentheses.
 *
 * The value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param val   integer value
 * @param depth logical indentation depth
 */
void rpt_int_as_hex(char * name, char * info, int val, int depth) {
   char buf[16];
   snprintf(buf, 15, "0x%08x", val);
   rpt_str(name, info, buf, depth);
}


/** Writes a string to the current output destination describing a single byte value,
 * indented per the specified indentation depth.
 *
 * The value is formatted as printable hex.
 *
 * Optionally, a description string can be specified along with the name.
 * The description string will be surrounded by parentheses.
 *
 * The value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param val   value
 * @param depth logical indentation depth
 */
void rpt_uint8_as_hex(char * name, char * info, unsigned char val, int depth) {
   char buf[16];
   snprintf(buf, 15, "0x%02x", val);
   rpt_str(name, info, buf, depth);
}


/** Writes a string to the current output destination describing a named integer
 * value having a symbolic string representation.
 *
 * The output is indented per the specified indentation depth.
 *
 * The integer value is converted to a string using the specified function.
 *
 * Optionally, a description string can be specified along with the name.
 * The description string will be surrounded by parentheses.
 *
 * The value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param val   integer value
 * @param func  interpretation function
 * @param depth logical indentation depth
 */
void rpt_mapped_int(char * name, char * info, int val, Value_To_Name_Function func, int depth)  {
   char * valueName = func(val);
   char buf[100];
   snprintf(buf, 100, "%d - %s", val, valueName);
   rpt_str(name, info, buf, depth);
}


/** Writes a string to the current output destination describing a sequence of bytes,
 * indented per the specified indentation depth.
 *
 * The value is formatted as printable hex.
 *
 * Optionally, a description string can be specified along with the name.
 * The description string will be surrounded by parentheses.
 *
 * The value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param bytes pointer to start of bytes to show
 * @param ct    number of bytes to show
 * @param hex_prefix_flag if true, the printable hex value will begin with "0x"
 * @param depth logical indentation depth
 */
void rpt_bytes_as_hex(
        const char *   name,
        char *   info,
        Byte *   bytes,
        int      ct,
        bool     hex_prefix_flag,
        int      depth) {
   // printf("(%s) bytes=%p, ct=%d\n", __func__, bytes, ct);
   int bufsz = 2*ct + 1;
   bufsz++;   // hack
   if (hex_prefix_flag)
      bufsz += 2;
   char * buf = malloc(bufsz);
   char * hex_prefix = (hex_prefix_flag) ? "0x"  : "";
   char * hs = hexstring(bytes, ct);

   snprintf(buf, bufsz-1, "%s%s", hex_prefix, hs);
   rpt_str(name, info, buf, depth);
   free(buf);
   free(hs);
}


//  Functions for reporting integers that are collections of named bits

#ifdef DEBUG
static
void report_flag_info( Flag_Info* pflag_info, int depth) {
   assert(pflag_info);
   rpt_structure_loc("FlagInfo", pflag_info, depth);
   int d1 = depth+1;
   rpt_str( "flag_name", NULL, pflag_info->flag_name, d1);
   rpt_str( "flag_info", NULL, pflag_info->flag_info, d1);
   rpt_int_as_hex("flag_val",  NULL, pflag_info->flag_val,  d1);
}


/* Function for debugging findFlagInfoDictionary.
 *
 * Reports the contents of a FlagDictionay record.
 */
static
void report_flag_info_dictionary(Flag_Dictionary* pDict, int depth) {
   assert(pDict);
   rpt_structure_loc("Flag_Dictionary", pDict, depth);
   int d1 = depth+1;
   rpt_int("flag_info_ct", NULL, pDict->flag_info_ct, d1);
   int ndx=0;
   for(;ndx < pDict->flag_info_ct; ndx++) {
      report_flag_info(&pDict->flag_info_recs[ndx], d1);
   }
}
#endif


static
Flag_Info * find_flag_info_in_dictionary(char * flag_name, Flag_Dictionary * pdict) {
   Flag_Info * result = NULL;
   // printf("(%s) Starting.  flag_name=%s, pdict=%p   \n", __func__, flagName, pdict );
   // printf("(%s) pdict->flag_info_ct=%d  \n", __func__, pdict->flag_info_ct );
   // report_flag_info_dictionary(pdict, 2);
   int ndx;
   for (ndx=0; ndx < pdict->flag_info_ct; ndx++) {
      // printf("(%s) ndx=%d  \n", __func__, ndx );
      // Flag_Info pcur_info = &(pdict->flag_info_recs[ndx]);
      // printf("(%s) pdict->flag_info_recs[ndx].flag_name=%s    \n", __func__, pdict->flag_info_recs[ndx].flag_name );
      if ( streq(flag_name, pdict->flag_info_recs[ndx].flag_name)) {
         // printf("(%s) Match  \n", __func__ );
         result =  &pdict->flag_info_recs[ndx];
         break;
      }
   }
   // printf("(%s) Returning: %p  \n", __func__, result );
   return result;
}


static
void char_buf_append(char * buffer, int bufsize, char * val_to_append) {
   assert(strlen(buffer) + strlen(val_to_append) < bufsize);
   strcat(buffer, val_to_append);
}


static
void flag_val_to_string_using_dictionary(
        int                flags_val,
        Flag_Name_Set *    pflag_name_set,
        Flag_Dictionary *  pdict,
        char *             buffer,
        int                bufsize )
{
   // printf("(%s) flagsVal=0x%02x, pflagNameSet=%p, pDict=%p \n", __func__, flagsVal, pflagNameSet, pDict );
   // printf("(%s) pflagNameSet->flagNameCt=%d  \n", __func__, pflagNameSet->flagNameCt );
   // printf("(%s) pDict->flagInfoCt=%d  \n", __func__, pDict->flagInfoCt );
   assert(buffer && bufsize > 1);
   buffer[0] = '\0';
   int ndx;
   bool first = true;
   for (ndx=0; ndx <  pflag_name_set->flag_name_ct; ndx++) {
      Flag_Info * pflag_info = find_flag_info_in_dictionary(pflag_name_set->flag_names[ndx], pdict);
      // printf("(%s) ndx=%d, pFlagInfp=%p   \n", __func__, ndx, pFlagInfo );
      if (flags_val & pflag_info->flag_val) {
         if (first)
            first = false;
         else
            char_buf_append(buffer, bufsize, ", ");
         char_buf_append(buffer, bufsize, pflag_info->flag_name);
      }
   }
   // printf("(%s) Returning |%s|\n", __func__, buffer );
}


/** Writes a string to the current output destination describing an integer
 * that is to be interpreted as a named collection of named bits.
 *
 * Output is indented per the specified indentation depth.
 * The description string will be surrounded by parentheses.
 *
 * The value is prefixed with a colon.
 *
 * @param name  name of value
 * @param info  if non-null, description of value
 * @param val   value to interpret
 * @param p_flag_name_set
 * @param p_dict
 * @param depth logical indentation depth
 */
void rpt_ifval2(char*           name,
               char*            info,
               int              val,
               Flag_Name_Set*   p_flag_name_set,
               Flag_Dictionary* p_dict,
               int              depth)
{
   char buf[1000];
   buf[0] = '\0';
   snprintf(buf, 7, "0x%04x", val);
   char_buf_append(buf, sizeof(buf), " - ");
   flag_val_to_string_using_dictionary(val, p_flag_name_set, p_dict, buf, sizeof(buf));
   rpt_str(name, info, buf, depth);
}
