clarify the comment about double-setting the status in SetError()

Change-Id: I67107220b7a84459592c726dab95483acd4f59f2
This commit is contained in:
Pascal Massimino 2015-04-01 00:32:52 -07:00 committed by James Zern
parent 9fecdd713e
commit 03e76e962e

View File

@ -75,10 +75,7 @@ void VP8Delete(VP8Decoder* const dec) {
int VP8SetError(VP8Decoder* const dec,
VP8StatusCode error, const char* const msg) {
// TODO This check would be unnecessary if alpha decompression was separated
// from VP8ProcessRow/FinishRow. This avoids setting 'dec->status_' to
// something other than VP8_STATUS_BITSTREAM_ERROR on alpha decompression
// failure.
// The oldest error reported takes precedence over the new one.
if (dec->status_ == VP8_STATUS_OK) {
dec->status_ = error;
dec->error_msg_ = msg;