Metacomment: If you update your OpenLaszlo/tools/ and reload tools/ trunk/svn/svn-bash.sh you will get the new patch-review tools which will include added files and will check the changepackage into svn rather than cluttering email. This is 'the new way'.

Comment: In the 'new debugger regime' the Debug message that I suggested to you will be uninformative because these functions are all anonymous functions (it would work fine if they were class methods, but Max does not want to use class definitions in the kernel). You can fix this by either giving the functions informative names, or putting the name literally in the error message. It would be great if you could fix the other places you have already used this idiom.

Otherwise, approved.

On 2006-10-25, at 16:43 EDT, Philip Romanik wrote:

(Note. This is a new file and the script wouldn't include it in the checkin. In addition to this change, is the mod to Library.lzs in the services directory)

Change 20061025-Philip-0 by [EMAIL PROTECTED] on 2006-10-25 16:01:39 EDT in /cygdrive/f/laszlo/svn/src/svn/openlaszlo/branches/legals/ WEB-INF/lps/lfc

Summary: stubs for LzCursor.js

New Features:

Bugs Fixed:

Technical Reviewer: ptw
QA Reviewer: (pending)
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details:
LzCursor.js is missing from the lfc. This file is needed to make dhtml complaints go away.


Tests:

Files:
M      services/platform/dhtml/LzCursor.js
M      services/Library.lzs

<LzCursor.zip>

Reply via email to