[ 
https://issues.apache.org/jira/browse/PROTON-774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14245230#comment-14245230
 ] 

ASF subversion and git services commented on PROTON-774:
--------------------------------------------------------

Commit ed79dbedda498c5bb3a8ef1e764395fafbd3a63b in qpid-proton's branch 
refs/heads/master from [~fadams]
[ https://git-wip-us.apache.org/repos/asf?p=qpid-proton.git;h=ed79dbe ]

PROTON-774: Fix warnings in log.c and url.c and re-enable -Werror in JavaScript 
CmakeLists.txt


> Fix warnings in log.c and url.c and re-enable -Werror
> -----------------------------------------------------
>
>                 Key: PROTON-774
>                 URL: https://issues.apache.org/jira/browse/PROTON-774
>             Project: Qpid Proton
>          Issue Type: Bug
>          Components: proton-c
>            Reporter: Fraser Adams
>            Priority: Trivial
>
> The files log.c and url.c generate warnings when compiled with 
> CLang/emscripten this prevents -Werror being used.
> The warnings were:
> [ 56%] Building C object 
> proton-c/bindings/javascript/CMakeFiles/qpid-proton-bitcode.dir/__/__/src/log.c.o
> /home/fadams/qpid/git/qpid-proton/proton-c/src/log.c:38:13: warning: 
> explicitly
>       assigning a variable of type 'bool' to itself [-Wself-assign]
>     enabled = enabled;
> and
> [ 58%] Building C object 
> proton-c/bindings/javascript/CMakeFiles/qpid-proton-bitcode.dir/__/__/src/url.c.o
> /home/fadams/qpid/git/qpid-proton/proton-c/src/url.c:133:19: warning: unused
>       function 'len' [-Wunused-function]
> static inline int len(const char *str) { return str ? strlen(str) : 0; }
> The trivial fixes were to remove the spurious function in url.c and rename 
> the formal parameter to pn_log_enable in log.c



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to