Re: [Bug-apl] Misleading indentation

[Leslie S Satenstein]

GCC 7 puts out a warning if the line following an if () is at the same or greater indentation than the consequence of the if()

It is done when the if () does not have arguments within { }

I too solved that issue.

 

Regards

 Leslie

Leslie Satenstein

Montréal Québec, Canada

---

From: Juergen Sauermann <address@hidden>
To: Kacper Gutowski <address@hidden>; address@hidden
Sent: Saturday, May 6, 2017 11:12 AM
Subject: Re: [Bug-apl] Misleading indentation

Hi Kacper,

thanks, maybe fixed in SVN 942. No idea what the compiler means, though.

Maybe I should respond with -Werror=misleading-warning-since-g++6.x

/// Jürgen

 

On 05/06/2017 12:24 PM, Kacper Gutowski wrote:

Attempting to compile r941 with g++ 6.3.0 fails due to it not liking
your coding style and -Werror:

Command.cc: In static member function ‘static bool Command::check_params(std::ostream&, const char*, int, const char*)’:
Command.cc:163:20: error: this ‘if’ clause does not guard... [-Werror=misleading-indentation]
          case '.': if (a[1] == '.' && a[2] == '.')
                    ^~
Command.cc:164:56: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
                       many = true;                     continue;  // ...
                                                        ^~~~~~~~

-k