[Suggestion] Naming schema for different binding types
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 20/100
- Tipo di issue
- Refactoring
- Chiarezza
- Da chiarire
- Stato di attività
- Ferma
- Stack tecnologico
- cpp
- Ambito
- computer-graphics
Direzione di ricerca
Esamina gli alias di BufferAllocation e gli esempi di BindingType::C/Cpp mostrati nell’issue. Il primo passo consiste nel raggiungere un accordo su una convenzione di denominazione; per considerarlo completato sarebbe necessario definire uno schema e applicarlo in modo coerente, ma l’issue non identifica file, test o un ambito concreto di implementazione.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
With @asuessenbach working hard on unifying the framework I was wondering if we could agree on some naming schema for the different binding types.
Using types aliases already makes things a lot easier to read like this:
// Usings
using BufferAllocationC = BufferAllocation<vkb::BindingType::C>;
using BufferAllocationCpp = BufferAllocation<vkb::BindingType::Cpp>;
// Usage
vkb::BufferAllocationC do_something();
vkb::BufferAllocationCpp do_something();
Maybe we can improve upon this a little bit by having the C-Binding be the default, and the only suffixing the C++ one?
Like this:
// Usings
using BufferAllocation = BufferAllocation<vkb::BindingType::C>;
using BufferAllocationCpp = BufferAllocation<vkb::BindingType::Cpp>;
// Usage
vkb::BufferAllocation do_something();
vkb::BufferAllocationCpp do_something();
Just some food for thought and maybe something we can discuss on the next call.
- Lingua principale
- C++
- Stelle
- 5.4k
- Fork
- 843
- Merge medio
- 17g 20h
- PR unite (30g)
- 3
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un 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 KhronosGroup/Vulkan-Samples
-
full_screen_exclusive should check for fullScreenExclusiveSupported in VkSurfaceCapabilities2KHRAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 73/100
KhronosGroup/Vulkan-Samples#1590 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
KhronosGroup/Vulkan-Samples#1582 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
KhronosGroup/Vulkan-Samples#1568 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
KhronosGroup/Vulkan-Samples#1566 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 35/100
KhronosGroup/Vulkan-Samples#1545 · 3 commenti ·
Tutte le issue di KhronosGroup/Vulkan-Samples
Issue simili
-
Remove source files from packaging for deb, zip, etcForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
OpenSpace/OpenSpace#4353 · 1 commento ·
I maintainer di solito rispondono entro 5 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
justcallmekoko/ESP32Marauder#1656 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Decompiler: convertCharRef() in xml.y overflows its int4 accumulator on long character referencesForse già presa @kaayzouee l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
NationalSecurityAgency/ghidra#10009 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
mutationpp/Mutationpp#337 ·