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