From 03e76e962ed4a1f53135c1a91f24300c4f4bb039 Mon Sep 17 00:00:00 2001
From: Pascal Massimino <pascal.massimino@gmail.com>
Date: Wed, 1 Apr 2015 00:32:52 -0700
Subject: [PATCH] clarify the comment about double-setting the status in
 SetError()

Change-Id: I67107220b7a84459592c726dab95483acd4f59f2
---
 src/dec/vp8.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/src/dec/vp8.c b/src/dec/vp8.c
index e73d7d31..22228ab5 100644
--- a/src/dec/vp8.c
+++ b/src/dec/vp8.c
@@ -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;