If I can suggest something, using a tool such as jslint on the code
would probably allow you to find all those missing semicolon errors,
as well as other valid but risky code syntax.
I ran a modified jslint (the one I use for my own code) on a few
firebug files and it returned very few warnings, meaning that you
write quite high-quality code. I think the most frequent one would be
the missing { } on one-liner "if" blocks, but jslint can be configured
to ignore them. By default jslint is quite pedantic but it's
configurable and can be modified to accomodate your needs, such as the
use of the "with", "get" or "const" statements.
Since I run jslint on my own js code, I get a lot less bug reports
caused by silly errors so I can focus on more serious ones...
On Mar 28, 12:48 pm, johnjbarton <[email protected]> wrote:
> Thanks colin. The problem here was a missing semicolon on a tracing
> statement that cause the detrace to continue to the semi on the var
> groups = []; statement, deleting it as well. (Detrace is the program
> that removes tracing statements, its kinda dumb). This will be fixed
> in 1.4a15 (as well as a few others I found with regexps).
>
> jjb
>
> On Mar 28, 8:29 am, colin <[email protected]> wrote:
>
> > On Mar 24, 4:29 pm, johnjbarton <[email protected]> wrote:
>
> > > Ok, but 1483 is fixed, so you probably have a different problem, one
> > > we don't have a test case for so we can't fix it.
>
> > I'm also seeing a problem with breakpoints not being triggered in FB
> > 1.4.0a14 (FFx 3.5b4pre nightly 20090327). I have no idea if it is
> > related to the other poster's issues, but here goes. Test case 1483
> > works for me, but I see this error in the console window:
>
> > groups is not defined
> > chrome://firebug/content/debugger.js
> > Line 2662
>
> > In diffing debugger.js from a13 and a14 (end-user versions) I notice
> > that refresh() has been rearranged. It seems that in that rearrange
> > the definition for the "groups" variable was moved/removed. In a13 it
> > is defined on line 2434 where it isn't defined in a14 (that I can see)
> > Copying line 2434 in a13 (var groups = [];) to line 2661 in a14
> > stopped this error and allows the breakpoints to work for me.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Firebug" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/firebug?hl=en
-~----------~----~----~----~------~----~------~--~---
