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 :-)
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