RE: Verbosity Control

Bishupriya,

See below:

Also, what do you think of adding sc_gen_report as proposed by Philipp? I see no reason to do so right now.

John A

-----Bishnupriya Bhattacharya <bpriya@cadence.com> wrote: -----
To: "john.aynsley@doulos.com" <john.aynsley@doulos.com>
From: Bishnupriya Bhattacharya <bpriya@cadence.com>
Date: 12/07/2010 05:41PM
Cc: Bishnupriya Bhattacharya <bpriya@cadence.com>
Subject: RE: Verbosity Control

    
John,
 
 
1) The enum SC_IGNORE_VERBOSITY = 9999 was proposed by Alan for the case where we have only one report() signature with an optional verbosity argument at the end. We are not going with that signature. Philip had also not liked this enum. So I don't think we need this enum.

[JA] SEE SEPARATE EMAIL TO ALAN
 
 
2) I agree on your point to not change the existing macros to supply verbosity argument - that can be misleading. However, I don't like that some SC_INFO messages are subject to verbosity check and some are not. This is not consistent. Its better - as per the original proposal - that the existing report() signature assigns a default verbosity value of 200 to the INFO messages and check these against max verbosity. Then all INFO messages have a verbosity property - either explicitly user assigned or an implicit default value, and all INFO messages are subject to verbosity check. This is nice and clean and fully backward compatible. 

[JA]  I'm not sure I understand. Are you saying change the definitions of the current SC_REPORT_INFO macro to call the overloaded sc_report_handler::report method with a verbosity, but leave SC_REPORT_WARNING/ERROR/FATAL unchanged? I would be fine with that.
 
 
3)  About the new macro, I like adding SC_REPORT_INFO_VERB(msg_type, msg, verbosity).
 
Users will either do
 
   sc_report_handler::report(SC_INFO, "myid", "mymsg", 500, __FILE__, __LINE__);
   sc_report_handler::report(SC_WARNING, "myid", "msg", __FILE__, __LINE__);
 
or
 
  SC_REPORT_INFO_VERB("myid", "mymsg", 500);
  SC_REPORT_WARNING("myid", "msg");

[JA] Yes, this is how I understood it.
 
 
We should have this discussion on the reflector also.
 
Thanks,
-Bishnupriya

-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.