Summary: | "A + B" with strings shouldn't copy if A or B is empty. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Kling <kling> | ||||
Component: | Web Template Framework | Assignee: | Andreas Kling <kling> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | benjamin, cmarcelo, commit-queue, keith_miller, kling | ||||
Priority: | P2 | Keywords: | Performance | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Andreas Kling
2015-10-12 10:45:59 PDT
Created attachment 262899 [details]
Patch
Comment on attachment 262899 [details] Patch Clearing flags on attachment: 262899 Committed r190882: <http://trac.webkit.org/changeset/190882> All reviewed patches have been landed. Closing bug. It looks like this patch is breaking the jsc command line REPL. If you try:
DYLD_FRAMEWORK_PATH=Safari-Gala-Production-SafariTen-190882-76008.app/Contents/Frameworks ./Safari-Gala-Production-SafariTen-190882-76008.app/Contents/Resources/jsc
>>> foo = 1
Exception: ReferenceError: Can't find variable: f
(In reply to comment #4) > It looks like this patch is breaking the jsc command line REPL. If you try: > > DYLD_FRAMEWORK_PATH=Safari-Gala-Production-SafariTen-190882-76008.app/ > Contents/Frameworks > ./Safari-Gala-Production-SafariTen-190882-76008.app/Contents/Resources/jsc > >>> foo = 1 > Exception: ReferenceError: Can't find variable: f Oh dang! I see what the bug is: StringTypeAdapter<const UChar*>::toString() always uses length=1 instead of the provided character count. Crazy that no test caught this. Will fix shortly. |