[asterisk-commits] mjordan: branch 11 r433687 - in /branches/11: apps/ main/ res/ utils/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Sat Mar 28 07:39:12 CDT 2015
Author: mjordan
Date: Sat Mar 28 07:39:08 2015
New Revision: 433687
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=433687
Log:
clang compiler warnings: Fix -Wparantheses-equality warnings
Clang will treat ((a == b)) as a warning, as it reasonably expects that the
developer may have intended to write (a == b) or ((a = b)). This patch cleans
up all instances where equality, not assignment, was intended between two
parantheses.
Review: https://reviewboard.asterisk.org/r/4531/
ASTERISK-24917
Repoted by: dkdegroot
patches:
rb4531.patch submitted by dkdegroot (License 6600)
Modified:
branches/11/apps/app_dictate.c
branches/11/apps/app_voicemail.c
branches/11/main/channel.c
branches/11/res/Makefile
branches/11/utils/Makefile
Modified: branches/11/apps/app_dictate.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/apps/app_dictate.c?view=diff&rev=433687&r1=433686&r2=433687
==============================================================================
--- branches/11/apps/app_dictate.c (original)
+++ branches/11/apps/app_dictate.c Sat Mar 28 07:39:08 2015
@@ -174,7 +174,7 @@
ast_queue_frame(chan, &fr);
digit = 0;
}
- if ((f->frametype == AST_FRAME_DTMF)) {
+ if (f->frametype == AST_FRAME_DTMF) {
int got = 1;
switch(mode) {
case DMODE_PLAY:
Modified: branches/11/apps/app_voicemail.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/apps/app_voicemail.c?view=diff&rev=433687&r1=433686&r2=433687
==============================================================================
--- branches/11/apps/app_voicemail.c (original)
+++ branches/11/apps/app_voicemail.c Sat Mar 28 07:39:08 2015
@@ -8968,7 +8968,7 @@
if (!res)
res = ast_say_number(chan, vms->newmessages, AST_DIGIT_ANY, ast_channel_language(chan), NULL);
if (!res) {
- if ((vms->newmessages == 1)) {
+ if (vms->newmessages == 1) {
res = ast_play_and_wait(chan, "vm-INBOX");
if (!res)
res = ast_play_and_wait(chan, "vm-message");
@@ -8982,7 +8982,7 @@
res = ast_play_and_wait(chan, "vm-youhave");
if (!res)
res = ast_say_number(chan, vms->oldmessages, AST_DIGIT_ANY, ast_channel_language(chan), NULL);
- if ((vms->oldmessages == 1)){
+ if (vms->oldmessages == 1){
res = ast_play_and_wait(chan, "vm-Old");
if (!res)
res = ast_play_and_wait(chan, "vm-message");
@@ -9174,7 +9174,7 @@
if ((vms->oldmessages || vms->newmessages) && !res) {
res = ast_play_and_wait(chan, "vm-and");
} else if (!res) {
- if ((vms->urgentmessages == 1))
+ if (vms->urgentmessages == 1)
res = ast_play_and_wait(chan, "vm-message");
else
res = ast_play_and_wait(chan, "vm-messages");
@@ -9187,7 +9187,7 @@
if (vms->oldmessages && !res)
res = ast_play_and_wait(chan, "vm-and");
else if (!res) {
- if ((vms->newmessages == 1))
+ if (vms->newmessages == 1)
res = ast_play_and_wait(chan, "vm-message");
else
res = ast_play_and_wait(chan, "vm-messages");
@@ -9339,7 +9339,7 @@
}
if (vms->newmessages) {
- if ((vms->newmessages == 1)) {
+ if (vms->newmessages == 1) {
res = ast_play_and_wait(chan, "digits/ett");
res = res ? res : ast_play_and_wait(chan, "vm-nytt");
res = res ? res : ast_play_and_wait(chan, "vm-message");
@@ -9383,7 +9383,7 @@
}
if (vms->newmessages) {
- if ((vms->newmessages == 1)) {
+ if (vms->newmessages == 1) {
res = ast_play_and_wait(chan, "digits/1");
res = res ? res : ast_play_and_wait(chan, "vm-ny");
res = res ? res : ast_play_and_wait(chan, "vm-message");
@@ -9418,7 +9418,7 @@
res = ast_play_and_wait(chan, "vm-youhave");
if (!res) {
if (vms->newmessages) {
- if ((vms->newmessages == 1))
+ if (vms->newmessages == 1)
res = ast_play_and_wait(chan, "digits/1F");
else
res = say_and_wait(chan, vms->newmessages, ast_channel_language(chan));
@@ -9427,7 +9427,7 @@
if (vms->oldmessages && !res)
res = ast_play_and_wait(chan, "vm-and");
else if (!res) {
- if ((vms->newmessages == 1))
+ if (vms->newmessages == 1)
res = ast_play_and_wait(chan, "vm-message");
else
res = ast_play_and_wait(chan, "vm-messages");
@@ -9474,7 +9474,7 @@
if (!res) {
if (vms->newmessages) {
if (!res) {
- if ((vms->newmessages == 1)) {
+ if (vms->newmessages == 1) {
res = ast_play_and_wait(chan, "digits/1M");
if (!res)
res = ast_play_and_wait(chan, "vm-message");
@@ -9525,7 +9525,7 @@
if (vms->newmessages) {
if (!res)
res = ast_say_number(chan, vms->newmessages, AST_DIGIT_ANY, ast_channel_language(chan), "f");
- if ((vms->newmessages == 1)) {
+ if (vms->newmessages == 1) {
if (!res)
res = ast_play_and_wait(chan, "vm-message");
if (!res)
@@ -9571,7 +9571,7 @@
if (vms->oldmessages && !res)
res = ast_play_and_wait(chan, "vm-and");
else if (!res) {
- if ((vms->newmessages == 1))
+ if (vms->newmessages == 1)
res = ast_play_and_wait(chan, "vm-message");
else
res = ast_play_and_wait(chan, "vm-messages");
@@ -9618,7 +9618,7 @@
if (vms->oldmessages && !res)
res = ast_play_and_wait(chan, "vm-and");
else if (!res) {
- if ((vms->newmessages == 1))
+ if (vms->newmessages == 1)
res = ast_play_and_wait(chan, "vm-message");
else
res = ast_play_and_wait(chan, "vm-messages");
@@ -9661,7 +9661,7 @@
if (vms->newmessages) {
res = ast_say_number(chan, vms->newmessages, AST_DIGIT_ANY, ast_channel_language(chan), "f");
if (!res) {
- if ((vms->newmessages == 1)) {
+ if (vms->newmessages == 1) {
res = ast_play_and_wait(chan, "vm-message");
if (!res)
res = ast_play_and_wait(chan, "vm-INBOXs");
@@ -9727,7 +9727,7 @@
res = say_and_wait(chan, vms->newmessages, ast_channel_language(chan));
}
if (!res) {
- if ((vms->newmessages == 1))
+ if (vms->newmessages == 1)
res = ast_play_and_wait(chan, "vm-novou");
if ((vms->newmessages) > 1 && (vms->newmessages < 5))
res = ast_play_and_wait(chan, "vm-nove");
@@ -9737,7 +9737,7 @@
if (vms->oldmessages && !res)
res = ast_play_and_wait(chan, "vm-and");
else if (!res) {
- if ((vms->newmessages == 1))
+ if (vms->newmessages == 1)
res = ast_play_and_wait(chan, "vm-zpravu");
if ((vms->newmessages) > 1 && (vms->newmessages < 5))
res = ast_play_and_wait(chan, "vm-zpravy");
@@ -9748,7 +9748,7 @@
if (!res && vms->oldmessages) {
res = say_and_wait(chan, vms->oldmessages, ast_channel_language(chan));
if (!res) {
- if ((vms->oldmessages == 1))
+ if (vms->oldmessages == 1)
res = ast_play_and_wait(chan, "vm-starou");
if ((vms->oldmessages) > 1 && (vms->oldmessages < 5))
res = ast_play_and_wait(chan, "vm-stare");
@@ -9756,7 +9756,7 @@
res = ast_play_and_wait(chan, "vm-starych");
}
if (!res) {
- if ((vms->oldmessages == 1))
+ if (vms->oldmessages == 1)
res = ast_play_and_wait(chan, "vm-zpravu");
if ((vms->oldmessages) > 1 && (vms->oldmessages < 5))
res = ast_play_and_wait(chan, "vm-zpravy");
Modified: branches/11/main/channel.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/main/channel.c?view=diff&rev=433687&r1=433686&r2=433687
==============================================================================
--- branches/11/main/channel.c (original)
+++ branches/11/main/channel.c Sat Mar 28 07:39:08 2015
@@ -4302,7 +4302,7 @@
ast_channel_name(chan), ast_getformatname(&f->subclass.format), ast_getformatname_multiple(to, sizeof(to), ast_channel_nativeformats(chan)));
ast_frfree(f);
f = &ast_null_frame;
- } else if ((f->frametype == AST_FRAME_VOICE)) {
+ } else if (f->frametype == AST_FRAME_VOICE) {
/* Send frame to audiohooks if present */
if (ast_channel_audiohooks(chan)) {
struct ast_frame *old_frame = f;
Modified: branches/11/res/Makefile
URL: http://svnview.digium.com/svn/asterisk/branches/11/res/Makefile?view=diff&rev=433687&r1=433686&r2=433687
==============================================================================
--- branches/11/res/Makefile (original)
+++ branches/11/res/Makefile Sat Mar 28 07:39:08 2015
@@ -33,6 +33,9 @@
ael/ael.tab.o: ael/ael.tab.c ael/ael.tab.h ../include/asterisk/ael_structs.h
ael/ael.tab.o: _ASTCFLAGS+=-I. -Iael -DYYENABLE_NLS=0
+ ifneq ($(AST_CLANG_BLOCKS),)
+ _ASTCFLAGS+=-Wno-parentheses-equality
+ endif
$(if $(filter res_ais,$(EMBEDDED_MODS)),modules.link,res_ais.so): ais/clm.o ais/evt.o
ais/clm.o ais/evt.o: _ASTCFLAGS+=$(call MOD_ASTCFLAGS,res_ais)
Modified: branches/11/utils/Makefile
URL: http://svnview.digium.com/svn/asterisk/branches/11/utils/Makefile?view=diff&rev=433687&r1=433686&r2=433687
==============================================================================
--- branches/11/utils/Makefile (original)
+++ branches/11/utils/Makefile Sat Mar 28 07:39:08 2015
@@ -138,6 +138,9 @@
$(ECHO_PREFIX) echo " [CP] $(subst $(ASTTOPDIR)/,,$<) -> $@"
$(CMD_PREFIX) cp "$<" "$@"
aelbison.o: _ASTCFLAGS+=-I$(ASTTOPDIR)/res/ael -DYYENABLE_NLS=0
+ ifneq ($(AST_CLANG_BLOCKS),)
+ _ASTCFLAGS+=-Wno-parentheses-equality
+ endif
pbx_ael.c: $(ASTTOPDIR)/pbx/pbx_ael.c
$(ECHO_PREFIX) echo " [CP] $(subst $(ASTTOPDIR)/,,$<) -> $@"
More information about the asterisk-commits
mailing list