append!(l1::MutableLinkedList, l2::MutableLinkedList) fails for empty l2
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 62/100
Direzione di ricerca
Apri src/mutable_list.jl all’implementazione di append!, intorno alla riga 153, e riproduci l’esempio A/B con un l2 vuoto. Segui questo caso e verifica che append!(A, B) termini; il lavoro è completato quando lo scenario con l2 vuoto non entra più in un ciclo infinito.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The append! function on master currently assumes l2.node.next != l2.node != l2.node.prev and fails if isempty(l2).
E.g.,
A = MutableLinkedList{Int}(1,2,3)
B = MutableLinkedList{Int}()
append!(A,B) # endless loop
This is not directly related to the more general issues #739 #886 #794 , which have been fixed on master.
The simplest solution seems to be adding a check for isempty(l2)? I can try making such PR later.
There don't seem to be any issues if isempty(l1).
- Lingua principale
- Julia
- Stelle
- 747
- Fork
- 261
- Merge medio
- 12g 6h
- PR unite (30g)
- 1
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di JuliaCollections/DataStructures.jl
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
JuliaCollections/DataStructures.jl#997 · 1 commento ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
-
several `mutable struct` type definitions do not require mutability, review and adjustForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
-
a slighly uncanny `UndefRefError` with default-empty SortedSetForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 52/100
JuliaCollections/DataStructures.jl#969 · 6 commenti ·
Tutte le issue di JuliaCollections/DataStructures.jl
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 76/100
-
documentation model-weights
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
FluxML/Metalhead.jl#311 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
jump-dev/MathOptInterface.jl#3113 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
JuliaMath/Polynomials.jl#647 ·