Patch for JS debugger

Harri Porten porten at trolltech.com
Sun Jun 2 14:02:19 BST 2002


On Sat, 1 Jun 2002, Koos Vriezen wrote:

> This is my second patch for the JS debugger in khtml.
> To enable the debugger 'EnableJavaScriptDebug=true' should be set in
> ~/.kde/share/config/konquerorrc under '[Java/JavaScript Settings]'.

Please commit. If it works I can't comment much on it as I didn't inspect
it anymore after Peter continued what I started ages ago.

I have one issue though: my original design goal was to let it be not
compiled be default as I expect some slowdown that the average user won't
like. That would mean not activating the KJS_DEBUGGER macro (your patch)
and ifdef'ing the debugger related code in libkjs code as well. Hmmm.
Compiling it into the ecma/* part probably doesn't make a speed
difference.

What do you think ? I'd rather see khtml developers compile with a switch
or provide a libkjsd binary if end users are interested in the debugger.

Harri.





More information about the kfm-devel mailing list