Issue 7045 - Float fraction forgotten in number to string conversion in rtl_ImplFloatToString() routine.
Summary: Float fraction forgotten in number to string conversion in rtl_ImplFloatToStr...
Status: CLOSED FIXED
Alias: None
Product: porting
Classification: Code
Component: code (show other issues)
Version: 641
Hardware: Mac Mac OS X, all
: P3 Trivial (vote)
Target Milestone: OOo 1.1 Beta
Assignee: Stephan Bergmann
QA Contact: issues@porting
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-08-16 17:15 UTC by rogper
Modified: 2002-12-04 14:47 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments

Note You need to log in before you can comment on or make changes to this issue.
Description rogper 2002-08-16 17:15:14 UTC
Probably a typo in $(SRCROOT)/sal/rtl/source/strimp.c

//Was: sal_uInt32 nFractBits	= nFractBits & FLOAT_FRACTMASK;
// Shouldn't it be:
sal_uInt32 nFractBits	= nFBits & FLOAT_FRACTMASK;
Comment 1 Martin Hollmichel 2002-10-02 20:19:01 UTC
Eike, can you please have a look on this ?
Comment 2 ooo 2002-10-30 00:47:34 UTC
This one skipped my attention being unconfirmed, adjusted my query..



Of course you're right, this is a bug. And it confirms that either no

one used that function to convert floats to strings so far, or wasn't

interested in the results.

Comment 3 ooo 2002-10-30 12:05:58 UTC
fixed in sal/rtl/source/strimp.c 1.4

However, not checked in for any other 
branch, someone familiar with the 
UDK should decide if and where the fix is needed. 
The function isn't
used in Office code but may be relevant in UDK context, if 
someone
really used floats..

Forwarding to JBU
Comment 4 joergbudi 2002-10-30 13:31:23 UTC
jbu->sb: Something for you, I think
Comment 5 Stephan Bergmann 2002-10-30 15:46:32 UTC
I'm planning to replace the rtl code to translate between [u]string
and float/double with code from tools/solmath (by moving that code to
sal) in the next days, which will obsolete this bug (and I will set
this bug to FIXED than).  But thanks anyway, Roger :)
Comment 6 Stephan Bergmann 2002-12-04 14:41:48 UTC
As already explained, setting this to FIXED now (as it has been
obsoleted by StarOffice-internal bugfix #103888#.
Comment 7 Stephan Bergmann 2002-12-04 14:47:20 UTC
closed