Skip to content
  • jim warner's avatar
    library: expunge that 'info' parameter in 'VAL' macros · 967fdcfb
    jim warner authored and Craig Small's avatar Craig Small committed
    When that issue referenced below was raised, and after
    my initial misstep, I suggested the 'info' params were
    added to those VAL macros simply for symmetry with the
    GET macros (which indeed require an 'info' parameter).
    
    However, in the back of my mind I hoped their presence
    was actually justified through an earlier incarnation.
    
    But that was a false hope because the 'info' parameter
    has always been part of the VAL macros ever since that
    'xtra-procps-debug.h' file was added way back in 2016.
    
    Even worse, a later commit added some casts to void so
    the unused variable compiler warnings were eliminated.
    That, perhaps, should have prompted some reassessment.
    
    So this patch will correct my earlier flawed decision.
    
    Reference(s):
    #332
    . Aug, 2016 - introduced XTRA_PROCPS_DEBUG
    commit e3270d46
    . Jun, 2018 - added '(void)info' to sources
    commit 97d078a9
    
    
    
    Signed-off-by: default avatarJim Warner <james.warner@comcast.net>
    967fdcfb