This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 25844 - CVS | DIFF, off-by-one error
Summary: CVS | DIFF, off-by-one error
Status: RESOLVED WONTFIX
Alias: None
Product: versioncontrol
Classification: Unclassified
Component: CVS (show other bugs)
Version: 3.x
Hardware: PC Windows ME/2000
: P3 blocker (vote)
Assignee: issues@versioncontrol
URL:
Keywords:
: 32015 (view as bug list)
Depends on: 26146
Blocks:
  Show dependency tree
 
Reported: 2002-07-19 20:10 UTC by _ gtzabari
Modified: 2007-01-04 17:14 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Exception Reporter:


Attachments
Off-by-one error in left window (37.53 KB, image/gif)
2002-07-19 20:11 UTC, _ gtzabari
Details
Similar differeces, the result is O.K. (15.63 KB, image/gif)
2002-07-22 14:37 UTC, Martin Entlicher
Details
New version of file (2.77 KB, text/plain)
2002-07-22 16:25 UTC, _ gtzabari
Details
Old version of file (2.72 KB, text/plain)
2002-07-22 16:27 UTC, _ gtzabari
Details
The diff between the attached sources. (27.32 KB, image/gif)
2002-07-23 10:15 UTC, Martin Entlicher
Details

Note You need to log in before you can comment on or make changes to this bug.
Description _ gtzabari 2002-07-19 20:10:13 UTC
dev build 200206201555

   Check out the screenshot, notice the off-by-one error 
in the left window on the second change.
Comment 1 _ gtzabari 2002-07-19 20:11:12 UTC
Created attachment 6822 [details]
Off-by-one error in left window
Comment 2 Martin Entlicher 2002-07-19 20:16:04 UTC
Is it reproducible??
If so, can you please attach the two file versions where this happens?
Thanks.
Comment 3 Martin Entlicher 2002-07-22 14:35:47 UTC
Well, I'm not able to reproduce it. Please attach the sources in which
this happen. I've tried making similar differences, but everything
works O.K.
Comment 4 Martin Entlicher 2002-07-22 14:37:34 UTC
Created attachment 6839 [details]
Similar differeces, the result is O.K.
Comment 5 _ gtzabari 2002-07-22 16:23:39 UTC
Attaching sources...
Comment 6 _ gtzabari 2002-07-22 16:25:17 UTC
Created attachment 6844 [details]
New version of file
Comment 7 _ gtzabari 2002-07-22 16:27:45 UTC
Created attachment 6845 [details]
Old version of file
Comment 8 _ gtzabari 2002-07-22 16:28:27 UTC
Please try CVS | DIFF using the two attachments (old, new 
version of file) and you should end up with the same 
result as me under the same build number.
Comment 9 Martin Entlicher 2002-07-23 10:13:36 UTC
Unfortunately I was not able to reproduce the bug neither in dev build
200206200100 nor in recent release34 build 200207182340.
Your build #200206201555 is probably some old continual build, which I
can not find. But #200206200100 should not differ that much.
The diff as I get it attached...
Comment 10 Martin Entlicher 2002-07-23 10:15:47 UTC
Created attachment 6858 [details]
The diff between the attached sources.
Comment 11 Martin Entlicher 2002-07-23 10:21:54 UTC
It's not clear to me why it does not work for you. There is missing
one deleted line in your screeshot. This has caused the problem.
Can you please attach the diff from the command-line CVS executable
"cvs diff ...", so that we can be sure, that the server returns the
correct list of differences?
Does the same problem appear in recent 3.4 builds?

In the mean time resolving as "worksforme".
Comment 12 _ gtzabari 2002-07-23 13:15:47 UTC
ha! Just remembered something that might be vital!

At work I use "Ignore whitespace" and "Ignore blank spaces"; maybe one
of those features is broken leading to the spaces we see!

Please try again on your end with these features enabled.
Comment 13 Jiri Kovalsky 2002-07-23 14:09:24 UTC
Reopening, I was able to reproduce it with mentioned switches. Good
memory, Gili ! ;-) RC1 build #200207222340 of NetBeans 3.4.
Comment 14 Jiri Kovalsky 2002-07-23 14:16:14 UTC
This is another good candidate for Release Notes because we can't make
it into NetBeans 3.4. At the same time it concerns Generic VCS module too.
Comment 15 Martin Entlicher 2002-07-23 14:35:31 UTC
O.K. the "Ignore Blank Lines" was the key switch, which caused the
text to be shifted.

Every switch, that cause some added/removed lines to be excluded from
the diff report will shift the text and coloring of differences in one
source with respect to the other.
Comment 16 Martin Entlicher 2002-07-30 19:19:13 UTC
Working on a fix into 4.0 ...
Comment 18 Jiri Kovalsky 2002-08-05 13:09:43 UTC
Works fine in development build #200208050100 of NetBeans 4.0.
Comment 19 Patrick Keegan 2002-08-08 21:22:03 UTC
proposed release note:

The line numbers do not always match when using the CVS Diff command 
with the Ignore Blank Lines option turned on.
Comment 20 Jiri Kovalsky 2002-08-09 09:46:17 UTC
Exactly !
Comment 21 Martin Entlicher 2002-11-08 08:37:56 UTC
Adding 3.4.1_CANDIDATE, this can go into 3.4.1 IMHO.
Comment 22 Jaroslav Tulach 2002-12-03 09:54:20 UTC
Hi. This issue is marked as 3.4.1_CANDIDATE. It means that it should be
integrated into release341 one branch. The plan at
http://www.netbeans.org/devhome/docs/releases/34/index.html expected beta1 to be
produced on Dec01. That did not happen due to a lot of outstanding not
integrated candidates like this one. 

Would it be possible to spend few minutes by backporting this fix? Thank you in
advance.
Comment 23 Martin Entlicher 2002-12-05 18:29:54 UTC
Merged into release341 branch.

/cvs/javacvs/src/org/netbeans/modules/cvsclient/commands/diff/DiffCommandDisplayer.java,v 
<--  DiffCommandDisplayer.java
new revision: 1.10.82.1; previous revision: 1.10
/cvs/javacvs/src/org/netbeans/modules/cvsclient/commands/diff/DiffDisplayer.java,v 
<--  DiffDisplayer.java
new revision: 1.19.20.1; previous revision: 1.19
/cvs/javacvs/src/org/netbeans/modules/javacvs/commands/CvsDiff.java,v 
<--  CvsDiff.java
new revision: 1.11.50.1; previous revision: 1.11
Comment 24 Richard Gregor 2003-02-28 14:04:33 UTC
Fix of this bug causes 28195 which is P2 so I did rollback of this fix
 and reopen this bug. We need to find another solution for fixing this
bug.
Comment 25 Martin Entlicher 2003-02-28 14:44:41 UTC
Unfortunately this fix caused a sever regression.
I'm scheduling this for 4.0.
Comment 26 Martin Entlicher 2003-03-18 15:36:16 UTC
*** Issue 32015 has been marked as a duplicate of this issue. ***
Comment 27 Martin Entlicher 2004-01-14 14:55:02 UTC
As described at
http://www.netbeans.org/servlets/ReadMsg?msgId=655179&listName=nbusers
the development of javacvs module has been stopped.
Comment 28 _ gtzabari 2004-01-14 16:30:17 UTC
Martin,

   If this issue is not fixed it affects Generic VCS as well so we
should not close this issue.
Comment 29 Patrick Keegan 2004-03-03 23:14:44 UTC
removing RELNOTE keyword; if this bug still occurs in vcsgeneric,
point me to a related bug report