Hey Zhimin,

2008/8/27 zhimin song <[EMAIL PROTECTED]>:
>          1) when using the examples/form/Form.C ,
> "edit->label()->decorationStyle().setForegroundColor(WColor());", the
> valgrind told
>
> ==8731== Conditional jump or move depends on uninitialised value(s)
> ==8731==    at 0x4117E3D: Wt::WColor::operator==(Wt::WColor const&) const
> (WColor.C:71)
> ==8731==    by 0x4117EA3: Wt::WColor::operator!=(Wt::WColor const&) const
> (WColor.C:76)
> ==8731==    by 0x41225F7:
> Wt::WCssDecorationStyle::setForegroundColor(Wt::WColor)
> (WCssDecorationStyle.C:96)
> ==8731==    by 0x81C9558: PApply::validate() (in /var/www/cx/cx.wt)

Oops, we will fix this.

>       2) when using "pf_data = new WStandardItemModel(0, 8);", Valgrind told
> there is a memory leak, also, I could not confirm where the leak happened,
> but when modify it into "pf_data = new WStandardItemModel(0, 8, this);",
> Valgrind told no leak. However, the application was always could run ok.

Perhaps you did not delete 'pf_data' after use? You need to do this
unless you pass a parent WObject pointer, in which case it will
automatically be deleted together with the parent.

>       3) Suggestion: I suggest to add some notation in the wt-configure.xml
> about the way to debug, especially users can write
> "<valgrind-path>/usr/bin/valgrind --leak-check=full</valgrind-path>", but
> not only "<valgrind-path>/usr/bin/valgrind</valgrind-path>"

Good point. Too bad the variable is also slighly misnamed. Initially,
you could only pass a path, but now we support a full command line.

Thanks !

Regards,
koen

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
witty-interest mailing list
witty-interest@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/witty-interest

Reply via email to