|
Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
| Summary: | ByteString::ToDouble non-existant, and warning-less | ||
|---|---|---|---|
| Product: | utilities | Reporter: | mmeeks <mmeeks> |
| Component: | code | Assignee: | Stephan Bergmann <stephan.bergmann.secondary> |
| Status: | CLOSED FIXED | QA Contact: | Unknown <non-migrated> |
| Severity: | Trivial | ||
| Priority: | P3 | CC: | issues, jens-heiner.rechtien |
| Version: | OOo 1.1 RC | ||
| Target Milestone: | OOo 2.0 | ||
| Hardware: | All | ||
| OS: | All | ||
| Issue Type: | DEFECT | Latest Confirmation in: | --- |
| Developer Difficulty: | --- | ||
@sb: Is there any reason why these methods do not call rtl_str_ToDouble and rtl_str_ToFloat ? The UniString methods do so. IMHO we should depredicate the ByteStrings at all. Michael, you have my full sympathy ... I can't believe that such stupid things survived for years in our code base :-( Heiner Either fix this or remove tools Uni/ByteString (in favor of rtl strings), whatever happens first... . I'll add an OSL_ASSERT, but no real fix, as tools strings are deprecated, anyway (see #i17875#). . . . |
The code says it all: float ByteString::ToFloat() const { DBG_CHKTHIS( ByteString, DbgCheckByteString ); return 0; } double ByteString::ToDouble() const { DBG_CHKTHIS( ByteString, DbgCheckByteString ); return 0; } Is this a feature ? - it wasted a load of my time, can't we just chain to the rtl:: versions ?