curran / curran/google-diff-match-patch
javascript diff_cleanupSemantic uses negative indexes in the equalities array
Offen
Dieses Issue hat noch niemand übernommen.
auto-migrated
Priority-Medium
Type-Defect
- Vorherrschende Sprache
- Python
- Sterne
- 17
- Forks
- 2
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
javascript diff_cleanupSemantic uses negative indexes in the equalities array.
This is because it equalitiesLength is allowed to go into a negative range.
Normally this doesn't cause any problems, because it will just be stored as a
property on the array, but Safari 1.5.x seems to have some issues and doesn't
always get the value that's stored in equalities[-1] back out. (Or perhaps it
didn't properly go in in some cases.)
Original issue reported on code.google.com by d...@twkie.net on 21 Jun 2013 at 10:48
Attachments:
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der angehängten diff_match_patch.js.patch und vergleiche sie mit der JavaScript-Implementierung von diff_cleanupSemantic im Repository. Überprüfe, wie equalitiesLength aktualisiert wird, und bestätige, dass die abgeschlossene Änderung negative Indizes des Gleichheitsarrays vermeidet und dabei das Verhalten der semantischen Bereinigung einschließlich der Safari-Kompatibilitätsfrage bewahrt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100