Spare a few more lines rather than having a condition with a nested ternary. Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx> --- src/util/virbuffer.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/src/util/virbuffer.c b/src/util/virbuffer.c index 04c8fd7291..a58481430a 100644 --- a/src/util/virbuffer.c +++ b/src/util/virbuffer.c @@ -64,11 +64,19 @@ virBufferAdjustIndent(virBufferPtr buf, int indent) { if (!buf || buf->error) return; - if (indent > 0 ? INT_MAX - indent < buf->indent - : buf->indent < -indent) { - virBufferSetError(buf, -1); - return; + + if (indent > 0) { + if (INT_MAX - indent < buf->indent) { + virBufferSetError(buf, -1); + return; + } + } else { + if (buf->indent < -indent) { + virBufferSetError(buf, -1); + return; + } } + buf->indent += indent; } -- 2.21.0 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list