From 17c244e644c6654d99bd1391bd999c4ab5f5cedc Mon Sep 17 00:00:00 2001
From: renu555 <renu.tyagi@samsung.com>
Date: Wed, 2 Jul 2014 09:19:00 +0530
Subject: [PATCH] Fixing unreachable condition.

  if (!isMultiLine) at line 563 suggests that isMultiline is 0 when if takes true branch. So the condition && at line 571 will always be false.
Also at line 568 !isMultiline in loop conditional check suggests that it depends only on one condition i.e. index <size because !isMultiline is always true.
Hence , it seems logical mistake at line 571 of using && instead of ||
---
 src/lib_json/json_writer.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib_json/json_writer.cpp b/src/lib_json/json_writer.cpp
index ffdcbd7..da68f1b 100644
--- a/src/lib_json/json_writer.cpp
+++ b/src/lib_json/json_writer.cpp
@@ -344,7 +344,7 @@ bool StyledWriter::isMultineArray(const Value &value) {
     for (int index = 0; index < size && !isMultiLine; ++index) {
       writeValue(value[index]);
       lineLength += int(childValues_[index].length());
-      isMultiLine = isMultiLine && hasCommentForValue(value[index]);
+      isMultiLine = isMultiLine || hasCommentForValue(value[index]);
     }
     addChildValues_ = false;
     isMultiLine = isMultiLine || lineLength >= rightMargin_;
@@ -568,7 +568,7 @@ bool StyledStreamWriter::isMultineArray(const Value &value) {
     for (int index = 0; index < size && !isMultiLine; ++index) {
       writeValue(value[index]);
       lineLength += int(childValues_[index].length());
-      isMultiLine = isMultiLine && hasCommentForValue(value[index]);
+      isMultiLine = isMultiLine || hasCommentForValue(value[index]);
     }
     addChildValues_ = false;
     isMultiLine = isMultiLine || lineLength >= rightMargin_;