On Wed, Dec 20, 2023 at 2:46 PM Thomas Monjalon <tho...@monjalon.net> wrote: > > 18/12/2023 15:38, David Marchand: > > Convert many libraries that call RTE_LOG(... "\n", ...) to RTE_LOG_LINE. > > > > Note: > > - for acl and sched libraries that still has some debug multilines > > messages, a direct call to RTE_LOG is used: this will make it easier to > > notice such special cases, > > [...] > > 129 files changed, 2280 insertions(+), 2279 deletions(-) > > It would be nice to keep RTE_LOG() only for multi-line as you do, > and use RTE_LOG_LINE only in wrapper macros specific per library. > This patch is using RTE_LOG_LINE directly in multiple places > in libraries which do not have a dedicated wrapper macro. > Would it be possible to introduce new wrappers for these libraries > in this patch?
It is doable and not too difficult to achieve with some scripting. I'll try in the v5 (since I need to send a new revision for build with -pedantic). -- David Marchand
