Brijesh-Thakkar commented on PR #21537: URL: https://github.com/apache/nuttx/pull/21537#issuecomment-6735547233
thanks all, I agree, the current solution does not work well because it overloads %hf and makes the kernel choose something optional, which is required by applications/tools only. proposed solution before coding: add dedicated %k b16_t support (conversion) in lib_vsprintf_internal and in scanner, if CONFIG_LIBC_PRINT_EXTENSION, to allow usage of width/precision and keep %hf as float. lib_bsprintf and lib_bscanf support %k for the member size only. no selection from SENSORS_USE_B16. in nuttx-apps, uORB formatting string for sensor_data_t is done using macro and the tools require LIBC_PRINT_EXTENSION if SENSORS_USE_B16 is defined. move the sensor_eng.voltage -> sensor_data_t part to a separate small PR because this is also necessary anyway. how about that? @xiaoxiang781216 @raiden00pl, any preference in the letter (k), and in reusing LIBC_PRINT_EXTENSION or having an independent option? -- 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]
