You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/11/17 15:44:05 UTC

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 opened a new pull request #894: testing/sensortest: Fix printf format warning

xiaoxiang781216 opened a new pull request #894:
URL: https://github.com/apache/incubator-nuttx-apps/pull/894


   ## Summary
   ````
   sensortest.c: In function 'print_gps':
   Error: sensortest.c:169:29: error: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type 'uint64_t' {aka 'long unsigned int'} [-Werror=format=]
     169 |   printf("%s: timestamp: %llu time_utc: %llu latitude: %f longitude: %f "
         |                          ~~~^
         |                             |
         |                             long long unsigned int
         |                          %lu
   ......
     172 |          " %u\n", name, event->timestamp, event->time_utc, event->latitude,
         |                         ~~~~~~~~~~~~~~~~
         |                              |
         |                              uint64_t {aka long unsigned int}
   Error: sensortest.c:169:44: error: format '%llu' expects argument of type 'long long unsigned int', but argument 4 has type 'uint64_t' {aka 'long unsigned int'} [-Werror=format=]
     169 |   printf("%s: timestamp: %llu time_utc: %llu latitude: %f longitude: %f "
         |                                         ~~~^
         |                                            |
         |                                            long long unsigned int
         |                                         %lu
   ......
     172 |          " %u\n", name, event->timestamp, event->time_utc, event->latitude,
         |                                           ~~~~~~~~~~~~~~~
         |                                                |
         |                                                uint64_t {aka long unsigned int}
   sensortest.c: In function 'print_gps_satellite':
   Error: sensortest.c:183:29: error: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type 'uint64_t' {aka 'long unsigned int'} [-Werror=format=]
     183 |   printf("%s: timestamp: %llu count: %u satellites: %u", name,
         |                          ~~~^
         |                             |
         |                             long long unsigned int
         |                          %lu
     184 |          event->timestamp, event->count, event->satellites);
         |          ~~~~~~~~~~~~~~~~
         |               |
         |               uint64_t {aka long unsigned int}
   ```
   
   ## Impact
   Fix the warning by PR https://github.com/apache/incubator-nuttx-apps/pull/893
   
   ## Testing
   Pass CI
   


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx-apps] gustavonihei merged pull request #894: testing/sensortest: Fix printf format warning

Posted by GitBox <gi...@apache.org>.
gustavonihei merged pull request #894:
URL: https://github.com/apache/incubator-nuttx-apps/pull/894


   


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #894: testing/sensortest: Fix printf format warning

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #894:
URL: https://github.com/apache/incubator-nuttx-apps/pull/894#issuecomment-971712099






-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #894: testing/sensortest: Fix printf format warning

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #894:
URL: https://github.com/apache/incubator-nuttx-apps/pull/894#issuecomment-972734803


   Does anyone merge this PR? mainline CI is broken now.


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx-apps] xiaoxiang781216 removed a comment on pull request #894: testing/sensortest: Fix printf format warning

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 removed a comment on pull request #894:
URL: https://github.com/apache/incubator-nuttx-apps/pull/894#issuecomment-971712152


   Note both PR https://github.com/apache/incubator-nuttx-apps/pull/894 and https://github.com/apache/incubator-nuttx/pull/4846 need merge to fix the CI break.


-- 
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-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org