Not approved. Please say: (x is Function) ? Debug.methodName(obj, x) : x
so you don't make a pointless call to methodName for things that can't possibly be methods. It will be safer too. Probably we should revise deprecated to only take names, since arguments.callee is going to go away in ES5... On 2011-01-09, at 14:02, Max Carlson wrote: > Change maxcarlson-20110109-AL2 by maxcarl...@friendly on 2011-01-09 10:22:17 > PST > in /Users/maxcarlson/openlaszlo/trunk-clean > for http://svn.openlaszlo.org/openlaszlo/trunk > > Summary: Improve deprecation warnings for inputtext.getText() > > Bugs Fixed: LPP-9625 - Remove @deprecated methods and properties from the LFC > > Technical Reviewer: ptw > QA Reviewer: hminsky > > Details: LzInputText - Pass 'text' as the property name to Debug.deprecated(). > > LzBootstrapDebugService - Use literal values when Debug.methodName() fails. > > Tests: inputtext.getText() warns to use 'foo.text'. > > Files: > M WEB-INF/lps/lfc/views/LzInputText.lzs > M WEB-INF/lps/lfc/compiler/LzBootstrapDebugService.lzs > > Changeset: > http://svn.openlaszlo.org/openlaszlo/patches/maxcarlson-20110109-AL2.tar
