[Edbrowse-dev] many topics

[Karl Dahlke <eklhad@comcast.net>]

1. My latest push fixes the form reset bug, which I wrote about last time.

2. JSAutoCompartment, I almost understand what it does,
but still don't know why it is needed.
Look at the function JS_DefineProperty in the api, for example.
It's first argument is the context.
And all the functions seem to be like that.
They were before anywaays.
Every time you tap into the javascript machinery
you are telling it explicitly the context to use.
It is the first argument to everything.
Why then would it have to be set,
or assigned a compartment, or any such thing?
If a context were assigned in some global fashion,
then we wouldn't have to pass it as an argument to every js function
now would we?
There's something inconsistent about this design.

3. I think things are somewhat stable, and would like to reindent using
the kernel formatting.
That is a global change.
Are you working on something at present, or is the code quiescent?
Adam I know you said you wanted to make some changes to url.c
for compiler warnings.
Is this in process or on hold,
whence I could make a global change?

4. I think I know how to restructure the file map,
so I don't have marching index numbers that eventually run out,
unless I write another gc module, which I don't want to do.
There will probably still be a limit on files you can edit,
but larger, and perhaps architecture dependent, 4 byte pointers and indexes etc,
and in any case better than where we're at today.
So I may do that after the re-indent.

5. After all these changes have been made,
maybe we all play with it for a month or so to look for more bugs,
then cut a new version.


Karl Dahlke