On 06/02/2010 15:14, Tom wrote:
kevin.morfitt@fearnside-systems.co.uk wrote:
Hi Wolfgang
On 06/02/2010 14:26, Wolfgang Denk wrote:
Dear "kevin.morfitt@fearnside-systems.co.uk",
In message4B6D687F.2060606@fearnside-systems.co.uk you wrote:
The patches are split so that each patch makes only one type of change, so there's only one thing that needs to be checked in each patch.
- patches 1 and 2 only make white-space changes (the change was too big
so I had to split it into 2 patches)
I think this should be merged into a single patch.
OK. It'll be about 140K though so I'llput it on a web site.
IMO So it could be posted to the mailing list, it would be better to split.
Was the whitespace changes done by hand or automatically with something like indent as mentioned http://www.denx.de/wiki/U-Boot/CodingStyle?
It was done automatically using Lindent but I had to do some manual tidying up because Lindent doesn't always get things right.
Kevin
Tom