Closed
Bug 719712
Opened 13 years ago
Closed 12 years ago
Firefox is isn't returning my original function code
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 761723
People
(Reporter: aeneasdardanus, Unassigned)
Details
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:9.0.1) Gecko/20100101 Firefox/9.0.1
Build ID: 20111220165912
Steps to reproduce:
wrote a function
and tried to return its source for display purposes
Actual results:
unexpected strange thing happened! It returned a redacted, "censored", and to somebody else taste a "beautified" internal superficial version of my code.
Expected results:
It should have returned the original source code, especially my comments.
My /* comments */ are missing!
And there are wrong unnecessary semicolon insertions.
Comment 1•13 years ago
|
||
we need steps to reproduce`and a testcase.
Your report is like "I did something and something went wrong" and that is not enough for a useful bug report
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Priority: P5 → --
Resolution: --- → INCOMPLETE
Hello Matt
Thanks for your reply.
This bug is not procedure dependent.
It is returning an internally edited/redacted; an incomplete superficial code, instead of the expected coders original/authentic source.
Anyway, here is a literal testcase example which I believe is sufficiently explanatory just by reading (since I'm not able to find a way editing one form this page):
The code:
----------------------------8<-----------------------
function myEmpty(){/*This is an empty function object*/ };
console.log(myEmpty);
//alert(myEmpty);
---------------------------->8-----------------------
THE RESULT:
-----------
LOG: function myEmpty() {} //FX [current version]
THE EXPECTED RESULT:
------------------
LOG: function myEmpty(){/*This is an empty function object*/ } IE
LOG: function myEmpty(){/*This is an empty function object*/ } Ch
LOG: function myEmpty(){/*This is an empty function object*/ } Op
LOG: function myEmpty(){/*This is an empty function object*/ } Sa
Hope this helps a little.
Regards
Status: RESOLVED → UNCONFIRMED
Priority: -- → P5
Resolution: INCOMPLETE → ---
Updated•13 years ago
|
Severity: major → normal
Priority: P5 → --
Comment 3•13 years ago
|
||
Is this DOM, JS or Error console ?
Component: Untriaged → DOM
Product: Firefox → Core
QA Contact: untriaged → general
Component: DOM → Developer Tools
Product: Core → Firefox
QA Contact: general → developer.tools
Comment 4•13 years ago
|
||
Actually, this is JS engine. And a duplicate. We don't store the original source; we end up serializing the bytecode.
Assignee: nobody → general
Component: Developer Tools → JavaScript Engine
Product: Firefox → Core
QA Contact: developer.tools → general
Whiteboard: DUPEME
Comment 5•13 years ago
|
||
ECMA-262 says:
> 15.3.4.2 Function.prototype.toString()
>
> An implementation-dependent representation of the function is returned.
> This representation has the syntax of a FunctionDeclaration. Note in
> particular that the use and placement of white space, line terminators,
> and semicolons within the representation String is implementation-dependent.
So technically this is not a bug. But we are considering switching to storing the original source and returning that in the medium future.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment 6•13 years ago
|
||
http://pokefarm.org/forum_thread?tid=96850 is another report about not saving the original, developer-provided source but rather decompiling from bytecode.
Comment 7•13 years ago
|
||
> But we are considering switching to
> storing the original source and returning that in the medium future.
Bug 718969 is the major part of this.
Comment 8•12 years ago
|
||
I would like to bump this up because its quite impossible to write code with annotations to be processed in runtime.
Hope this will be fixed as soon as possible.
I would like to report this [Bug 719712] as Resolved {(at least) since FX v.21.0 release}.
Comment 10•12 years ago
|
||
Yes, this has been fixed by removing the decompiler and storing function source code text instead.
Status: NEW → RESOLVED
Closed: 13 years ago → 12 years ago
OS: Windows 7 → All
Hardware: x86_64 → All
Resolution: --- → FIXED
Updated•11 years ago
|
Resolution: FIXED → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•