You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by "bneradt (via GitHub)" <gi...@apache.org> on 2023/04/06 19:17:30 UTC

[GitHub] [trafficserver] bneradt opened a new pull request, #9586: Fix clang 16 warnings

bneradt opened a new pull request, #9586:
URL: https://github.com/apache/trafficserver/pull/9586

   fedora:38 is scheduled to use clang 16. This fixes the non-libswoc warnings that clang raised by clang 16. https://github.com/SolidWallOfCode/libswoc/pull/84 addresses the libswoc warning which we'll pull in via a separate PR.


-- 
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: github-unsubscribe@trafficserver.apache.org

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


[GitHub] [trafficserver] bneradt merged pull request #9586: Fix clang 16 warnings

Posted by "bneradt (via GitHub)" <gi...@apache.org>.
bneradt merged PR #9586:
URL: https://github.com/apache/trafficserver/pull/9586


-- 
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: github-unsubscribe@trafficserver.apache.org

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


[GitHub] [trafficserver] bneradt commented on pull request #9586: Fix clang 16 warnings

Posted by "bneradt (via GitHub)" <gi...@apache.org>.
bneradt commented on PR #9586:
URL: https://github.com/apache/trafficserver/pull/9586#issuecomment-1522195090

   > @bneradt This has merge conflicts, so to be safe, can you please make a PR for 9.2.x which fixes only what's needed there?
   
   Sure thing:
   https://github.com/apache/trafficserver/pull/9586


-- 
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: github-unsubscribe@trafficserver.apache.org

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


[GitHub] [trafficserver] zwoop commented on pull request #9586: Fix clang 16 warnings

Posted by "zwoop (via GitHub)" <gi...@apache.org>.
zwoop commented on PR #9586:
URL: https://github.com/apache/trafficserver/pull/9586#issuecomment-1520947018

   @bneradt This has merge conflicts, so to be safe, can you please make a PR for 9.2.x which fixes only what's needed there?


-- 
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: github-unsubscribe@trafficserver.apache.org

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


[GitHub] [trafficserver] bneradt commented on pull request #9586: Fix clang 16 warnings

Posted by "bneradt (via GitHub)" <gi...@apache.org>.
bneradt commented on PR #9586:
URL: https://github.com/apache/trafficserver/pull/9586#issuecomment-1499559624

   I added 9.2.x to this because we'll want that branch maintained for the newer compilers. If it doesn't cherry-pick cleanly, let me know and I can create a new PR for 9.2.x.


-- 
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: github-unsubscribe@trafficserver.apache.org

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