dperez schrieb:
> Hi,
> 
> In order to facilitate debugging support, when the --add-new-lines switch is
> given, I would place a new line after a "{" and not e.g. after a ",".  In
> this way I could place breakpoints more precisely.  The rule would be a
> statement per line.
> 
> Example snippet:
> 
> qx.Class.define("qx.ui.toolbar.CheckBox",
> {extend:qx.ui.toolbar.Button,
> construct:function(vText,
> vIcon,
> vChecked){this.base(arguments,
> vText,
> vIcon);
> if(vChecked!=null){this.setChecked(vChecked);
> }},
> 
> 
> I would like something similar to:
> 
> qx.Class.define("qx.ui.toolbar.CheckBox",
> {extend:qx.ui.toolbar.Button,
> construct:function(vText,vIcon,vChecked){
> this.base(arguments,vText,vIcon);
> if(vChecked!=null){
> this.setChecked(vChecked);
> }},
> 
> 
> For example with the current version, I cannot place a breakpoint only when
> the "if" statement is true.
> 
> What do you think?
> I know I can work with source version, but for investigating puntual
> problems is better to debug with the compiled version (loads much quicker).

I think this should work. We need to be sure however that after the 
change each function declaration is on a separate line, like before. 
Otherwise Firebugs profiler, which use the line feeds to identify a 
function IMHO, does not work anymore.

Can you create a patch and test it?

Sebastian


> 
> Regards,
> David


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to