You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2020/09/17 00:46:25 UTC
[GitHub] [trafficserver] maskit opened a new pull request #7199: Rename ambiguous log variable
maskit opened a new pull request #7199:
URL: https://github.com/apache/trafficserver/pull/7199
I got a compile error after updating Xcode.
```
Making all in cpp-api
CXX logger_example/LoggerExample.lo
logger_example/LoggerExample.cc:64:15: error: reference to 'log' is ambiguous
LOG_DEBUG(log,
^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/math.h:1003:1: note: candidate found by name lookup is 'log'
log(_A1 __lcpp_x) _NOEXCEPT {return ::log((double)__lcpp_x);}
^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/math.h:997:46: note: candidate found by name lookup is 'log'
inline _LIBCPP_INLINE_VISIBILITY long double log(long double __lcpp_x) _NOEXCEPT {return ::logl(__lcpp_x);}
^
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/math.h:996:46: note: candidate found by name lookup is 'log'
inline _LIBCPP_INLINE_VISIBILITY float log(float __lcpp_x) _NOEXCEPT {return ::logf(__lcpp_x);}
^
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include/math.h:373:15: note: candidate found by name lookup is 'log'
extern double log(double);
^
logger_example/LoggerExample.cc:36:8: note: candidate found by name lookup is '(anonymous namespace)::log'
Logger log;
^
logger_example/LoggerExample.cc:64:15: error: unknown type name 'log'; did you mean 'long'?
LOG_DEBUG(log,
^~~
long
```
```
$ clang --version
Apple clang version 12.0.0 (clang-1200.0.32.2)
```
----------------------------------------------------------------
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] [trafficserver] maskit commented on pull request #7199: Rename ambiguous log variable
Posted by GitBox <gi...@apache.org>.
maskit commented on pull request #7199:
URL: https://github.com/apache/trafficserver/pull/7199#issuecomment-693741930
[approve ci Clang-Analyzer]
[approve ci RAT]
[approve ci autest]
[approve ci clang-format]
----------------------------------------------------------------
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] [trafficserver] zwoop commented on pull request #7199: Rename ambiguous log variable
Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #7199:
URL: https://github.com/apache/trafficserver/pull/7199#issuecomment-700245886
Cherry-picked to v9.0.x branch.
----------------------------------------------------------------
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] [trafficserver] maskit merged pull request #7199: Rename ambiguous log variable
Posted by GitBox <gi...@apache.org>.
maskit merged pull request #7199:
URL: https://github.com/apache/trafficserver/pull/7199
----------------------------------------------------------------
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] [trafficserver] randall commented on pull request #7199: Rename ambiguous log variable
Posted by GitBox <gi...@apache.org>.
randall commented on pull request #7199:
URL: https://github.com/apache/trafficserver/pull/7199#issuecomment-694379347
[approve ci Clang-Analyzer]
[approve ci RAT]
[approve ci autest]
[approve ci clang-format]
----------------------------------------------------------------
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