Otpvondoiats commented on PR #13536: URL: https://github.com/apache/nuttx/pull/13536#issuecomment-2370872746
> > > > @yamt please review the new implementation which disable the extension by default. > > > > > > > > > i feel it's difficult to use a kconfig which alters the behavior of very basic things like printf. when you want to enable it, you need to audit every uses of printf family functions in your system. > > > isn't it simper to provide an alternative api, say, nuttx_printf, to provide the extension? (similarly to linux printk) > > > > > > hiļ¼This type of API is a special interface, and different systems implement it differently. If add a new API, also need to enable a similar kconfig to control the opening of the interface. > > the point is that it can be done w/o affecting users of the standard printf family. Yes, there is no problem with the new API, but the current patch is only to solve the uorblistener printing problem. The new interface plan can be added and updated later, and this work is a big project. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
