codee-com/open-catalog

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

Open

#38 ouverte le 24 sept. 2024

Voir sur GitHub
 (2 commentaires) (2 réactions) (0 assignés)Fortran (12 forks)auto 404
Fortrangood first issuenew check

Métriques du dépôt

Stars
 (105 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

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'

Guide contributeur

Direction de recherche
Identifiez toutes les occurrences où une variable entière est utilisée dans un contexte logique à l'intérieur d'instructions if et remplacez les par des expressions logiques appropriées. Par exemple, remplacez 'if (val)' par 'if (val > 0)' ou similaire.
Stack technique
Aucun
Domaine
documentation
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Fortran
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85