curran / curran/google-diff-match-patch

javascript diff_cleanupSemantic uses negative indexes in the equalities array

Aperta
#89 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
auto-migrated Priority-Medium Type-Defect
Lingua principale
Python
Stelle
17
Fork
2
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

```
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:
- [diff_match_patch.js.patch](https://storage.googleapis.com/google-code-attachments/google-diff-match-patch/issue-89/comment-0/diff_match_patch.js.patch)

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Inizia con il diff_match_patch.js.patch allegato e confrontalo con l’implementazione JavaScript di diff_cleanupSemantic presente nel repository. Verifica come viene aggiornato equalitiesLength e conferma che la modifica completata evita gli indici negativi nell’array delle uguaglianze, preservando al contempo il comportamento della pulizia semantica, inclusa la questione di compatibilità con Safari.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
tooling
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.