Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Two WebARKitLog.h headers share the include guard WEBARKIT_LOG_H

Chiusa Adatta ai principianti
  • Qualcosa non va?
#84 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
70/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
c, cpp
Ambito
ar-vr-xr

Direzione di ricerca

I due header sono include/WebARKit/WebARKitLog.h e WebARKit/include/WebARKitLog.h, e iniziano entrambi con la stessa guardia WEBARKIT_LOG_H. Confrontali, poi controlla le implementazioni corrispondenti in lib/SRC/WebARKit/WebARKitLog.cpp e WebARKit/WebARKitLog.cpp. Il lavoro è completo quando ogni header ha la propria guardia e un'unità di traduzione che include entrambi compila. Scegli con un maintainer tra rinominare la guardia o far convergere le due versioni, perché la issue lascia aperta questa scelta.

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

Descrizione

bug C/C++ code

Description

WebARKitLib has two different logging headers with the same include guard:

header API depends on
include/WebARKit/WebARKitLog.h webarkitLOGi/e(const std::string&, …) overloads, implemented in lib/SRC/WebARKit/WebARKitLog.cpp <emscripten.h>, unconditionally
WebARKit/include/WebARKitLog.h ARUtil-style webarkitLog() with WEBARKIT_LOGd/i/w/e macros and webarkitLogLevel, implemented in WebARKit/WebARKitLog.cpp none, native-compatible

Both start with:

#ifndef WEBARKIT_LOG_H
#define WEBARKIT_LOG_H

so a translation unit that includes both gets only the first one. Code that uses the other logger then fails to compile, because its functions or macros are undeclared. The two implementations define different symbols, so they do link together.

Why it matters now

The planned ARToolKitNFTCore (webarkit/jsartoolkitNFT#683) has to compile natively, so it will log through WebARKit/include/WebARKitLog.h. The jsartoolkitNFT Embind adapters keep using include/WebARKit/WebARKitLog.h. The design works around the clash: the core includes its logger only in .cpp files, never in a public header. That is a rule to remember rather than a guarantee.

Possible fixes

  • Give each header its own guard (e.g. WEBARKIT_LOG_EM_H for the Emscripten one), or
  • make include/WebARKit/WebARKitLog.h include <emscripten.h> only under #ifdef __EMSCRIPTEN__ and converge on one logger.

Refs webarkit/jsartoolkitNFT#683.

Lingua principale
C++
Stelle
21
Fork
8
Merge medio
7h 20m
PR unite (30g)
26

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 webarkit/WebARKitLib

Tutte le issue di webarkit/WebARKitLib

Issue simili

Altre issue su C++

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.