Rick, You had two commits you made with respect to this thread on the list.
svn revision: 5103 svn revision: 5159 The first, we said it was pretty debatable that it was a bug, probably not a bug per se. The second we called an enhancement. Would you consider including these in the 4.0.1 release? The reason being, I'd like to be able to provide an ooDialog build from trunk that users could drop into their 4.0.1 install to help with testing, documenting, and finding design flaws. It would need those two commits in 4.0.1 to work. I know, I'm still probably naive to think users will jump in and help, <grin>, but they might. I'm all for keeping the release strictly a bug fix release, so I won't push for it if others are against it. But, I don't think there is much risk involved, i.e., it introduces a new bug that we don't catch until after the release. -- Mark Miesfeld On Mon, Aug 24, 2009 at 3:17 PM, Rick McGuire <object.r...@gmail.com> wrote: > Ok, the changes I just committed should fix your problem. The API > problem will be addressed later. > > Rick > > On Mon, Aug 24, 2009 at 7:06 PM, Mark Miesfeld<miesf...@gmail.com> wrote: >> On Mon, Aug 24, 2009 at 3:52 PM, Rick McGuire<object.r...@gmail.com> wrote: >> >>> The code for CSELF searches through all of the scopes of the objects >>> in inheritance order looking for the first occurrence of a CSELF >>> variable in the object, and that occurrence is the value returned for >>> all CSELF references. This is essentially first one found wins, but >>> it is the mechanism that allows subclasses to access the CSELF value >>> defined by the base class. >> >> Okay, that is pretty much what I understood. >> >> So in this definition >> >> ::class 'WindowBase' public mixinclass object >> ::method init_windowBase external "LIBRARY oodialog wb_init_windowBase" >> ::attribute hwnd get external "LIBRARY oodialog wb_getHwnd" >> >> ::class 'DialogControl' public inherit WindowBase WindowExtensions >> >> I expected that a CSELF in a DialogControl method would find the CSELF >> I set in WindowBase, as long as I did not set a CSELF in >> DialogControl. Which is what we want, and does work that way. >> >>> I suspect this really should have been defined by starting with the >>> scope of the current method and searching back along that inheritance >>> chain instead. I would hesitate to call this a bug, but it is >>> something we could add to trunk for the next release. It really is >>> bad for a mixin class to see a CSELF value for anything other than >>> scopes along its own inheritance chain. >> >> Now, if I add a CSELF to DialogControl, I expected my WindowBase code >> to still pick up the WindowBase CSELF and the DialogControl code to >> pick up the DialogControl CSELF. >> >> We don't have to call it a bug, <grin>, but it does seem like a good >> thing to change in trunk for the next release. >> >> -- >> Mark Miesfeld >> >> ------------------------------------------------------------------------------ >> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day >> trial. Simplify your report design, integration and deployment - and focus on >> what you do best, core application coding. Discover what's new with >> Crystal Reports now. http://p.sf.net/sfu/bobj-july >> _______________________________________________ >> Oorexx-devel mailing list >> Oorexx-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/oorexx-devel >> > > ------------------------------------------------------------------------------ > Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day > trial. Simplify your report design, integration and deployment - and focus on > what you do best, core application coding. Discover what's new with > Crystal Reports now. http://p.sf.net/sfu/bobj-july > _______________________________________________ > Oorexx-devel mailing list > Oorexx-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/oorexx-devel > ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev _______________________________________________ Oorexx-devel mailing list Oorexx-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/oorexx-devel
