Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(78)

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

Created:
10 years, 4 months ago by Evan Martin
Modified:
9 years, 7 months 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

Messages

Total messages: 3 (0 generated)
brettw
LGTM
10 years, 4 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 ...
10 years, 4 months ago (2010-07-28 20:57:52 UTC) #2
Evan Martin
10 years, 4 months ago (2010-07-28 20:58:41 UTC) #3
the *PLOG family of loggers try to log errno in a thread-safe manner

Powered by Google App Engine
This is Rietveld 408576698