Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Function order #74

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Function order #74

wants to merge 2 commits into from

Commits on Mar 4, 2021

  1. Move checkFmt to top of file to avoid compiler bug

    Some versions of gfortran ICE with the function in
    the body of the file. Defining it first seems to be
    fine and should be valid Fortran 2008.
    See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85138
    and #57
    andreww committed Mar 4, 2021
    Configuration menu
    Copy the full SHA
    6739255 View commit details
    Browse the repository at this point in the history
  2. Move getTextContent_len towards the top of the file

    Some versions of gfortran ICE with the function in
    the body of the file. Defining it first seems to be
    fine and should be valid Fortran 2008.
    See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85138
    and #57
    
    Do the actual move in the m4 and rebuild (we comit the
    resulting F90 files so users don't need an m4 implementation
    to build FoX). At the same time propogate a couple of changes
    from the F90 back into other m4 files so we don't loose those
    changes.
    andreww committed Mar 4, 2021
    Configuration menu
    Copy the full SHA
    881e668 View commit details
    Browse the repository at this point in the history