<html>
<head>
<style>
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Verdana
}
</style>
</head>
<body class='hmmessage'>
Hell, +1 on solving the issue at hand without going into long architectural discussion.<BR><BR>
Just chipping in. :D<BR>
<BR>Best regards,<BR>Stefan Andersson<BR>Tribal Media AB<BR><BR><BR><BR> <BR>> Date: Wed, 15 Apr 2009 00:37:58 +0200<BR>> From: melanie@t-data.com<BR>> To: opensim-dev@lists.berlios.de<BR>> Subject: Re: [Opensim-dev] Supplying IScene instead of Scene for the future region modules mechanism<BR>> <BR>> Hi,<BR>> <BR>> as a mid to long range goal, +1, actually.<BR>> <BR>> but in the short run, the ability to load and unload regions is <BR>> blocked by the existing module API, and to fix this basic piece of <BR>> functionality, they need to be migrated to the new API, asap.<BR>> <BR>> If this is dragged into a long architectural discussion, we won't <BR>> get region restarts for many more months.<BR>> <BR>> So, I'd rather see this iteration of the region module API pass <BR>> Scene, and remove the old API very soon, and then think about <BR>> architecting and refactoring when that is not a blocker to <BR>> adding/repairing basic functionality.<BR>> <BR>> Melanie<BR>> <BR>> <BR>> Stefan Andersson wrote:<BR>> > Um,<BR>> > <BR>> > <BR>> > <BR>> > I believe you're saying "supplying a smaller subset of the functionalities of Scene", as being able to supply something else than a concrete implementation should never really be a problem - in fact, in most cases supplying an interface is more desirable.<BR>> > <BR>> > <BR>> > <BR>> > That said, what I was advocating, is that what is now Scene, probably could do well with an overhaul, and an explicit enumeration of what a "Scene" implementation really needs to provide, on one side as a contract with the core, and on the other side, as a contract with the module API.<BR>> > <BR>> > <BR>> > <BR>> > I believe that those things should probably look (subtly or radically) different. Not to _hide_ core functionality from the module, but to provide _tailored_ functionality, enumerated for smooth decoupling and encapsulation.<BR>> > <BR>> > Best regards,<BR>> > Stefan Andersson<BR>> > Tribal Media AB<BR>> > <BR>> > <BR>> > <BR>> > <BR>> >> Date: Tue, 14 Apr 2009 20:32:48 +0200<BR>> >> From: melanie@t-data.com<BR>> >> To: opensim-dev@lists.berlios.de<BR>> >> Subject: Re: [Opensim-dev] Supplying IScene instead of Scene for the future region modules mechanism<BR>> >> <BR>> >> I'm not happy with supplying IScene. It would basically curtail the <BR>> >> functionality of region modules to what core believes should be <BR>> >> possible, and will lead to ugly upcasting "(Scene)IScene" that the <BR>> >> code is already rife with.<BR>> >> So, I'm not seeing that as a good idea at all, it limits things too <BR>> >> much.<BR>> >> <BR>> >> Melanie<BR>> >> <BR>> >> Justin Clark-Casey wrote:<BR>> >> > Hey Homer (since this is primarily addressed to you :),<BR>> >> > <BR>> >> > I see you're making some progress on the up-and-coming new region modules mechanism.<BR>> >> > <BR>> >> > Instead of passing Scene itself to region modules, could we create an interface so that we better control the amount of <BR>> >> > innards that we expose to region modules? It's convenient-ish to give the original Scene class to modules now, but it <BR>> >> > will cause us problems down the road.<BR>> >> > <BR>> >> > I'm quite happy to pitch in with this if you want. I suggest renaming the existing IScene to ISceneBase (since that's <BR>> >> > what it really is) and creating a new IScene that's implemented by Scene.<BR>> >> > <BR>> >> > It strikes me that it's going to be more convenient to do this when we introduce the new system than as a separate change.<BR>> >> > <BR>> >> > Thoughts?<BR>> >> > <BR>> >> _______________________________________________<BR>> >> Opensim-dev mailing list<BR>> >> Opensim-dev@lists.berlios.de<BR>> >> https://lists.berlios.de/mailman/listinfo/opensim-dev<BR>> > <BR>> > <BR>> > <BR>> > ------------------------------------------------------------------------<BR>> > <BR>> > _______________________________________________<BR>> > Opensim-dev mailing list<BR>> > Opensim-dev@lists.berlios.de<BR>> > https://lists.berlios.de/mailman/listinfo/opensim-dev<BR>> _______________________________________________<BR>> Opensim-dev mailing list<BR>> Opensim-dev@lists.berlios.de<BR>> https://lists.berlios.de/mailman/listinfo/opensim-dev<BR></body>
</html>