refactor: Deduplicate 'pointer type from origin' derivation
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 74/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- compilers
Direzione di ricerca
Start by reading createPtrFromOrigin in packages/typegpu/src/data/ptr.ts, then compare its uses in packages/typegpu/src/tgsl/wgslGenerator.ts, packages/typegpu/src/tgsl/forOfUtils.ts, and packages/typegpu/src/data/ref.ts. Check the existing isAddressableOrigin from #3075 and confirm that the shared derivation concretizes the inner type while d.ref() keeps its explicit fallback. Done means the three paths use one consistent helper without changing their intended fallback behavior.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The same "derive a pointer type from a snippet's origin" logic is written out in three places, each slightly differently:
packages/typegpu/src/tgsl/wgslGenerator.ts(_aliasConstStatement):createPtrFromOrigin(eq.origin, concretize(type))+invariant(ptrType !== undefined, ...)packages/typegpu/src/tgsl/forOfUtils.ts(getElementType): same as above, copy-pastedpackages/typegpu/src/data/ref.ts(d.ref()GPU impl):createPtrFromOrigin(value.origin, type) ?? ptrFn(concretize(type)). It falls back to a function-scope pointer instead of asserting, and only concretizes on the fallback path
Proposal
Add a single helper next to createPtrFromOrigin in data/ptr.ts, e.g. ptrTypeFor(snippet), that concretizes the inner type and asserts addressability. d.ref() would then pick its fallback explicitly using isAddressableOrigin (added in #3075).
Follow-up to the review of #3075.
- Lingua principale
- TypeScript
- Stelle
- 3.3k
- Fork
- 135
- Merge medio
- 4g 1h
- PR unite (30g)
- 64
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
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 software-mansion/TypeGPU
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
software-mansion/TypeGPU#3219 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
texture.write(VideoFrame) uses the padded coded size instead of the visible sizeForse già presa @RaphaelFakhri l’ha presa 1 giorno fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
software-mansion/TypeGPU#3213 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
software-mansion/TypeGPU#3008 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
software-mansion/TypeGPU#2948 · 2 commenti ·
I maintainer di solito rispondono entro 2 giorni
-
docs: Fix camera orientation in "Monocular Light Injection" exampleForse già presa @pullfrog l’ha presa 61 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
software-mansion/TypeGPU#2890 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di software-mansion/TypeGPU
Issue simili
-
frontend
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 86/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
ranxianglei/billion-context#4172 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
awslabs/threat-composer#362 ·
I maintainer di solito rispondono entro 1 giorno
-
Area/Cloud Needs Triage Type/Bug
Difficoltà 1/5 1-3 ore Idoneità per principianti 86/100
wso2/product-integrator#2847 ·
I maintainer di solito rispondono entro 1 giorno