[Asterisk-code-review] BuildSystem: Remove two dead exceptions for compiler Clang. (asterisk[16])

Friendly Automation asteriskteam at digium.com
Thu Aug 19 08:40:57 CDT 2021


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/16288 )

Change subject: BuildSystem: Remove two dead exceptions for compiler Clang.
......................................................................

BuildSystem: Remove two dead exceptions for compiler Clang.

Commit 305ce3d added -Wno-parentheses-equality to Makefile.rules,
turning the previous two warning suppressions from commit e9520db
redundant. Let us remove the latter.

Change-Id: I0b471254b31e6e05902062761dded4b3e626c7ac
---
M res/Makefile
M utils/Makefile
2 files changed, 0 insertions(+), 6 deletions(-)

Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, but someone else must approve
  George Joseph: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/res/Makefile b/res/Makefile
index 7511622..8b1b759 100644
--- a/res/Makefile
+++ b/res/Makefile
@@ -33,9 +33,6 @@
 
 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
 
 ifneq ($(findstring REBUILD_PARSERS,$(MENUSELECT_CFLAGS)),)
 ael/ael_lex.c: ael/ael.flex
diff --git a/utils/Makefile b/utils/Makefile
index 3a1e218..3df7976 100644
--- a/utils/Makefile
+++ b/utils/Makefile
@@ -143,9 +143,6 @@
 	$(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)/,,$<) -> $@"

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/16288
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I0b471254b31e6e05902062761dded4b3e626c7ac
Gerrit-Change-Number: 16288
Gerrit-PatchSet: 2
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210819/5502586e/attachment.html>


More information about the asterisk-code-review mailing list