Code review comment for lp:~charlesk/indicator-messages/fix-937438

Revision history for this message
Ted Gould (ted) wrote :

On Tue, 2012-02-21 at 16:34 +0000, charles wrote:
> The code seems to be fine. It looks like Coverity is throwing this
> warning because it's confused by the overlapping goto's.

Yeah, I agree, but in general we should probably get rid of goto's. I
was probably just being lazy when I wrote this code anyway, I like the
updated code better :-)

  review approve
  status approved

review: Approve

« Back to merge proposal