[Top][All Lists]
[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
Re: [avrdude-dev] Fix warnings patch, E. Weddington, 2003/08/28
- Re: [avrdude-dev] Fix warnings patch,
Brian Dean <=
Re: [avrdude-dev] Fix warnings patch, E. Weddington, 2003/08/28
Re: [avrdude-dev] Fix warnings patch, eric, 2003/08/28
Re: [avrdude-dev] Fix warnings patch, E. Weddington, 2003/08/28
Re: [avrdude-dev] Fix warnings patch, E. Weddington, 2003/08/28
Re: [avrdude-dev] Fix warnings patch, E. Weddington, 2003/08/29