From c8d6cd853ffdc31dfcfdd81176ed9b86e2c68543 Mon Sep 17 00:00:00 2001 From: Laine Stump Date: Wed, 12 Mar 2014 18:05:31 -0600 Subject: [PATCH] build: detect/prohibit hardcoded XML indent in syntax-check This rule wouldn't be able to find any case of a hardcoded indent that was in the middle of a string, but then virBuffer doesn't add indentation in the middle of a string either. --- cfg.mk | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/cfg.mk b/cfg.mk index 9ad8cf7529..f52295982c 100644 --- a/cfg.mk +++ b/cfg.mk @@ -1,5 +1,5 @@ # Customize Makefile.maint. -*- makefile -*- -# Copyright (C) 2008-2013 Red Hat, Inc. +# Copyright (C) 2008-2014 Red Hat, Inc. # Copyright (C) 2003-2008 Free Software Foundation, Inc. # This program is free software: you can redistribute it and/or modify @@ -517,6 +517,11 @@ sc_prohibit_virBufferAsprintf_with_string_literal: halt='use virBufferAddLit, not virBufferAsprintf, with a string literal' \ $(_sc_search_regexp) +sc_forbid_manual_xml_indent: + @prohibit='virBuffer.*" +<' \ + halt='use virBufferAdjustIndent instead of spaces when indenting xml' \ + $(_sc_search_regexp) + # Not only do they fail to deal well with ipv6, but the gethostby* # functions are also not thread-safe. sc_prohibit_gethostby: