Bug 733265

Summary: Decimal float constant suffixes are incorrectly separated
Product: Red Hat Enterprise Linux 6 Reporter: Petr Pisar <ppisar>
Component: indentAssignee: Petr Pisar <ppisar>
Status: CLOSED ERRATA QA Contact: qe-baseos-tools-bugs
Severity: high Docs Contact:
Priority: high    
Version: 6.2CC: aldyh, mnowak, pmuller, ppisar
Target Milestone: rcKeywords: Patch
Target Release: ---   
Hardware: x86_64   
OS: Linux   
URL: https://lists.gnu.org/archive/html/bug-indent/2011-08/msg00000.html
Whiteboard:
Fixed In Version: indent-2.2.10-6.el6 Doc Type: Bug Fix
Doc Text:
Cause Running indent on code with decimal float constants (e.g. `double f = 3.0DF;'). Consequence The suffix become separated from the number by a space resulting to invalid C code. Fix indent tool has been modified to understand decimal float suffixes as proposed by N1312 draft of ISO/IEC WDTR24732. Result Indent does not break code with decimal float constants anymore.
 Story Points: ---
Clone Of: 733051 Environment:
Last Closed: 2012-06-20 12:00:29 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Proposed fix
none
Fix ported back to 2.2.10 none

Description Petr Pisar 2011-08-25 10:00:32 UTC 
+++ This bug was initially created as a clone of Bug #733051 +++

Description of problem:
If you have a constant suffix, indent will put a space between the number and the suffix, causing a compilation syntax error.

Example:
double f = 3.0DF;


Version-Release number of selected component (if applicable):
indent-2.2.11-1.fc14.x86_64

How reproducible:

Indent the following, and notice the extra space inserted:

double f = 3.0DF;

 
Actual results:

double f = 3.0 DF;

Expected results:

double f = 3.0DF;

--- Additional comment from ppisar on 2011-08-25 08:21:52 GMT ---

The "DF" suffix is non-standard extension supported by GCC or XLC to utilize decimal float representations on some PowerPC architectures.

--- Additional comment from ppisar on 2011-08-25 08:25:16 GMT ---

Known decimal float suffixes are "DF", "DD", "DL" and their lower-cased variants.
------

RHEL-6 affected too.
Comment 1 Petr Pisar 2011-08-25 10:02:02 UTC 
Created attachment 519800 [details]
Proposed fix
Comment 4 Petr Pisar 2012-01-04 13:03:03 UTC 
Created attachment 550680 [details]
Fix ported back to 2.2.10
Comment 6 Petr Pisar 2012-01-04 13:28:23 UTC 
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
Cause
    Running indent on code with decimal float constants (e.g.
    `double f = 3.0DF;').
Consequence
    The suffix become separated from the number by a space
    resulting to invalid C code.
Fix
    indent tool has been modified to understand decimal float
    suffixes as proposed by N1312 draft of ISO/IEC WDTR24732.
Result
    Indent does not break code with decimal float constants
    anymore.
Comment 10 errata-xmlrpc 2012-06-20 12:00:29 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2012-0753.html