Hi Fritz,

On Wednesday 16 December 2009 Fritz Zaucker wrote:
> I just rebuilt one of our applications with the latest trunk and found no
> problems, neither in the build process nor at run-time. Obviously, this is
> not a rigerous test, but at least it keeps me confident in running the
> latest and greatest Qooxdoo ...
Thanks for the testing :)

> One comment:
> 
> I am getting the following deprication message (for a while now and finally
> bothered to track it down (it's in the Smart.js file of the smart table
> model contrib):
> 
>    012123 The constant 'EVENT_TYPE_DATA_CHANGED' is deprecated: New code
>           should not use this.  Instead, use the text string literally.
> 
> Looking it up in the API viewer I "guessed" that I should use the string
> "dataChanged" instead (my first guess was to use
>  "EVENT_TYPE_DATA_CHANGED"). So perhaps the deprecation message could be
>  improved a bit:
> 
>         The constant 'EVENT_TYPE_DATA_CHANGED' is deprecated: New code
>          should not use this.  Instead, use the text string 'dataChanged'
>          literally.
> 
> Just a minor point, but it might save some people a couple of thoughts.
I just comitted this change to the framework. Thanks for watching out!

> P.S.: What's the idea behind using literal strings instead of constants,
>        btw?
We want to reduce/remove all constants in the framework. They do not have any 
benefit. Additionally you can't recognize automatically any typos which may 
lead to errors in your application.

cheers,
  Alex

------------------------------------------------------------------------------
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to