Chromium Code Reviews
Help | Chromium Project | Sign in
(3)

Issue 3067013: posix: print error message when mkdtemp fails. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 10 months ago by Evan Martin
Modified:
4 years ago
Reviewers:
brettw, Nico
CC:
chromium-reviews, John Grabowski, pam+watch_chromium.org, brettw-cc_chromium.org
Base URL:
http://src.chromium.org/git/chromium.git
Visibility:
Public.

Description

posix: print error message when mkdtemp fails. Failure seen on the buildbots. It would help to know why. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=54016

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M base/file_util_posix.cc View 1 chunk +3 lines, -1 line 1 comment Download
Trybot results:
Commit: CQ not working?

Messages

Total messages: 3 (0 generated)
brettw
LGTM
4 years, 10 months ago (2010-07-28 20:57:12 UTC) #1
Nico
lg http://codereview.chromium.org/3067013/diff/1/2 File base/file_util_posix.cc (right): http://codereview.chromium.org/3067013/diff/1/2#newcode424 base/file_util_posix.cc:424: DPLOG(ERROR) << "mkdtemp"; << errno? or does DPLOG ...
4 years, 10 months ago (2010-07-28 20:57:52 UTC) #2
Evan Martin
4 years, 10 months ago (2010-07-28 20:58:41 UTC) #3
the *PLOG family of loggers try to log errno in a thread-safe manner
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld ec887be