codee-com/open-catalog

[Fortran] Replace `if (integer)` with `if (logical)`

Open

#38 aperta il 24 set 2024

Vedi su GitHub
 (2 commenti) (2 reazioni) (0 assegnatari)Fortran (12 fork)auto 404
Fortrangood first issuenew check

Metriche repository

Star
 (105 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

In Fortran there is no equivalence between logical and integer variables, so 0 is not a replacement for .false., and 1 does not replace .true.. Yet, some compilers (ifort) will happily compile if (1).

Example:

if (0) write(*, *) 'Test is false'
if (1) write(*, *) 'Test is true'

A slightly more meaningful example

integer val
val = 1
if (val) write(*, *) 'Test is true'

This should be replaced by

integer val
val = 1
if (val > 0) write(*, *) 'Test is true'

Guida contributor

Direzione di ricerca
Identifica tutte le occorrenze in cui una variabile intera viene utilizzata in un contesto logico all'interno di istruzioni if e sostituiscile con espressioni logiche appropriate. Ad esempio, sostituisci 'if (val)' con 'if (val > 0)' o simili.
Tech stack
Nessuno
Dominio
documentation
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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
Fortran
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85