[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [Subclipse-users] diff flags for "create patch"

From: Mark Phippard <markphip_at_gmail.com>
Date: Thu, 18 Jun 2009 08:20:07 -0400

On Thu, Jun 18, 2009 at 2:12 AM, Stephen Ball<Stephen.Ball_at_ingres.com> wrote:
> I’m working with subclipse and the CDT, and I have a constant problem with
> the “C” editor in that it will insert white space in files I change (on a
> bunch of lines I haven’t changed). For general Eclipse use this isn’t a big
> deal because there’s an option in the compare editor to ignore the white
> space. But we’re using reviewboard to post submission plans, which require
> me to generate a patch containing the diffs. Conveniently svn diff allows
> the use of “-x –w” to ignore white space, which works perfectly, but I can’t
> find any way to inject this into the Subclipse generated “diff” command.

The JavaHL diff API does not have these flags exposed to it.

> This wouldn’t be a major deal, but we deal with fairly large C files (a few
> thousand lines), and when I make a 5 line change in one section it turns
> into 50 diffs all over the place in the file and none of the reviewers can
> make out what changes I actually made.

You are going to have the exact same problem when you commit. All
those whitespace changes are going to be committed. This is going to
complicate diff and more importantly merge and update for other users.
 Wouldn't it be better to just set your editor preferences in CDT so
that it did not do this?

-- 
Thanks
Mark Phippard
http://markphip.blogspot.com/
------------------------------------------------------
http://subclipse.tigris.org/ds/viewMessage.do?dsForumId=1047&dsMessageId=2363150
To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_subclipse.tigris.org].
Received on 2009-06-18 14:20:21 CEST

This is an archived mail posted to the Subclipse Users mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.