On Fri, Jul 20, 2012 at 5:58 AM, Martin Wittemann <[email protected]
> wrote:
> *qx.lang.Array*
> toArray: can be replaced with cast
>
I have only rare need for toArray, but I know it's there and can find it
when necessary. The problem with replacing it with cast is that cast is an
uncommonly-used function that is easily forgotten. Removing toArray would
be reasonable, but I'd request that a comment be left in place showing the
use of cast in its stead.
clone: its a simple concat
>
Yes, it's a simple concat, but makes the code much more clear as to the
meaning: I want to make an identical copy of this thing. I would really
like to see this one remain available as qx.lang.Array.clone.
> *qx.lang.String*
> contains: is a simple indexOf
>
This is another case like clone. A boolean contains() method makes the code
much cleaner than testing the return value of indexOf, and this is a very
common operation. I'd like to see this one remain as
qx.lang.String.contains.
*qx.lang.Function*
> Here, we move / remove the following functions:
> bind: will be available on every function as polyfill
>
I use qx.lang.Function.bind() all over the place. It will be a real pain to
migrate these, but I don't see any readability problem caused by removing
it, as would be the case with qx.lang.Array.toArray() and
qx.lang.String.contains().
Now its your turn! Do you have any objections on any of the planed
> removals? Any hints of what we should / could add as polyfill?
>
Other than those I mentioned above, the other proposed changes seem very
reasonable.
Thanks!
Derrell
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
qooxdoo-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
