lucasssvaz commented on PR #1774: URL: https://github.com/apache/nuttx-apps/pull/1774#issuecomment-1561641977
> @lucasssvaz 45KB is a little big than my expect, we may compress the result on PC and decompress on device with lzf, since lzf is very small(~5KB). And nuttx already integrate lzf and use in many place: https://github.com/apache/nuttx/tree/master/libs/libc/lzf https://github.com/apache/nuttx-apps/tree/master/system/lzf Just did some testing to find out the exact values: ``` Maximum possible: NuttX Image: text data bss dec hex filename 127708 5892 5264 138864 21e70 nuttx Output generated by nxdiag is 51.2 Kbytes -------------------------------------------------------- Minimum possible: NuttX Image: text data bss dec hex filename 85444 440 5264 91148 1640c nuttx Output generated by nxdiag is 192 bytes ``` As most of the size increase is caused by enabling `TESTING_NXDIAG_HOST_PACKAGES` and `TESTING_NXDIAG_HOST_MODULES` that, on the majority of cases, won't be enabled and the application is modular, wouldn't it be better to leave the compression as a future improvement ? -- 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: commits-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
