I agree, I'd say call the class <tt><tt>AssetMetaData, </tt></tt>but just call the property (in AssetBase) MetaData.<br><br><br><b><i>Sean Dague <sdague@gmail.com></i></b> wrote:<blockquote class="replbq" style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;"> Mike Mazur wrote:<br>> Hi,<br>> <br>> Thanks for the feedback...<br>> <br>> On Thu, 29 Jan 2009 15:15:38 +0000<br>> Justin Clark-Casey <jjustincc@googlemail.com> wrote:<br>> <br>>> Mike Mazur wrote:<br>>>> So instead of having:<br>>>><br>>>>   AssetBase ab = new AssetBase();<br>>>>   ab.Name = "some name";<br>>>>   ab.Description = "some description";<br>>>><br>>>> I would write:<br>>>><br>>>>   AssetBase ab = new AssetBase();<br>>>>   ab.Metadata.Name = "some name";<br>>>>   ab.Metadata.Description = "some description";<br>>>><br>>> Just to
 clarify, you're anticipating changing the<br>>> OpenSim.Framework.AssetBase class (instead of having a parallel class<br>>> in the AssetServer code)?  If so, sounds good to me.<br>> <br>> Yes, I'd like to modify OpenSim.Framework.AssetBase.<br>> <br>>>> Should the Metadata class be renamed to AssetMetadata? Any other<br>>>> thoughts?<br>>>  From a readability perspective I would rather see it remain as just<br>>> Metadata, since something like<br>>><br>>> AssetBase myAsset = new AssetBase();<br>>> myAsset.AssetMetadata.Name = "Heloooo";<br>>><br>>> looks kind of redundant to me (though this may just be a taste issue).<br>> <br>> Yeah, that seems redundant, although since the class lives in<br>> OpenSim.Framework, and not something more specific (like<br>> OpenSim.Framework.Assets, for example), the OpenSim.Framework.Metadata<br>> class would be pretty ambiguous; what is the
 Metadata for?<br>> <br>> For the time being I'll go ahead with OpenSim.Framework.Metadata.<br><br>It's fine for the object to be called AssetMetaData, just don't make the<br>property that.<br><br>We don't have UserProfile.StringFirstname.  The type is encoded already<br>in the object, don't encode it also in the name.<br><br> -Sean<br><br>-- <br>Sean Dague / Neas Bade<br>sdague@gmail.com<br>http://dague.net<br><br><br>_______________________________________________<br>Opensim-dev mailing list<br>Opensim-dev@lists.berlios.de<br>https://lists.berlios.de/mailman/listinfo/opensim-dev<br></jjustincc@googlemail.com></blockquote><br><p> 



      <p>