avrdude-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [avrdude-dev] Fix warnings patch


From: Brian Dean
Subject: Re: [avrdude-dev] Fix warnings patch
Date: Thu, 28 Aug 2003 15:37:57 -0400
User-agent: Mutt/1.4.1i

On Thu, Aug 28, 2003 at 07:14:26PM +0000, E. Weddington wrote:

> It gets rid of 2 warnings about "variable may not be 
> initialized" for prescale and cmatch, and has the added 
> benefit of getting rid of a goto.
> 
> It's a little easier to see when patched. :-/

Excellent!  Though, in general, a few well-placed goto's can sometimes
make the code easier to follow, but I have no opinion about that
either way in the code you were working on.

Looks good to me, I don't see any issues.

Cheers,
-Brian




reply via email to

[Prev in Thread] Current Thread [Next in Thread]