Ok, my last push changed over 400 lines of code.
Really?
Yes, and all bug fixes; some serious,
serious enough to cause a segfault.
What happened?
My first mistake was in jsrt.
I didn't include any tests for the URL class.
That class is very special and very complicated, over a thousand lines of code.
With no regression tests, bugs could creep in and nobody would know.
And they did.
My latest push adds tests for this class, so this won't happen again.
See jsrt line 536.
I'm not going to describe all the bugs, but here is a funny one.
We had, for a long time, code like this:
if(condition)
do this
else
do that
But no error legs in js was leading to nulls and segfaults,
so we put in error legs all over the place.
This we needed to do, and thanks to everyone for your help.
There were so many, the process was rather mechanical,
and understandably so.
Just get er done.
Thus the above code became:
if(condition)
if(do this and it fails) {
js fail
return
}
else
if(do that and it fails) {
js fail
return
}
If you've studied formal language theory and compilers,
then you know this is the region of the C grammar that is ambiguous.
But the compiler doesn't tell you that,
it just picks a derivation and marches on.
Which one?
Not the one we expect.
I changed the code as below and several bugs went away.
if(condition) {
if(do this and it fails) {
js fail
return
}
} else {
if(do that and it fails) {
js fail
return
}
}
There were quite a few blocks like this.
So there you go - bitten by the if else ambiguity of C.
There were other bugs too, dealing with string allocation
and a malformed url constructor.
I think they are fixed now; jsrt passes anyways.
I'm glad I cleaned all this up before release 3.5.1.
Karl Dahlke
_______________________________________________
Edbrowse-dev mailing list
[email protected]
http://lists.the-brannons.com/mailman/listinfo/edbrowse-dev
