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

Issue 538005: linux: grab device name before closing it (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 3 months ago by Evan Martin
Modified:
2 years, 11 months ago
Reviewers:
awong
CC:
chromium-reviews_googlegroups.com, scherkus, fbarchard, awong, Alpha
Visibility:
Public.

Description

linux: grab device name before closing it

PcmClose frees the handle regardless of whether there's an error
while shutting down, so grab the PcmName result first in case we
want to use it in an error message.

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

Patch Set 1 #

Patch Set 2 : better message #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Lint Patch
M media/audio/linux/alsa_output.cc View 1 1 chunk +3 lines, -2 lines 0 comments ? errors Download
Trybot results:
Commit:

Messages

Total messages: 2
Evan Martin
(i confirmed this by checking the alsa source)
4 years, 3 months ago #1
awong
4 years, 3 months ago #2
LGTM.

Good catch!
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6