This is the mail archive of the
mailing list for the glibc project.
Re: [PATCH] Fix location where math-vector-fortran.h is installed.
Revised patch looks good to me except:
> -! Platform-specific declarations of SIMD math functions for Fortran. -*- f90 -*-
> +! Platform-specific declarations of SIMD math functions for Fortran.
Let's keep the -*- f90 -*- annotations, they will also be helpful for
future people editing these files. For instance, on my computer, both
emacs and vim use the Fortran syntax highlighting rules for these
files when the annotation is present, and the C rules when it's
> + # Skip Fortran header files
> + if 'finclude' in header:
> + continue
I think this conditional should be
if header.startswith('finclude/') or '/finclude/' in header
so that it only applies to headers in the directory 'finclude', not
any hypothetical future headers that happen to have the string
'finclude' in their base names. (I don't know why that would happen,
but if it ever did, I can see someone being really confused why it
wasn't getting tested, so some extra defensiveness now seems like a