Thread (1 message) 1 message, 1 author, 2017-12-28

Re: [PATCH v2 8/9] rebase -i: learn to abbreviate command names

From: Junio C Hamano <hidden>
Date: 2017-12-28 18:55:19

Liam Beguin [off-list ref] writes:
Since this came up, would it be a good thing to add -Wignored-qualifiers
to the DEVELOPER flags?
Quite frankly, I am not sure if catching that particular warning
violation buys us much. As a return value from a function is never
an lvalue, what triggers the warning may certainly be an indication
of a sloppy coding, but otherwise I do not see it as diagnosing a
potential error.  "The programmer thought that the returned value
will only be assigned to a const variable and will never be
modified, but the language does not guarantee such a behaviour out
of the caller"---does such an incorrect expectation lead to an error
in the codepath that involves such a function?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help