Re: [mdb-dev] mdb-sql: Readline Completion Support

[Mark K. <kil...@nb...>]

Michael Wood wrote:
> Hi
> 
> Since I am basically just a lurker on this list, feel free to ignore
> these comments:

Me too.  I want to use the MDB tools to look at a db under my Linux box 
(I *hate* rebooting into Windows!)

> 2.)  Gratuitous whitespace changes get in the way of what your patch is
> doing.  The reviewer has to waste time looking through the whitespace
> changes to see if you made any other changes in that section besides
> just the whitespace changes.

In most diff programs, you can ignore whitespace changes.  Take the GNU 
diff on my system:

-b     Ignore changes in amount of white space.
-B     Ignore changes that just insert or delete blank lines.

White space changes are actually a good thing, if they improve 
readability, so I wouldn't poo-poo them entirely...

You can add these options to your .cvsrc if you are using cvs.

Just a hint...

MK
-- 
"An expert is a man who has made all the mistakes which can be made,
in a narrow field."   - Niels Bohr
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
Mark Kilfoil                                   <kil...@nb...>
  Software Designer/Developer, Radio Host, Gamer, Curmudgeon, Oddball
     From the FINE mega-city of Fredericton, New Brunswick, Canada
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -