[dpdk-dev] [PATCH v2 6/9] app/procinfo: add code for debug crypto

Varghese, Vipin vipin.varghese at intel.com
Mon Oct 29 15:29:19 CET 2018


Hi Stepehen,

<snipped>

> > > This is a very big macro, better have static function for this instead of
> macro.
> > >
> >
> > There are two thoughts in choosing MACRO over function.
> > 1. The information need to display in certain format within the same context.
> > 2. As the API are modified, co locating all at same scope is easier to clean up
> and correct in future.
> >
> > So I feel use of MACRO over function in this instance.
> 
> 
> I don't agree with your arguments. Macros, are ugly and error prone. This is
> not performance critical so it should be a function. The only reason to use
> macro's is if it is not possible to write it as a function (as in a template for code
> generation).

Thanks for the sharing your feedback. But In my comments I never stated the MACRO were used for performance or critical blocks. As mentioned in my explanation, the MACRO is used for printing the desired format and format the code. 

Since I have received 2 request for change, I will do the same and release v3.

Thanks
Vipin Varghese


More information about the dev mailing list