Comment on WARN_UNUSED_RESULT about how to defeat it.

Comment on WARN_UNUSED_RESULT about how to defeat it.

Comment change only.

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=86022

[Scott Hess - ex-Googler, 2011-05-19 21:11:18]

I was impressed by how long it took me to figure out when I needed it.  When I
saw other-Scott ask on the IRC channel, it seemed stupid that it should take
more than a half second to figure it out.

@mark because I need an OWNER.

[sky, 2011-05-19 22:23:35]

LGTM

[Mark Mentovai, 2011-05-19 23:17:13]

LG otherwise.

Or LGTM to satisfy stupid PRESUBMIT.

http://codereview.chromium.org/7050022/diff/1/base/compiler_specific.h
File base/compiler_specific.h (right):

http://codereview.chromium.org/7050022/diff/1/base/compiler_specific.h#newcod...
base/compiler_specific.h:114: // To explicitly ignore a result, see
|ignore_result()| in basic_types.h.
Say <base/basictypes.h>, with no underscore.

[Scott Hess - ex-Googler, 2011-05-19 23:19:42]

Commenting are hard.

On 2011/05/19 23:17:13, Mark Mentovai wrote:
> LG otherwise.
> 
> Or LGTM to satisfy stupid PRESUBMIT.
> 
> http://codereview.chromium.org/7050022/diff/1/base/compiler_specific.h
> File base/compiler_specific.h (right):
> 
>
http://codereview.chromium.org/7050022/diff/1/base/compiler_specific.h#newcod...
> base/compiler_specific.h:114: // To explicitly ignore a result, see
> |ignore_result()| in basic_types.h.
> Say <base/basictypes.h>, with no underscore.

[Mark Mentovai, 2011-05-19 23:32:06]

That’s better! LGTM.

[Mark Mentovai, 2011-05-19 23:33:52]

It kind of sucks that Chrome allays
separates_words_with_underscores_in_filenames, but this file adheres to a
different standard.

[commit-bot: I haz the power, 2011-05-20 00:48:37]

Change committed as 86022