[Patch, fortran] Cosmetics: Dup. code removal, indent fix, typo fix.

Andre Vehreschild vehre@gmx.de
Fri Jan 30 12:10:00 GMT 2015


Hi all,

I fear this fix is not so obvious in one location, I therefore ask for a
review. The attached patch fixes:

- a duplicate code fragment (possibly due to merged twice),
- the indentation in the trans-expr.c block (in my first patch), and
- a typo on the datatype-size to create for the charlen.

The length of a char-array is stored as a 4-byte BT_INTEGER. Due to a typo a
1-byte BT_INTEGER was requested. The patch fixes this. I know this patch mixes
several trivial issues. Should I do separate patches for each of them, or what
is the most desirable method?

Bootstraps and regtests ok on x86_64-linux-gnu/FC20.

Regards,
	Andre
-- 
Andre Vehreschild * Email: vehre ad gmx dot de 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cosmetics_2.clog
Type: application/octet-stream
Size: 249 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/fortran/attachments/20150130/57051aa3/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cosmetics_2.patch
Type: text/x-patch
Size: 2561 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/fortran/attachments/20150130/57051aa3/attachment.bin>


More information about the Fortran mailing list