[Opensim-dev] [Opensim-commits] r8236 - in trunk/OpenSim: ApplicationPlugins/Rest/Inventory Data/MSSQL Data/MySQL Data/NHibernate Data/SQLite Data/Tests Framework Framework/AssetLoader/Filesystem Framework/Communications/Cache Framework/Communica

[Kyle]

Sorry everyone I replied to the wrong email doh, need coffee...

-----Original Message-----
From: opensim-dev-bounces at lists.berlios.de
[mailto:opensim-dev-bounces at lists.berlios.de] On Behalf Of Kyle
Sent: Saturday, February 07, 2009 9:12 AM
To: opensim-dev at lists.berlios.de
Subject: Re: [Opensim-dev] [Opensim-commits] r8236 - in trunk/OpenSim:
ApplicationPlugins/Rest/Inventory Data/MSSQL Data/MySQL Data/NHibernate
Data/SQLite Data/Tests Framework Framework/AssetLoader/Filesystem
Framework/Communications/Cache Framework/Communica

That last post scares me, is it on one side only?

-----Original Message-----
From: opensim-dev-bounces at lists.berlios.de
[mailto:opensim-dev-bounces at lists.berlios.de] On Behalf Of Mike Mazur
Sent: Saturday, February 07, 2009 8:33 AM
To: opensim-dev at lists.berlios.de
Subject: Re: [Opensim-dev] [Opensim-commits] r8236 - in trunk/OpenSim:
ApplicationPlugins/Rest/Inventory Data/MSSQL Data/MySQL Data/NHibernate
Data/SQLite Data/Tests Framework Framework/AssetLoader/Filesystem
Framework/Communications/Cache Framework/Communica

Hi,

On Sat, Feb 7, 2009 at 12:59 AM, Justin Clark-Casey
<jjustincc at googlemail.com> wrote:
> What would happen if some future plugin to OpenSim wanted to add extra
metadata fields?  Would then need to extend the
> AssetBase class?  This seems a little icky to me (as opposed to extending
a separate AssetMetadata class) but maybe
> that's just me.

Good question. I guess there's a benefit of having the Metadata class
separate from the Asset class hierarchy.

> Also to be penickity, could we prefix non-public fields with m_ rather
than just _ ?  I think this is the standard that
> we're following, though I have no desire to get religious about it.

I was just following the existing code in AssetBase. When I make the
next changes in there I'll prefix the attributes with m_.

Thanks,
Mike
_______________________________________________
Opensim-dev mailing list
Opensim-dev at lists.berlios.de
https://lists.berlios.de/mailman/listinfo/opensim-dev


_______________________________________________
Opensim-dev mailing list
Opensim-dev at lists.berlios.de
https://lists.berlios.de/mailman/listinfo/opensim-dev