eclipse-platform / eclipse-platform/eclipse.platform

Make Text Compare more smart

Offen
#66 9 Kommentare 4 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Java
Sterne
165
Forks
174
Ø Merge
2 T. 8 Std.
Gemergte PRs (30 T.)
22

Beschreibung

I recently noticed that text-compare seems to present a undesirable changeset if the following happen:

1. You have some repeated code block with common start (e.g. `` in a pom.xml or `@Test` in a Unit test) and ending (e.g. `` in a pom.xml or `}` in a Unit test
2. You insert a new block in between
3. Now the change set spans into the next block

![grafik](https://user-images.githubusercontent.com/1331477/174729794-fad66f74-3d98-4e3e-aa90-05db79fd2e1e.png)

**What would be expected is that the whole block is marked as an insertion (here line 74 - 78)**

Github for example displays this more nice

![grafik](https://user-images.githubusercontent.com/1331477/174731417-3eaa11a0-aa9c-4e00-bb5b-e9f5ed582191.png)

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, die text-compare-Implementierung zu lokalisieren und das im Issue beschriebene Szenario mit wiederholten Blöcken zu reproduzieren. Vergleiche den generierten changeset mit dem erwarteten Ergebnis, bei dem der eingefügte Block als eine Einfügung markiert ist, und nutze den verlinkten pull request als Kontext für den bisherigen Stand.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
java
Bereich
tooling
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.