[Bug fortran/31675] New: Fortran front-end and libgfortran should have a common header file

[Bug fortran/31675] New: Fortran front-end and libgfortran should have a common header file

[fxcoudert at gcc dot gnu dot org]

We should have a header file that's used for stuff shared by the front-end and
library, such as magic constants used in library functions arguments. It would
avoid code duplication between gcc/fortran/gfortran.h and libgfortran.h, with
all the benefits of maintainance and readability you can imagine.


-- 
           Summary: Fortran front-end and libgfortran should have a common
                    header file
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: P3
         Component: fortran
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: fxcoudert at gcc dot gnu dot org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31675

[Bug fortran/31675] Fortran front-end and libgfortran should have a common header file

[fxcoudert at gcc dot gnu dot org]

-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2007-04-23 22:40:05
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31675

[Bug fortran/31675] Fortran front-end and libgfortran should have a common header file

[fxcoudert at gcc dot gnu dot org]

-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |fxcoudert at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2007-04-23 22:40:05         |2007-08-22 14:40:49
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31675

[Bug fortran/31675] Fortran front-end and libgfortran should have a common header file

[fxcoudert at gcc dot gnu dot org]

------- Comment #1 from fxcoudert at gcc dot gnu dot org  2007-09-03 16:44 -------
Subject: Bug 31675

Author: fxcoudert
Date: Mon Sep  3 16:44:15 2007
New Revision: 128050

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=128050
Log:
        PR fortran/31675

        * libgfortran.h: New file.
        * iso-fortran-env.def: Use macros in the new header instead of
        hardcoded integer constants.
        * Make-lang.in (F95_PARSER_OBJS, GFORTRAN_TRANS_DEPS): Add
        fortran/libgfortran.h.
        * gfortran.h (GFC_STD_*, GFC_FPE_*, options_convert,
        ioerror_codes): Remove.
        * trans.c (ERROR_ALLOCATION): Remove.
        (gfc_call_malloc, gfc_allocate_with_status,
        gfc_allocate_array_with_status): Use LIBERROR_ALLOCATION.
        * trans-types.h (GFC_DTYPE_*): Remove.
        * trans-decl.c (gfc_generate_function_code): Use
        GFC_CONVERT_NATIVE instead of CONVERT_NATIVE.
        * trans-io.c (set_parameter_value, set_parameter_ref): Use
        LIBERROR_* macros instead of IOERROR_ macros.
        * trans-intrinsic.c (gfc_conv_intrinsic_function): Use
        LIBERROR_END and LIBERROR_EOR instead of hardcoded constants.
        * options.c (gfc_init_options): Use GFC_CONVERT_NATIVE instead of
        CONVERT_NATIVE.
        (gfc_handle_option): Use GFC_CONVERT_* macros instead of CONVERT_*.

        * libgfortran.h: Include gcc/fortran/libgfortran.h.
        Remove M_PI, GFC_MAX_DIMENSIONS, GFC_DTYPE_*, GFC_NUM_RANK_BITS,
        error_codes, GFC_STD_*, GFC_FPE_* and unit_convert.
        * runtime/environ.c (variable_table): Use GFC_*_UNIT_NUMBER instead
        of hardcoded constants.
        (do_parse, init_unformatted): Use GFC_CONVERT_* macros instead of
        CONVERT_*.
        * runtime/string.c (find_option): Use LIBERROR_BAD_OPTION instead
        of ERROR_BAD_OPTION.
        * runtime/error.c (translate_error, generate_error): Use
        LIBERROR_* macros instead of ERROR_*.
        * io/file_pos.c (formatted_backspace, unformatted_backspace,
        st_backspace, st_rewind, st_flush): Rename macros.
        * io/open.c (convert_opt, edit_modes, new_unit, already_open,
        st_open): Likewise.
        * io/close.c (st_close): Likewise.
        * io/list_read.c (next_char, convert_integer, parse_repeat,
        read_logical, read_integer, read_character, parse_real,
        check_type, list_formatted_read_scalar, namelist_read,
        nml_err_ret): Likewise.
        * io/read.c (convert_real, read_l, read_decimal, read_radix,
        read_f): Likewise.
        * io/inquire.c (inquire_via_unit): Likewise.
        * io/unit.c (get_internal_unit): Likewise.
        * io/transfer.c (read_sf, read_block, read_block_direct,
        write_block, write_buf, unformatted_read, unformatted_write,
        formatted_transfer_scalar, us_read, us_write, data_transfer_init,
        skip_record, next_record_r, write_us_marker, next_record_w_unf,
        next_record_w, finalize_transfer, st_read, st_write_done):
        Likewise.
        * io/format.c (format_error): Likewise.

Added:
    trunk/gcc/fortran/libgfortran.h
Modified:
    trunk/gcc/fortran/ChangeLog
    trunk/gcc/fortran/Make-lang.in
    trunk/gcc/fortran/gfortran.h
    trunk/gcc/fortran/iso-fortran-env.def
    trunk/gcc/fortran/options.c
    trunk/gcc/fortran/trans-decl.c
    trunk/gcc/fortran/trans-intrinsic.c
    trunk/gcc/fortran/trans-io.c
    trunk/gcc/fortran/trans-types.h
    trunk/gcc/fortran/trans.c
    trunk/libgfortran/ChangeLog
    trunk/libgfortran/io/close.c
    trunk/libgfortran/io/file_pos.c
    trunk/libgfortran/io/format.c
    trunk/libgfortran/io/inquire.c
    trunk/libgfortran/io/list_read.c
    trunk/libgfortran/io/open.c
    trunk/libgfortran/io/read.c
    trunk/libgfortran/io/transfer.c
    trunk/libgfortran/io/unit.c
    trunk/libgfortran/libgfortran.h
    trunk/libgfortran/runtime/environ.c
    trunk/libgfortran/runtime/error.c
    trunk/libgfortran/runtime/string.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31675

[Bug fortran/31675] Fortran front-end and libgfortran should have a common header file

[fxcoudert at gcc dot gnu dot org]

------- Comment #2 from fxcoudert at gcc dot gnu dot org  2007-09-03 16:47 -------
Fixed.


-- 

fxcoudert at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|---                         |4.3.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=31675