Fixes another compile error when warnings are set

Fixes another compile error when warnings are set

Or it may be just for arm, not sure which setting is triggering the
compile error.

BUG=none
TEST=none
R=viettrungluu@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282196

[darin (slow to review), 2014-06-28 05:29:00]

https://codereview.chromium.org/355263002/diff/1/mojo/examples/wget/wget.cc
File mojo/examples/wget/wget.cc (right):

https://codereview.chromium.org/355263002/diff/1/mojo/examples/wget/wget.cc#n...
mojo/examples/wget/wget.cc:96: assert(num_bytes == fwrite(buf, num_bytes, 1,
stdout));
doesn't assert(x) compile to nothing in release builds?

[sky, 2014-06-28 15:38:51]

Ugh, you're right. Will do another fix when I get back.

  -Scott


On Fri, Jun 27, 2014 at 10:29 PM, <darin@chromium.org> wrote:

>
> https://codereview.chromium.org/355263002/diff/1/mojo/
> examples/wget/wget.cc
> File mojo/examples/wget/wget.cc (right):
>
> https://codereview.chromium.org/355263002/diff/1/mojo/
> examples/wget/wget.cc#newcode96
> mojo/examples/wget/wget.cc:96: assert(num_bytes == fwrite(buf,
> num_bytes, 1, stdout));
> doesn't assert(x) compile to nothing in release builds?
>
> https://codereview.chromium.org/355263002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[sky, 2014-07-09 20:56:25]

How about a printf as this isn't really expected?

[darin (slow to review), 2014-07-09 21:02:21]

https://codereview.chromium.org/355263002/diff/20001/mojo/examples/wget/wget.cc
File mojo/examples/wget/wget.cc (right):

https://codereview.chromium.org/355263002/diff/20001/mojo/examples/wget/wget....
mojo/examples/wget/wget.cc:96: if (fwrite(buf, num_bytes, 1, stdout) !=
num_bytes) {
LGTM

[sky, 2014-07-09 21:12:39]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2014-07-09 21:18:13]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/sky@chromium.org/355263002/20001

[commit-bot: I haz the power, 2014-07-10 00:45:35]

Change committed as 282196