Transforms patch not respecting if indentation
Started by Michael Paquieralmost 11 years ago2 messageshackers
Hi all,
Coverity complained about a small indentation issue in ruleutils.c:
+ appendStringInfoString(buf, "\n TRANSFORM ");
+ for (i = 0; i < ntypes; i++)
+ {
+ if (i != 0)
+ appendStringInfoString(buf, ", ");
+ appendStringInfo(buf, "FOR TYPE %s",
format_type_be(trftypes[i]));
+ }
Regards,
--
MichaelAttachments:
20150504_ruleutils_ident.patchtext/x-diff; charset=US-ASCII; name=20150504_ruleutils_ident.patchDownload+1-1
Re: Transforms patch not respecting if indentation
On Mon, May 4, 2015 at 6:45 AM, Michael Paquier <michael.paquier@gmail.com>
wrote:
Hi all,
Coverity complained about a small indentation issue in ruleutils.c: + appendStringInfoString(buf, "\n TRANSFORM "); + for (i = 0; i < ntypes; i++) + { + if (i != 0) + appendStringInfoString(buf, ", "); + appendStringInfo(buf, "FOR TYPE %s", format_type_be(trftypes[i])); + }
Applied, thanks.
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/