WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
27053
Ambiguity in LabelScope initialization
https://bugs.webkit.org/show_bug.cgi?id=27053
Summary
Ambiguity in LabelScope initialization
Norbert Leser
Reported
2009-07-07 18:52:25 PDT
Created
attachment 32413
[details]
Code patch for BytecodeGenerator.cpp Codewarrior compiler (WINSCW) latest b482 on Symbian cannot resolve type of "0" unambiguously. Set expression explicitly to PassRefPtr<Label>::PassRefPtr(). Rather than implementing conditional block for COMPILER(WINSCW) it appears to be save and appropriate to modify the main-line code accordingly.
Attachments
Code patch for BytecodeGenerator.cpp
(1.42 KB, patch)
2009-07-07 18:52 PDT
,
Norbert Leser
ggaren
: review+
Details
Formatted Diff
Diff
Updated patch for bug# 27053
(1.40 KB, patch)
2009-07-07 19:39 PDT
,
Norbert Leser
eric
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Geoffrey Garen
Comment 1
2009-07-07 19:09:16 PDT
Comment on
attachment 32413
[details]
Code patch for BytecodeGenerator.cpp r=me I think you can just write "PassRefPtr<Label>()" instead of "PassRefPtr<Label>::PassRefPtr()".
Norbert Leser
Comment 2
2009-07-07 19:39:23 PDT
Created
attachment 32424
[details]
Updated patch for
bug# 27053
I just verified Geoffrey's suggestion about simplifying the statement, and he's right. The updated patch contains the respective modification.
Eric Seidel (no email)
Comment 3
2009-07-10 23:14:20 PDT
Comment on
attachment 32424
[details]
Updated patch for
bug# 27053
OK. Your ChangeLog seems to have an extra return in it though.
Simon Hausmann
Comment 4
2009-07-13 01:20:40 PDT
Landed in
r45791
. The ChangeLog was made from the wrong directory, btw. I moved it up to JavaScriptCore/ChangeLog and fixed the path.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug