Re: [PATCH] log: add l_notice

Marcel Holtmann <[email protected]> Wed, 14 Feb 2024 17:27:42 +0100
Newsgroups dev.linux.lists.ell
Message-ID <[email protected]>
Hi Denis,

>>> #define L_LOG_WARNING 4
>>> +#define L_LOG_NOTICE 5
>>> #define L_LOG_INFO 6
>>> #define L_LOG_DEBUG 7
>> for what? What is the difference to just use l_info?
> 
> Per conversation on Slack, James wants to categorize certain messages for easier searching.  Since NOTICE level exists, I suggested it would fit the usecase.

I never really liked having many different levels. That was the reason I left l_notice out initially. I see no real benefit of it except us discussing "is this suppose to be l_info and l_notice" and in the end it will be just random.

So if you tell me / convince me when to use l_notice of l_info or use l_warn instead of l_notice, then we can talk.

Regards

Marcel