Ignore result from fwrite

Print error message if dumping wasm file fails

BUG=

Review-Url: https://codereview.chromium.org/2806313002
Cr-Commit-Position: refs/heads/master@{#45007}
Committed: https://chromium.googlesource.com/v8/v8/+/d15687df02ea9a47bdc4755a0415419092f97f37

[landell, 2017-04-10 09:39:29]

Description was changed from

==========
Ignore result from fwrite

Shut up compiler warning

BUG=
==========

to

==========
Ignore result from fwrite

Shut up compiler warning

BUG=
==========

[landell, 2017-04-10 09:39:29]

landell@opera.com changed reviewers:
+ titzer@chromium.org

[landell, 2017-04-10 09:40:04]

@reviewer: PTAL. Small remove warning fix.

[landell, 2017-04-10 09:40:05]

@reviewer: PTAL. Small remove warning fix.

[landell, 2017-04-12 07:03:13]

landell@opera.com changed reviewers:
+ mtrofin@google.com

[landell, 2017-04-26 06:37:19]

On 2017/04/10 09:40:05, landell wrote:
> @reviewer: PTAL. Small remove warning fix.

Ping!

[Mircea Trofin, 2017-04-26 14:16:56]

On 2017/04/26 06:37:19, landell wrote:
> On 2017/04/10 09:40:05, landell wrote:
> > @reviewer: PTAL. Small remove warning fix.
> 
> Ping!

Lgtm

Alternatively, we could stdout on error?

[landell, 2017-04-27 06:47:04]

On 2017/04/26 14:16:56, Mircea Trofin wrote:
> On 2017/04/26 06:37:19, landell wrote:
> > On 2017/04/10 09:40:05, landell wrote:
> > > @reviewer: PTAL. Small remove warning fix.
> > 
> > Ping!
> 
> Lgtm
> 
> Alternatively, we could stdout on error?

Sure, is there a log macro I should use or just a regular printf?

[Mircea Trofin, 2017-04-27 15:21:23]

On 2017/04/27 06:47:04, landell wrote:
> On 2017/04/26 14:16:56, Mircea Trofin wrote:
> > On 2017/04/26 06:37:19, landell wrote:
> > > On 2017/04/10 09:40:05, landell wrote:
> > > > @reviewer: PTAL. Small remove warning fix.
> > > 
> > > Ping!
> > 
> > Lgtm
> > 
> > Alternatively, we could stdout on error?
> 
> Sure, is there a log macro I should use or just a regular printf?

I think we just use OFStream - like this:

OFStream os(stderr);
os << "Error while dumping wasm file" << std::endl;

Oh - you were probably going to do that, but please also update the CL
description to reflect this update.

[landell, 2017-04-28 07:22:05]

Description was changed from

==========
Ignore result from fwrite

Shut up compiler warning

BUG=
==========

to

==========
Print error message if dumping wasm file fails

BUG=
==========

[landell, 2017-05-02 07:44:15]

The CQ bit was checked by landell@opera.com

[landell, 2017-05-02 07:44:16]

The patchset sent to the CQ was uploaded after l-g-t-m from mtrofin@chromium.org
Link to the patchset: https://codereview.chromium.org/2806313002/#ps20001
(title: "Print message on error")

[commit-bot: I haz the power, 2017-05-02 07:44:24]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-02 08:12:46]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493711055746240,
"parent_rev": "0740168409d8f8b494d907b07c1ead8cdf70e3e8", "commit_rev":
"d15687df02ea9a47bdc4755a0415419092f97f37"}

[commit-bot: I haz the power, 2017-05-02 08:12:54]

Description was changed from

==========
Print error message if dumping wasm file fails

BUG=
==========

to

==========
Print error message if dumping wasm file fails

BUG=

Review-Url: https://codereview.chromium.org/2806313002
Cr-Commit-Position: refs/heads/master@{#45007}
Committed:
https://chromium.googlesource.com/v8/v8/+/d15687df02ea9a47bdc4755a0415419092f...
==========

[commit-bot: I haz the power, 2017-05-02 08:12:55]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/d15687df02ea9a47bdc4755a0415419092f...