On Mon, 20 Nov 2023, Andrew Cooper wrote: > The codebase is now -Wwrite-strings clean. Activate the option to cause > string literals to have a const type, and prevent any violations of MISRA Rule > 7.4 being reintroduced. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
Excellent! Acked-by: Stefano Stabellini <sstabell...@kernel.org> > --- > CC: Jan Beulich <jbeul...@suse.com> > CC: Roger Pau Monné <roger....@citrix.com> > CC: Wei Liu <w...@xen.org> > CC: Stefano Stabellini <sstabell...@kernel.org> > CC: Julien Grall <jul...@xen.org> > CC: Volodymyr Babchuk <volodymyr_babc...@epam.com> > CC: Bertrand Marquis <bertrand.marq...@arm.com> > CC: Michal Orzel <michal.or...@amd.com> > CC: Roberto Bagnara <roberto.bagn...@bugseng.com> > CC: Nicola Vetrini <nicola.vetr...@bugseng.com> > --- > xen/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/xen/Makefile b/xen/Makefile > index 7b869f4b3037..ca571103c868 100644 > --- a/xen/Makefile > +++ b/xen/Makefile > @@ -392,7 +392,7 @@ endif > CFLAGS-$(CONFIG_CC_SPLIT_SECTIONS) += -ffunction-sections -fdata-sections > > CFLAGS += -nostdinc -fno-builtin -fno-common > -CFLAGS += -Werror -Wredundant-decls -Wno-pointer-arith > +CFLAGS += -Werror -Wredundant-decls -Wwrite-strings -Wno-pointer-arith > $(call cc-option-add,CFLAGS,CC,-Wvla) > CFLAGS += -pipe -D__XEN__ -include $(srctree)/include/xen/config.h > CFLAGS-$(CONFIG_DEBUG_INFO) += -g > -- > 2.30.2 > >
