Tensegritics/ClojureDart

Port new 1.11 behavior for associative varargs

Open

#109 aperta il 13 giu 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Clojure (115 fork)batch import
help wanted

Metriche repository

Star
 (1618 star)
Metriche merge PR
 (Merge medio 11g 17h) (6 PR mergiate in 30 g)

Descrizione

I think this is related to issue #96.

Calling a variadic function that contains argument destructuring fails with the following exception: Unhandled Exception: Invalid argument(s): Index out of bounds

Example:

(defn one [& {:keys [a b]
              :or {a "a" b "b"}}]
  (dart:core/print (str a b)))

  (one) ;=> ab
  (one nil) ;=> ...Index out of bounds...
  (one {}) ;=> ...Index out of bounds...
  (one {:a "x"}) ;=> ...Index out of bounds...
  (one {:a "x" :b "y"}) ;=> ...Index out of bounds...


(defn two [a & {:keys [b c]
                :or {b "b" c "c"}}]
  (dart:core/print (str a b c)))
  
  (two "x") ;=> xbc
  (two "x" nil) ;=> ...Index out of bounds...
  (two "x" {}) ;=> ...Index out of bounds...
  (two "x" {:b "y"}) ;=> ...Index out of bounds...
  (two "x" {:b "y" :c "z"}) ;=> ...Index out of bounds...

Guida contributor

Direzione di ricerca
Inizia esaminando l'issue correlato #96 e il codice sorgente di ClojureDart, in particolare la generazione del codice per le funzioni variadiche. Osserva l'implementazione della destrutturazione `&` nel compilatore, probabilmente in file come `src/clojure/dart/compiler.clj` o simili. Fai riferimento alle note di rilascio di Clojure 1.11 per il nuovo comportamento degli argomenti variadici associativi per comprendere la correzione prevista.
Tech stack
dart
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ClojureDart compiler architectureClojure variadic functionsDart function parametersIssue #96 context
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
20