Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 6769323a authored by Yi Kong's avatar Yi Kong Committed by Gerrit Code Review
Browse files

Merge "Only set -Wall once."

parents 82126214 ed4a27b2

cc/cc_test.go

+1 −1
Original line number Diff line number Diff line
@@ -4041,7 +4041,7 @@ func TestIncludeDirectoryOrdering(t *testing.T) { 
	conly := []string{"-fPIC", "${config.CommonGlobalConlyflags}"}

	cppOnly := []string{"-fPIC", "${config.CommonGlobalCppflags}", "${config.DeviceGlobalCppflags}", "${config.ArmCppflags}"}



	cflags := []string{"-Wall", "-Werror", "-std=candcpp"}

	cflags := []string{"-Werror", "-std=candcpp"}

	cstd := []string{"-std=gnu11", "-std=conly"}

	cppstd := []string{"-std=gnu++17", "-std=cpp", "-fno-rtti"}

cc/compiler.go

+1 −2
Original line number Diff line number Diff line
@@ -590,9 +590,8 @@ func (compiler *baseCompiler) compilerFlags(ctx ModuleContext, flags Flags, deps 
		} else if !inList("-Werror", flags.Local.CFlags) && !inList("-Werror", flags.Local.CppFlags) {

			if warningsAreAllowed(ctx.ModuleDir()) {

				addToModuleList(ctx, modulesAddedWallKey, module)

				flags.Local.CFlags = append([]string{"-Wall"}, flags.Local.CFlags...)

			} else {

				flags.Local.CFlags = append([]string{"-Wall", "-Werror"}, flags.Local.CFlags...)

				flags.Local.CFlags = append([]string{"-Werror"}, flags.Local.CFlags...)

			}

		}

	}