Hi, On Fri, 2007-11-09 at 10:48 -0600, [EMAIL PROTECTED] wrote: > Author: cneumann > Date: Fri Nov 9 10:48:43 2007 > New Revision: 1012 > Trac changeset: http://opensg.vrsource.org/trac/changeset/1012 > ViewVC: http://realityforge.vrsource.org/viewvc/opensg?view=rev&revision=1012 > > Added: > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditMFCPointerFieldHandle.h > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditMFCPointerFieldHandle.inl > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditMParentFCPointerFieldHandle.h > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditMParentFCPointerFieldHandle.inl > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditMPointerFieldBaseHandle.h > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditMPointerFieldBaseHandle.inl > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditSFCPointerFieldHandle.h > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditSFCPointerFieldHandle.inl > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditSParentFCPointerFieldHandle.h > > branches/Carsten_PtrWork2/Source/System/FieldContainer/Fields/OSGEditSParentFCPointerFieldHandle.inl
just one clarification, as I saw these again. *Handle classes are not there and were never meant to change the field directly. They either should allow you access to the field (get as it was, edit for value fields) or call back into the container the field was coming from (edit for pointer fields). All the are allowed to do is the initial parameter check. Adding a second place that can modify fields is IMHO not a good idea, because you would have to introduce special cases every time it is not a simple set. And you would have to keep track that once you change the class functions you have to change/update the handle classes as well. kind regards, gerrit ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Opensg-core mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensg-core
