Bug 63515 - AtomicString::show() and WTFString::show() should be const.
Summary: AtomicString::show() and WTFString::show() should be const.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Text (show other bugs)
Version: 528+ (Nightly build)
Hardware: All Unspecified
: P2 Minor
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-28 00:03 PDT by Shinya Kawanaka
Modified: 2011-06-28 01:54 PDT (History)
2 users (show)

See Also:

Attachments
Patch (2.80 KB, patch)
2011-06-28 00:11 PDT, Shinya Kawanaka 		no flags Details | Formatted Diff | Diff
Patch (2.90 KB, patch)
2011-06-28 00:48 PDT, Shinya Kawanaka 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Shinya Kawanaka 2011-06-28 00:03:35 PDT 
AtomicString::show() and WTFString::show are not const, so we have to use const_cast to call them occasionally.
These methods should be const.
Comment 1 Shinya Kawanaka 2011-06-28 00:11:35 PDT 
Created attachment 98870 [details]
Patch
Comment 2 Kent Tamura 2011-06-28 00:14:58 PDT 
(In reply to comment #0)
> AtomicString::show() and WTFString::show are not const, so we have to use const_cast to call them occasionally.

So you should remove such const_casts in this patch. Otherwise, the patch has no benefit.
Comment 3 Shinya Kawanaka 2011-06-28 00:37:43 PDT 
Since this is a debug feature, there is no code to call show() in Sources (maybe). However, when doing something like printf-debug, the lack of const is painful.

This bug is related to 
https://bugs.webkit.org/show_bug.cgi?id=61149
Comment 4 Kent Tamura 2011-06-28 00:40:20 PDT 
(In reply to comment #3)
> Since this is a debug feature, there is no code to call show() in Sources (maybe). However, when doing something like printf-debug, the lack of const is painful.

Ok, I understand.  Please note such reasons to ChangeLog.
Comment 5 Shinya Kawanaka 2011-06-28 00:48:31 PDT 
Created attachment 98875 [details]
Patch
Comment 6 Kent Tamura 2011-06-28 00:50:10 PDT 
Comment on attachment 98875 [details]
Patch

ok.
Comment 7 WebKit Review Bot 2011-06-28 01:54:11 PDT 
Comment on attachment 98875 [details]
Patch

Clearing flags on attachment: 98875

Committed r89906: <http://trac.webkit.org/changeset/89906>
Comment 8 WebKit Review Bot 2011-06-28 01:54:16 PDT 
All reviewed patches have been landed.  Closing bug.