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 2020/07/30 08:27:27 UTC
[GitHub] [incubator-nuttx] yamt opened a new pull request #1485: Fix random issues reported by clang-check
yamt opened a new pull request #1485:
URL: https://github.com/apache/incubator-nuttx/pull/1485
## Summary
fix random issues reported by clang-check
## Impact
the one in icmp_recvfrom might be serious
(i haven't investigated if it can actually be triggered)
## Testing
compile tested only
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1485: Fix random issues reported by clang-check
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #1485:
URL: https://github.com/apache/incubator-nuttx/pull/1485#issuecomment-666222501
LGTM, @yamt how about integrate your local setting into CI process?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] jerpelea merged pull request #1485: Fix random issues reported by clang-check
Posted by GitBox <gi...@apache.org>.
jerpelea merged pull request #1485:
URL: https://github.com/apache/incubator-nuttx/pull/1485
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1485: Fix random issues reported by clang-check
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #1485:
URL: https://github.com/apache/incubator-nuttx/pull/1485#issuecomment-666400376
Ok, it's good to document it first, then we can integrate into CI later.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] yamt commented on pull request #1485: Fix random issues reported by clang-check
Posted by GitBox <gi...@apache.org>.
yamt commented on pull request #1485:
URL: https://github.com/apache/incubator-nuttx/pull/1485#issuecomment-666232417
@xiaoxiang781216 i will consider to document the process at least.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org