Re: [Fwd: [Crystal-cvs] SF.net SVN: crystal:[38136] CS/branches/soc2012/editor]
"Christian Van Brussel" <[email protected]>
| Newsgroups | gmane.comp.graphics.crystalspace.devel |
|---|---|
| Message-ID | <[email protected]> |
> Should I also remove the redundant > Graph-related classes that litter the source code right now? Like > GraphLink, Graph, GraphNode, MyGraphNode1, etc.? Yes, these classes and names are there for historical reasons, although they shouldn't appear in that test program. They have been copied from the graph editor tool (from btgraphtest/graphedit), although they should have been kept there. It would be much more cleaner to remove then from varedittest, clearly. > Shouldn't > iModifiableDescription provide a way to get a name + textual description > for the described object? Yes, the iModifiableDescription should also have a name + description, but that's actually the iModifiableParameter that should loose its methods about that. The reason for this is that no text at all should actually be stored in the iModifiableDescription and iModifiableParameter, because of translation issues. All text should therefore be represented instead by an ID that has to be used with the iTranslator class in order to find the text to be displayed. And such ID can be built automatically from the ID of either the iModifiableDescription or the iModifiableParameter, therefore we don't need any explicit methods within those interfaces in order to get access to their names and description. Instead, we need to define the rule to be applied for the construction of the ID's. ------------------------------------------------------------------------------ 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/