Clang format check not properly working for code behind compiler directives

Aperta
  • Qualcosa non va?
#1,537 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
55/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
cpp
Ambito
ci-cd, tooling

Direzione di ricerca

Inizia esaminando la configurazione del controllo Clang-format del repository e il punto di ingresso CI che formatta i file C++ modificati. Riproduci il controllo con lo snippet #ifdef/#else mostrato, quindi verifica che il codice formattato localmente dietro direttive del compilatore superi CI senza riformattazione manuale.

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

Descrizione

build

I've been battling Clang format for a larger PR and it seems that the Clang format check is not properly working for code that is guarded by a compiler define.

E.g. this code:

#ifdef SOMETHING
#else
	VkRenderPassBeginInfo render_pass_begin_info    = vkb::initializers::render_pass_begin_info();
	render_pass_begin_info.renderPass               = render_pass;
	render_pass_begin_info.renderArea.offset.x      = 0;
	render_pass_begin_info.renderArea.offset.y      = 0;
	render_pass_begin_info.renderArea.extent.width  = width;
	render_pass_begin_info.renderArea.extent.height = height;
	render_pass_begin_info.clearValueCount          = 5;
	render_pass_begin_info.pClearValues             = clear_values;
	render_pass_begin_info.framebuffer              = framebuffers[current_image_index];
#endif

Triggers a Clang format CI failrure, even though the formatting was chnaged like that Clang format itself. And this looks exactly like our formatting guidelines

Reverting this by hand to look like this:

#ifdef SOMETHING
#else
	VkRenderPassBeginInfo render_pass_begin_info = vkb::initializers::render_pass_begin_info();
	render_pass_begin_info.renderPass = render_pass;
	render_pass_begin_info.renderArea.offset.x = 0;
	render_pass_begin_info.renderArea.offset.y = 0;
	render_pass_begin_info.renderArea.extent.width = width;
	render_pass_begin_info.renderArea.extent.height = height;
	render_pass_begin_info.clearValueCount = 5;
	render_pass_begin_info.pClearValues = clear_values;
	render_pass_begin_info.framebuffer = framebuffers[current_image_index];

Fixes CI.

IMO that's wrong and needs to be fixed, otherwise every time I do a local Clang format for files changed I need to wait for CI to fail, decipher a cryptic message and manually fix stuff again.

Lingua principale
C++
Stelle
5.4k
Fork
844
Merge medio
7g 7h
PR unite (30g)
2

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di KhronosGroup/Vulkan-Samples

Tutte le issue di KhronosGroup/Vulkan-Samples

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.