Re: [PATCH] Fix -Wmisleading-indentation warning in abg-leaf-reporter.cc.

Re: [PATCH] Fix -Wmisleading-indentation warning in abg-leaf-reporter.cc.

[Dodji Seketeli]

Hello,

Mark Wielaard <mark@klomp.org> a écrit:

[...]

>     * src/abg-leaf-reporter.cc (leaf_reporter::report): Fix misleading
>     indentation.

I have just pushed this to master.

Thanks!

-- 
		Dodji

[PATCH] Fix -Wmisleading-indentation warning in abg-leaf-reporter.cc.

[Mark Wielaard]

abg-leaf-reporter.cc: In member function ‘virtual void abigail::comparison::leaf_reporter::report(const abigail::comparison::qualified_type_diff&, std::ostream&, const string&) const’:
abg-leaf-reporter.cc:158:3: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
   if (!diff_to_be_reported(&d))
   ^~
abg-leaf-reporter.cc:161:5: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
     report_local_qualified_type_changes(d, out, indent);
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

    * src/abg-leaf-reporter.cc (leaf_reporter::report): Fix misleading
    indentation.

Signed-off-by: Mark Wielaard <mark@klomp.org>
---
 src/abg-leaf-reporter.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/abg-leaf-reporter.cc b/src/abg-leaf-reporter.cc
index f68f17c..6fe4b19 100644
--- a/src/abg-leaf-reporter.cc
+++ b/src/abg-leaf-reporter.cc
@@ -158,7 +158,7 @@ leaf_reporter::report(const qualified_type_diff& d, ostream& out,
   if (!diff_to_be_reported(&d))
     return;
 
-    report_local_qualified_type_changes(d, out, indent);
+  report_local_qualified_type_changes(d, out, indent);
 }
 
 /// Report the changes carried by a @ref pointer_diff node.
-- 
1.8.3.1