asmaloney/libE57Format

MinGW fails to compile due to warnings

Aperta

#247 aperta il 21 apr 2023

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (82 fork)auto 404
bughelp wanted

Metriche repository

Star
 (187 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

(From here: https://github.com/asmaloney/libE57Format/issues/245#issuecomment-1518088249)

I think his fix in CRCpp (which is what I used here) won't work for MinGW.

@UshnaGul - can you try changing the checks in CheckedFile.cpp from

#if defined( WIN32 ) || defined( _WIN32 ) || defined( WINCE )

to

#if defined( _MSC_VER )

That should turn off the pragmas MinGW doesn't understand. Not sure if it will throw other errors though.

Guida contributor

Direzione di ricerca
Modificare i controlli del preprocessore #if defined in CheckedFile.cpp per utilizzare MSC VER invece di WIN32 per evitare gli avvisi pragma di MinGW.
Tech stack
ccpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++MinGW
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70