result of code comparison

[Andreas Greuel]

here's another test case:

- Function: Compare Code
- Code type: Plain Text
- Base Text: IF RNGZ2 < 0
- New Text: IF P672-NGZ2 < 0
- Press Execute

expected: "R" replaced with "P672-"
displayed: "RNGZ" replaced with "P67" and " < 0" replaced with "-NGZ2 < 0"

[Cheney, Edward A SFC USARMY (US)]

Andreas,

This is a really hard one. If in the "new" sample P672 is replaced with P673 the ideal pattern is realized in the output. The problem in this case is that 2 of RNG72 identifies with the 2 of P672. What makes your designed output more correct than the current output is that "NGZ2" is a longer sequences of characters than the "2" of P672. I will sleep on this over the night and consider how I can account for character length in the matching.

Thanks,
Austin
________________________________________
From: prett...@googlegroups.com [prett...@googlegroups.com] on behalf of Andreas Greuel [andreas...@googlemail.com]
Sent: Monday, May 26, 2014 3:12 AM
To: prett...@googlegroups.com
Subject: [Pretty Diff] result of code comparison

--
You received this message because you are subscribed to the Google Groups "Pretty Diff" group.
To unsubscribe from this group and stop receiving emails from it, send an email to pretty-diff...@googlegroups.com.
To post to this group, send email to prett...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/pretty-diff/604d1f9a-a29e-46a9-ad04-3fb79cab5cd3%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[Andreas Greuel]

here's a similar test case:

- Function: Compare Code
- Code type: Plain Text

- Base Text: MOVE 0 TO RBETR
- New Text: MOVE 0 TO P672-BETR
- Press Execute

expected: "R" replaced with "P672-"

displayed: "P672-BET" inserted before "RBETR", which aren't the correct changes to produce the new text.

[Cheney, Edward A SFC USARMY (US)]

Andreas,

Please try the tool now and see if it is better.

Austin
________________________________________
From: prett...@googlegroups.com [prett...@googlegroups.com] on behalf of Andreas Greuel [andreas...@googlemail.com]

Sent: Wednesday, May 28, 2014 2:29 AM
To: prett...@googlegroups.com
Subject: Re: [Pretty Diff] result of code comparison

--

You received this message because you are subscribed to the Google Groups "Pretty Diff" group.
To unsubscribe from this group and stop receiving emails from it, send an email to pretty-diff...@googlegroups.com.
To post to this group, send email to prett...@googlegroups.com.

To view this discussion on the web visit https://groups.google.com/d/msgid/pretty-diff/09804e76-cef6-4ea4-9a21-cefbcf6c3a2e%40googlegroups.com.

[Andreas Greuel]

the 2nd test case works fine now, but unfortunately the following test case doesn't display the correct differences:

- Function: Compare Code
- Code type: Plain Text

- Base Text: MOVE 'D1' TO RZUGUP
- New Text: MOVE 'D1' TO P672SCHA
- Press Execute

[Austin Cheney]

Andreas,

Please try it now.

Thanks,
Austin

[Andreas Greuel]

all issues fixed so far, here's another one:

- Function: Compare Code
- Code type: Plain Text

- Base Text: MOVE DZUDAV TO R67DA
- New Text: MOVE DZUDAV TO P672ZUDA
- Press Execute

expected: "R" replaced with "P" and "2ZU" inserted
displayed: "R" replaced with "P" and "DA" replaced with "2ZUDA"

[Akshay phadte]

Hi Austin,

I am facing a similar issue:
Base Text :the pretty diff is very good lib ,i want to use this lib for text or string compare.

New Text: the pretty diff is very good lib ,i want to use this particular lib for text or string compare.


Result is showing "particular lib" as the modified text, but actually i have just added "particular" string to New text.

Similarly if my New Text: the pretty diff very is very good lib ,i want to use this particular lib for text or string compare.

The result is not up to the mark.


Thanks
Akshay

[Akshay phadte]

On Friday, June 13, 2014 at 12:43:52 AM UTC-4, Austin Cheney wrote:

Hi Austin,


I am also facing similar issues:

Base Text:the pretty diff is very good lib ,i want to use this lib for text or string compare.

New Text:the pretty diff very is very good lib ,i want to use this particular lib for text or string compare.


compare Result:"is very good lib ,i " and " particular lib "


Thanks
Akshay

[Austin Cheney]

Akshay,

I opened an issue for this: https://github.com/prettydiff/prettydiff/issues/241

The next release is dedicated to fixing diff and node related bugs: https://github.com/prettydiff/prettydiff/milestones/v1.16.5

[Akshay phadte]

Hi Austin,

Thanks For the early query updates.

Base Text : ver_con_31_con_31 (265) - document deleted. EST2016/02/12 at 13:43
New Text : ver_con_31_con_31 (265) - document deleted. EIndEStTEST2016/02/12 at 13:43
Result : EIndEStTEST2016/02/12
Expected Result :EIndEStT

I have found the diff breaking in the above senario. Can we have a fix for it?

Thanks
Akshay

[in...@prettydiff.com]

Akshay,

Thanks for that. I have opened an issue for tracking:
https://github.com/prettydiff/prettydiff/issues/289

Austin

> --
> You received this message because you are subscribed to the Google Groups
> "Pretty Diff" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to pretty-diff...@googlegroups.com.
> To post to this group, send email to prett...@googlegroups.com.
> To view this discussion on the web visit

> https://groups.google.com/d/msgid/pretty-diff/b4b30800-a2bc-4111-9ef6-4ab896f0e11b%40googlegroups.com.