diff options
Diffstat (limited to 'OpenSim/Region/CoreModules')
18 files changed, 847 insertions, 200 deletions
diff --git a/OpenSim/Region/CoreModules/Asset/FlotsamAssetCache.cs b/OpenSim/Region/CoreModules/Asset/FlotsamAssetCache.cs index 7d7176f..e1d4d78 100644 --- a/OpenSim/Region/CoreModules/Asset/FlotsamAssetCache.cs +++ b/OpenSim/Region/CoreModules/Asset/FlotsamAssetCache.cs | |||
@@ -52,7 +52,7 @@ using OpenSim.Services.Interfaces; | |||
52 | [assembly: Addin("FlotsamAssetCache", "1.1")] | 52 | [assembly: Addin("FlotsamAssetCache", "1.1")] |
53 | [assembly: AddinDependency("OpenSim", "0.5")] | 53 | [assembly: AddinDependency("OpenSim", "0.5")] |
54 | 54 | ||
55 | namespace Flotsam.RegionModules.AssetCache | 55 | namespace OpenSim.Region.CoreModules.Asset |
56 | { | 56 | { |
57 | [Extension(Path = "/OpenSim/RegionModules", NodeName = "RegionModule")] | 57 | [Extension(Path = "/OpenSim/RegionModules", NodeName = "RegionModule")] |
58 | public class FlotsamAssetCache : ISharedRegionModule, IImprovedAssetCache, IAssetService | 58 | public class FlotsamAssetCache : ISharedRegionModule, IImprovedAssetCache, IAssetService |
diff --git a/OpenSim/Region/CoreModules/Asset/Tests/FlotsamAssetCacheTests.cs b/OpenSim/Region/CoreModules/Asset/Tests/FlotsamAssetCacheTests.cs index c91b25f..1c2bfd0 100644 --- a/OpenSim/Region/CoreModules/Asset/Tests/FlotsamAssetCacheTests.cs +++ b/OpenSim/Region/CoreModules/Asset/Tests/FlotsamAssetCacheTests.cs | |||
@@ -35,7 +35,6 @@ using Nini.Config; | |||
35 | using NUnit.Framework; | 35 | using NUnit.Framework; |
36 | using OpenMetaverse; | 36 | using OpenMetaverse; |
37 | using OpenMetaverse.Assets; | 37 | using OpenMetaverse.Assets; |
38 | using Flotsam.RegionModules.AssetCache; | ||
39 | using OpenSim.Framework; | 38 | using OpenSim.Framework; |
40 | using OpenSim.Region.Framework.Scenes; | 39 | using OpenSim.Region.Framework.Scenes; |
41 | using OpenSim.Region.Framework.Scenes.Serialization; | 40 | using OpenSim.Region.Framework.Scenes.Serialization; |
diff --git a/OpenSim/Region/CoreModules/Avatar/Attachments/Tests/AttachmentsModuleTests.cs b/OpenSim/Region/CoreModules/Avatar/Attachments/Tests/AttachmentsModuleTests.cs index d9a619d..48f3a23 100644 --- a/OpenSim/Region/CoreModules/Avatar/Attachments/Tests/AttachmentsModuleTests.cs +++ b/OpenSim/Region/CoreModules/Avatar/Attachments/Tests/AttachmentsModuleTests.cs | |||
@@ -461,7 +461,7 @@ namespace OpenSim.Region.CoreModules.Avatar.Attachments.Tests | |||
461 | 461 | ||
462 | SceneObjectGroup rezzedAtt = presence.GetAttachments()[0]; | 462 | SceneObjectGroup rezzedAtt = presence.GetAttachments()[0]; |
463 | 463 | ||
464 | scene.IncomingCloseAgent(presence.UUID); | 464 | scene.IncomingCloseAgent(presence.UUID, false); |
465 | 465 | ||
466 | // Check that we can't retrieve this attachment from the scene. | 466 | // Check that we can't retrieve this attachment from the scene. |
467 | Assert.That(scene.GetSceneObjectGroup(rezzedAtt.UUID), Is.Null); | 467 | Assert.That(scene.GetSceneObjectGroup(rezzedAtt.UUID), Is.Null); |
diff --git a/OpenSim/Region/CoreModules/Avatar/Inventory/Archiver/Tests/InventoryArchiverTests.cs b/OpenSim/Region/CoreModules/Avatar/Inventory/Archiver/Tests/InventoryArchiverTests.cs index b112b6d..12a05b3 100644 --- a/OpenSim/Region/CoreModules/Avatar/Inventory/Archiver/Tests/InventoryArchiverTests.cs +++ b/OpenSim/Region/CoreModules/Avatar/Inventory/Archiver/Tests/InventoryArchiverTests.cs | |||
@@ -350,38 +350,38 @@ namespace OpenSim.Region.CoreModules.Avatar.Inventory.Archiver.Tests | |||
350 | Assert.That(sog1.RootPart.CreatorID, Is.EqualTo(m_uaLL1.PrincipalID)); | 350 | Assert.That(sog1.RootPart.CreatorID, Is.EqualTo(m_uaLL1.PrincipalID)); |
351 | } | 351 | } |
352 | 352 | ||
353 | /// <summary> | 353 | // /// <summary> |
354 | /// Test loading a V0.1 OpenSim Inventory Archive (subject to change since there is no fixed format yet) where | 354 | // /// Test loading a V0.1 OpenSim Inventory Archive (subject to change since there is no fixed format yet) where |
355 | /// an account exists with the same name as the creator, though not the same id. | 355 | // /// an account exists with the same name as the creator, though not the same id. |
356 | /// </summary> | 356 | // /// </summary> |
357 | [Test] | 357 | // [Test] |
358 | public void TestLoadIarV0_1SameNameCreator() | 358 | // public void TestLoadIarV0_1SameNameCreator() |
359 | { | 359 | // { |
360 | TestHelpers.InMethod(); | 360 | // TestHelpers.InMethod(); |
361 | // log4net.Config.XmlConfigurator.Configure(); | 361 | // TestHelpers.EnableLogging(); |
362 | 362 | // | |
363 | UserAccountHelpers.CreateUserWithInventory(m_scene, m_uaMT, "meowfood"); | 363 | // UserAccountHelpers.CreateUserWithInventory(m_scene, m_uaMT, "meowfood"); |
364 | UserAccountHelpers.CreateUserWithInventory(m_scene, m_uaLL2, "hampshire"); | 364 | // UserAccountHelpers.CreateUserWithInventory(m_scene, m_uaLL2, "hampshire"); |
365 | 365 | // | |
366 | m_archiverModule.DearchiveInventory(m_uaMT.FirstName, m_uaMT.LastName, "/", "meowfood", m_iarStream); | 366 | // m_archiverModule.DearchiveInventory(m_uaMT.FirstName, m_uaMT.LastName, "/", "meowfood", m_iarStream); |
367 | InventoryItemBase foundItem1 | 367 | // InventoryItemBase foundItem1 |
368 | = InventoryArchiveUtils.FindItemByPath(m_scene.InventoryService, m_uaMT.PrincipalID, m_item1Name); | 368 | // = InventoryArchiveUtils.FindItemByPath(m_scene.InventoryService, m_uaMT.PrincipalID, m_item1Name); |
369 | 369 | // | |
370 | Assert.That( | 370 | // Assert.That( |
371 | foundItem1.CreatorId, Is.EqualTo(m_uaLL2.PrincipalID.ToString()), | 371 | // foundItem1.CreatorId, Is.EqualTo(m_uaLL2.PrincipalID.ToString()), |
372 | "Loaded item non-uuid creator doesn't match original"); | 372 | // "Loaded item non-uuid creator doesn't match original"); |
373 | Assert.That( | 373 | // Assert.That( |
374 | foundItem1.CreatorIdAsUuid, Is.EqualTo(m_uaLL2.PrincipalID), | 374 | // foundItem1.CreatorIdAsUuid, Is.EqualTo(m_uaLL2.PrincipalID), |
375 | "Loaded item uuid creator doesn't match original"); | 375 | // "Loaded item uuid creator doesn't match original"); |
376 | Assert.That(foundItem1.Owner, Is.EqualTo(m_uaMT.PrincipalID), | 376 | // Assert.That(foundItem1.Owner, Is.EqualTo(m_uaMT.PrincipalID), |
377 | "Loaded item owner doesn't match inventory reciever"); | 377 | // "Loaded item owner doesn't match inventory reciever"); |
378 | 378 | // | |
379 | AssetBase asset1 = m_scene.AssetService.Get(foundItem1.AssetID.ToString()); | 379 | // AssetBase asset1 = m_scene.AssetService.Get(foundItem1.AssetID.ToString()); |
380 | string xmlData = Utils.BytesToString(asset1.Data); | 380 | // string xmlData = Utils.BytesToString(asset1.Data); |
381 | SceneObjectGroup sog1 = SceneObjectSerializer.FromOriginalXmlFormat(xmlData); | 381 | // SceneObjectGroup sog1 = SceneObjectSerializer.FromOriginalXmlFormat(xmlData); |
382 | 382 | // | |
383 | Assert.That(sog1.RootPart.CreatorID, Is.EqualTo(m_uaLL2.PrincipalID)); | 383 | // Assert.That(sog1.RootPart.CreatorID, Is.EqualTo(m_uaLL2.PrincipalID)); |
384 | } | 384 | // } |
385 | 385 | ||
386 | /// <summary> | 386 | /// <summary> |
387 | /// Test loading a V0.1 OpenSim Inventory Archive (subject to change since there is no fixed format yet) where | 387 | /// Test loading a V0.1 OpenSim Inventory Archive (subject to change since there is no fixed format yet) where |
diff --git a/OpenSim/Region/CoreModules/Framework/EntityTransfer/EntityTransferModule.cs b/OpenSim/Region/CoreModules/Framework/EntityTransfer/EntityTransferModule.cs index 9090f64..486f9d2 100644 --- a/OpenSim/Region/CoreModules/Framework/EntityTransfer/EntityTransferModule.cs +++ b/OpenSim/Region/CoreModules/Framework/EntityTransfer/EntityTransferModule.cs | |||
@@ -650,7 +650,7 @@ namespace OpenSim.Region.CoreModules.Framework.EntityTransfer | |||
650 | // an agent cannot teleport back to this region if it has teleported away. | 650 | // an agent cannot teleport back to this region if it has teleported away. |
651 | Thread.Sleep(2000); | 651 | Thread.Sleep(2000); |
652 | 652 | ||
653 | sp.Scene.IncomingCloseAgent(sp.UUID); | 653 | sp.Scene.IncomingCloseAgent(sp.UUID, false); |
654 | } | 654 | } |
655 | else | 655 | else |
656 | { | 656 | { |
diff --git a/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTexture.cs b/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTexture.cs new file mode 100644 index 0000000..fce9490 --- /dev/null +++ b/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTexture.cs | |||
@@ -0,0 +1,61 @@ | |||
1 | /* | ||
2 | * Copyright (c) Contributors, http://opensimulator.org/ | ||
3 | * See CONTRIBUTORS.TXT for a full list of copyright holders. | ||
4 | * | ||
5 | * Redistribution and use in source and binary forms, with or without | ||
6 | * modification, are permitted provided that the following conditions are met: | ||
7 | * * Redistributions of source code must retain the above copyright | ||
8 | * notice, this list of conditions and the following disclaimer. | ||
9 | * * Redistributions in binary form must reproduce the above copyright | ||
10 | * notice, this list of conditions and the following disclaimer in the | ||
11 | * documentation and/or other materials provided with the distribution. | ||
12 | * * Neither the name of the OpenSimulator Project nor the | ||
13 | * names of its contributors may be used to endorse or promote products | ||
14 | * derived from this software without specific prior written permission. | ||
15 | * | ||
16 | * THIS SOFTWARE IS PROVIDED BY THE DEVELOPERS ``AS IS'' AND ANY | ||
17 | * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED | ||
18 | * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE | ||
19 | * DISCLAIMED. IN NO EVENT SHALL THE CONTRIBUTORS BE LIABLE FOR ANY | ||
20 | * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES | ||
21 | * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; | ||
22 | * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND | ||
23 | * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | ||
24 | * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS | ||
25 | * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | ||
26 | */ | ||
27 | |||
28 | using System; | ||
29 | using System.Drawing; | ||
30 | using OpenSim.Region.Framework.Interfaces; | ||
31 | |||
32 | namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | ||
33 | { | ||
34 | public class DynamicTexture : IDynamicTexture | ||
35 | { | ||
36 | public string InputCommands { get; private set; } | ||
37 | public Uri InputUri { get; private set; } | ||
38 | public string InputParams { get; private set; } | ||
39 | public byte[] Data { get; private set; } | ||
40 | public Size Size { get; private set; } | ||
41 | public bool IsReuseable { get; private set; } | ||
42 | |||
43 | public DynamicTexture(string inputCommands, string inputParams, byte[] data, Size size, bool isReuseable) | ||
44 | { | ||
45 | InputCommands = inputCommands; | ||
46 | InputParams = inputParams; | ||
47 | Data = data; | ||
48 | Size = size; | ||
49 | IsReuseable = isReuseable; | ||
50 | } | ||
51 | |||
52 | public DynamicTexture(Uri inputUri, string inputParams, byte[] data, Size size, bool isReuseable) | ||
53 | { | ||
54 | InputUri = inputUri; | ||
55 | InputParams = inputParams; | ||
56 | Data = data; | ||
57 | Size = size; | ||
58 | IsReuseable = isReuseable; | ||
59 | } | ||
60 | } | ||
61 | } \ No newline at end of file | ||
diff --git a/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTextureModule.cs b/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTextureModule.cs index 18bd018..1f340df 100644 --- a/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTextureModule.cs +++ b/OpenSim/Region/CoreModules/Scripting/DynamicTexture/DynamicTextureModule.cs | |||
@@ -42,13 +42,29 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
42 | { | 42 | { |
43 | public class DynamicTextureModule : IRegionModule, IDynamicTextureManager | 43 | public class DynamicTextureModule : IRegionModule, IDynamicTextureManager |
44 | { | 44 | { |
45 | //private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); | 45 | private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); |
46 | 46 | ||
47 | private const int ALL_SIDES = -1; | 47 | private const int ALL_SIDES = -1; |
48 | 48 | ||
49 | public const int DISP_EXPIRE = 1; | 49 | public const int DISP_EXPIRE = 1; |
50 | public const int DISP_TEMP = 2; | 50 | public const int DISP_TEMP = 2; |
51 | 51 | ||
52 | /// <summary> | ||
53 | /// If true then where possible dynamic textures are reused. | ||
54 | /// </summary> | ||
55 | public bool ReuseTextures { get; set; } | ||
56 | |||
57 | /// <summary> | ||
58 | /// If false, then textures which have a low data size are not reused when ReuseTextures = true. | ||
59 | /// </summary> | ||
60 | /// <remarks> | ||
61 | /// LL viewers 3.3.4 and before appear to not fully render textures pulled from the viewer cache if those | ||
62 | /// textures have a relatively high pixel surface but a small data size. Typically, this appears to happen | ||
63 | /// if the data size is smaller than the viewer's discard level 2 size estimate. So if this is setting is | ||
64 | /// false, textures smaller than the calculation in IsSizeReuseable are always regenerated rather than reused | ||
65 | /// to work around this problem.</remarks> | ||
66 | public bool ReuseLowDataTextures { get; set; } | ||
67 | |||
52 | private Dictionary<UUID, Scene> RegisteredScenes = new Dictionary<UUID, Scene>(); | 68 | private Dictionary<UUID, Scene> RegisteredScenes = new Dictionary<UUID, Scene>(); |
53 | 69 | ||
54 | private Dictionary<string, IDynamicTextureRender> RenderPlugins = | 70 | private Dictionary<string, IDynamicTextureRender> RenderPlugins = |
@@ -56,6 +72,15 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
56 | 72 | ||
57 | private Dictionary<UUID, DynamicTextureUpdater> Updaters = new Dictionary<UUID, DynamicTextureUpdater>(); | 73 | private Dictionary<UUID, DynamicTextureUpdater> Updaters = new Dictionary<UUID, DynamicTextureUpdater>(); |
58 | 74 | ||
75 | /// <summary> | ||
76 | /// Record dynamic textures that we can reuse for a given data and parameter combination rather than | ||
77 | /// regenerate. | ||
78 | /// </summary> | ||
79 | /// <remarks> | ||
80 | /// Key is string.Format("{0}{1}", data | ||
81 | /// </remarks> | ||
82 | private Cache m_reuseableDynamicTextures; | ||
83 | |||
59 | #region IDynamicTextureManager Members | 84 | #region IDynamicTextureManager Members |
60 | 85 | ||
61 | public void RegisterRender(string handleType, IDynamicTextureRender render) | 86 | public void RegisterRender(string handleType, IDynamicTextureRender render) |
@@ -69,17 +94,17 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
69 | /// <summary> | 94 | /// <summary> |
70 | /// Called by code which actually renders the dynamic texture to supply texture data. | 95 | /// Called by code which actually renders the dynamic texture to supply texture data. |
71 | /// </summary> | 96 | /// </summary> |
72 | /// <param name="id"></param> | 97 | /// <param name="updaterId"></param> |
73 | /// <param name="data"></param> | 98 | /// <param name="texture"></param> |
74 | public void ReturnData(UUID id, byte[] data) | 99 | public void ReturnData(UUID updaterId, IDynamicTexture texture) |
75 | { | 100 | { |
76 | DynamicTextureUpdater updater = null; | 101 | DynamicTextureUpdater updater = null; |
77 | 102 | ||
78 | lock (Updaters) | 103 | lock (Updaters) |
79 | { | 104 | { |
80 | if (Updaters.ContainsKey(id)) | 105 | if (Updaters.ContainsKey(updaterId)) |
81 | { | 106 | { |
82 | updater = Updaters[id]; | 107 | updater = Updaters[updaterId]; |
83 | } | 108 | } |
84 | } | 109 | } |
85 | 110 | ||
@@ -88,7 +113,16 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
88 | if (RegisteredScenes.ContainsKey(updater.SimUUID)) | 113 | if (RegisteredScenes.ContainsKey(updater.SimUUID)) |
89 | { | 114 | { |
90 | Scene scene = RegisteredScenes[updater.SimUUID]; | 115 | Scene scene = RegisteredScenes[updater.SimUUID]; |
91 | updater.DataReceived(data, scene); | 116 | UUID newTextureID = updater.DataReceived(texture.Data, scene); |
117 | |||
118 | if (ReuseTextures | ||
119 | && !updater.BlendWithOldTexture | ||
120 | && texture.IsReuseable | ||
121 | && (ReuseLowDataTextures || IsDataSizeReuseable(texture))) | ||
122 | { | ||
123 | m_reuseableDynamicTextures.Store( | ||
124 | GenerateReusableTextureKey(texture.InputCommands, texture.InputParams), newTextureID); | ||
125 | } | ||
92 | } | 126 | } |
93 | } | 127 | } |
94 | 128 | ||
@@ -104,6 +138,27 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
104 | } | 138 | } |
105 | } | 139 | } |
106 | 140 | ||
141 | /// <summary> | ||
142 | /// Determines whether the texture is reuseable based on its data size. | ||
143 | /// </summary> | ||
144 | /// <remarks> | ||
145 | /// This is a workaround for a viewer bug where very small data size textures relative to their pixel size | ||
146 | /// are not redisplayed properly when pulled from cache. The calculation here is based on the typical discard | ||
147 | /// level of 2, a 'rate' of 0.125 and 4 components (which makes for a factor of 0.5). | ||
148 | /// </remarks> | ||
149 | /// <returns></returns> | ||
150 | private bool IsDataSizeReuseable(IDynamicTexture texture) | ||
151 | { | ||
152 | // Console.WriteLine("{0} {1}", texture.Size.Width, texture.Size.Height); | ||
153 | int discardLevel2DataThreshold = (int)Math.Ceiling((texture.Size.Width >> 2) * (texture.Size.Height >> 2) * 0.5); | ||
154 | |||
155 | // m_log.DebugFormat( | ||
156 | // "[DYNAMIC TEXTURE MODULE]: Discard level 2 threshold {0}, texture data length {1}", | ||
157 | // discardLevel2DataThreshold, texture.Data.Length); | ||
158 | |||
159 | return discardLevel2DataThreshold < texture.Data.Length; | ||
160 | } | ||
161 | |||
107 | public UUID AddDynamicTextureURL(UUID simID, UUID primID, string contentType, string url, | 162 | public UUID AddDynamicTextureURL(UUID simID, UUID primID, string contentType, string url, |
108 | string extraParams, int updateTimer) | 163 | string extraParams, int updateTimer) |
109 | { | 164 | { |
@@ -167,22 +222,61 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
167 | public UUID AddDynamicTextureData(UUID simID, UUID primID, string contentType, string data, | 222 | public UUID AddDynamicTextureData(UUID simID, UUID primID, string contentType, string data, |
168 | string extraParams, int updateTimer, bool SetBlending, int disp, byte AlphaValue, int face) | 223 | string extraParams, int updateTimer, bool SetBlending, int disp, byte AlphaValue, int face) |
169 | { | 224 | { |
170 | if (RenderPlugins.ContainsKey(contentType)) | 225 | if (!RenderPlugins.ContainsKey(contentType)) |
226 | return UUID.Zero; | ||
227 | |||
228 | Scene scene; | ||
229 | RegisteredScenes.TryGetValue(simID, out scene); | ||
230 | |||
231 | if (scene == null) | ||
232 | return UUID.Zero; | ||
233 | |||
234 | SceneObjectPart part = scene.GetSceneObjectPart(primID); | ||
235 | |||
236 | if (part == null) | ||
237 | return UUID.Zero; | ||
238 | |||
239 | // If we want to reuse dynamic textures then we have to ignore any request from the caller to expire | ||
240 | // them. | ||
241 | if (ReuseTextures) | ||
242 | disp = disp & ~DISP_EXPIRE; | ||
243 | |||
244 | DynamicTextureUpdater updater = new DynamicTextureUpdater(); | ||
245 | updater.SimUUID = simID; | ||
246 | updater.PrimID = primID; | ||
247 | updater.ContentType = contentType; | ||
248 | updater.BodyData = data; | ||
249 | updater.UpdateTimer = updateTimer; | ||
250 | updater.UpdaterID = UUID.Random(); | ||
251 | updater.Params = extraParams; | ||
252 | updater.BlendWithOldTexture = SetBlending; | ||
253 | updater.FrontAlpha = AlphaValue; | ||
254 | updater.Face = face; | ||
255 | updater.Url = "Local image"; | ||
256 | updater.Disp = disp; | ||
257 | |||
258 | object objReusableTextureUUID = null; | ||
259 | |||
260 | if (ReuseTextures && !updater.BlendWithOldTexture) | ||
171 | { | 261 | { |
172 | DynamicTextureUpdater updater = new DynamicTextureUpdater(); | 262 | string reuseableTextureKey = GenerateReusableTextureKey(data, extraParams); |
173 | updater.SimUUID = simID; | 263 | objReusableTextureUUID = m_reuseableDynamicTextures.Get(reuseableTextureKey); |
174 | updater.PrimID = primID; | 264 | |
175 | updater.ContentType = contentType; | 265 | if (objReusableTextureUUID != null) |
176 | updater.BodyData = data; | 266 | { |
177 | updater.UpdateTimer = updateTimer; | 267 | // If something else has removed this temporary asset from the cache, detect and invalidate |
178 | updater.UpdaterID = UUID.Random(); | 268 | // our cached uuid. |
179 | updater.Params = extraParams; | 269 | if (scene.AssetService.GetMetadata(objReusableTextureUUID.ToString()) == null) |
180 | updater.BlendWithOldTexture = SetBlending; | 270 | { |
181 | updater.FrontAlpha = AlphaValue; | 271 | m_reuseableDynamicTextures.Invalidate(reuseableTextureKey); |
182 | updater.Face = face; | 272 | objReusableTextureUUID = null; |
183 | updater.Url = "Local image"; | 273 | } |
184 | updater.Disp = disp; | 274 | } |
275 | } | ||
185 | 276 | ||
277 | // We cannot reuse a dynamic texture if the data is going to be blended with something already there. | ||
278 | if (objReusableTextureUUID == null) | ||
279 | { | ||
186 | lock (Updaters) | 280 | lock (Updaters) |
187 | { | 281 | { |
188 | if (!Updaters.ContainsKey(updater.UpdaterID)) | 282 | if (!Updaters.ContainsKey(updater.UpdaterID)) |
@@ -191,11 +285,29 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
191 | } | 285 | } |
192 | } | 286 | } |
193 | 287 | ||
288 | // m_log.DebugFormat( | ||
289 | // "[DYNAMIC TEXTURE MODULE]: Requesting generation of new dynamic texture for {0} in {1}", | ||
290 | // part.Name, part.ParentGroup.Scene.Name); | ||
291 | |||
194 | RenderPlugins[contentType].AsyncConvertData(updater.UpdaterID, data, extraParams); | 292 | RenderPlugins[contentType].AsyncConvertData(updater.UpdaterID, data, extraParams); |
195 | return updater.UpdaterID; | ||
196 | } | 293 | } |
197 | 294 | else | |
198 | return UUID.Zero; | 295 | { |
296 | // m_log.DebugFormat( | ||
297 | // "[DYNAMIC TEXTURE MODULE]: Reusing cached texture {0} for {1} in {2}", | ||
298 | // objReusableTextureUUID, part.Name, part.ParentGroup.Scene.Name); | ||
299 | |||
300 | // No need to add to updaters as the texture is always the same. Not that this functionality | ||
301 | // apppears to be implemented anyway. | ||
302 | updater.UpdatePart(part, (UUID)objReusableTextureUUID); | ||
303 | } | ||
304 | |||
305 | return updater.UpdaterID; | ||
306 | } | ||
307 | |||
308 | private string GenerateReusableTextureKey(string data, string extraParams) | ||
309 | { | ||
310 | return string.Format("{0}{1}", data, extraParams); | ||
199 | } | 311 | } |
200 | 312 | ||
201 | public void GetDrawStringSize(string contentType, string text, string fontName, int fontSize, | 313 | public void GetDrawStringSize(string contentType, string text, string fontName, int fontSize, |
@@ -215,6 +327,13 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
215 | 327 | ||
216 | public void Initialise(Scene scene, IConfigSource config) | 328 | public void Initialise(Scene scene, IConfigSource config) |
217 | { | 329 | { |
330 | IConfig texturesConfig = config.Configs["Textures"]; | ||
331 | if (texturesConfig != null) | ||
332 | { | ||
333 | ReuseTextures = texturesConfig.GetBoolean("ReuseDynamicTextures", false); | ||
334 | ReuseLowDataTextures = texturesConfig.GetBoolean("ReuseDynamicLowDataTextures", false); | ||
335 | } | ||
336 | |||
218 | if (!RegisteredScenes.ContainsKey(scene.RegionInfo.RegionID)) | 337 | if (!RegisteredScenes.ContainsKey(scene.RegionInfo.RegionID)) |
219 | { | 338 | { |
220 | RegisteredScenes.Add(scene.RegionInfo.RegionID, scene); | 339 | RegisteredScenes.Add(scene.RegionInfo.RegionID, scene); |
@@ -224,6 +343,11 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
224 | 343 | ||
225 | public void PostInitialise() | 344 | public void PostInitialise() |
226 | { | 345 | { |
346 | if (ReuseTextures) | ||
347 | { | ||
348 | m_reuseableDynamicTextures = new Cache(CacheMedium.Memory, CacheStrategy.Conservative); | ||
349 | m_reuseableDynamicTextures.DefaultTTL = new TimeSpan(24, 0, 0); | ||
350 | } | ||
227 | } | 351 | } |
228 | 352 | ||
229 | public void Close() | 353 | public void Close() |
@@ -269,9 +393,60 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
269 | } | 393 | } |
270 | 394 | ||
271 | /// <summary> | 395 | /// <summary> |
396 | /// Update the given part with the new texture. | ||
397 | /// </summary> | ||
398 | /// <returns> | ||
399 | /// The old texture UUID. | ||
400 | /// </returns> | ||
401 | public UUID UpdatePart(SceneObjectPart part, UUID textureID) | ||
402 | { | ||
403 | UUID oldID; | ||
404 | |||
405 | lock (part) | ||
406 | { | ||
407 | // mostly keep the values from before | ||
408 | Primitive.TextureEntry tmptex = part.Shape.Textures; | ||
409 | |||
410 | // FIXME: Need to return the appropriate ID if only a single face is replaced. | ||
411 | oldID = tmptex.DefaultTexture.TextureID; | ||
412 | |||
413 | if (Face == ALL_SIDES) | ||
414 | { | ||
415 | oldID = tmptex.DefaultTexture.TextureID; | ||
416 | tmptex.DefaultTexture.TextureID = textureID; | ||
417 | } | ||
418 | else | ||
419 | { | ||
420 | try | ||
421 | { | ||
422 | Primitive.TextureEntryFace texface = tmptex.CreateFace((uint)Face); | ||
423 | texface.TextureID = textureID; | ||
424 | tmptex.FaceTextures[Face] = texface; | ||
425 | } | ||
426 | catch (Exception) | ||
427 | { | ||
428 | tmptex.DefaultTexture.TextureID = textureID; | ||
429 | } | ||
430 | } | ||
431 | |||
432 | // I'm pretty sure we always want to force this to true | ||
433 | // I'm pretty sure noone whats to set fullbright true if it wasn't true before. | ||
434 | // tmptex.DefaultTexture.Fullbright = true; | ||
435 | |||
436 | part.UpdateTextureEntry(tmptex.GetBytes()); | ||
437 | } | ||
438 | |||
439 | return oldID; | ||
440 | } | ||
441 | |||
442 | /// <summary> | ||
272 | /// Called once new texture data has been received for this updater. | 443 | /// Called once new texture data has been received for this updater. |
273 | /// </summary> | 444 | /// </summary> |
274 | public void DataReceived(byte[] data, Scene scene) | 445 | /// <param name="data"></param> |
446 | /// <param name="scene"></param> | ||
447 | /// <param name="isReuseable">True if the data given is reuseable.</param> | ||
448 | /// <returns>The asset UUID given to the incoming data.</returns> | ||
449 | public UUID DataReceived(byte[] data, Scene scene) | ||
275 | { | 450 | { |
276 | SceneObjectPart part = scene.GetSceneObjectPart(PrimID); | 451 | SceneObjectPart part = scene.GetSceneObjectPart(PrimID); |
277 | 452 | ||
@@ -281,7 +456,8 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
281 | String.Format("DynamicTextureModule: Error preparing image using URL {0}", Url); | 456 | String.Format("DynamicTextureModule: Error preparing image using URL {0}", Url); |
282 | scene.SimChat(Utils.StringToBytes(msg), ChatTypeEnum.Say, | 457 | scene.SimChat(Utils.StringToBytes(msg), ChatTypeEnum.Say, |
283 | 0, part.ParentGroup.RootPart.AbsolutePosition, part.Name, part.UUID, false); | 458 | 0, part.ParentGroup.RootPart.AbsolutePosition, part.Name, part.UUID, false); |
284 | return; | 459 | |
460 | return UUID.Zero; | ||
285 | } | 461 | } |
286 | 462 | ||
287 | byte[] assetData = null; | 463 | byte[] assetData = null; |
@@ -319,56 +495,29 @@ namespace OpenSim.Region.CoreModules.Scripting.DynamicTexture | |||
319 | IJ2KDecoder cacheLayerDecode = scene.RequestModuleInterface<IJ2KDecoder>(); | 495 | IJ2KDecoder cacheLayerDecode = scene.RequestModuleInterface<IJ2KDecoder>(); |
320 | if (cacheLayerDecode != null) | 496 | if (cacheLayerDecode != null) |
321 | { | 497 | { |
322 | cacheLayerDecode.Decode(asset.FullID, asset.Data); | 498 | if (!cacheLayerDecode.Decode(asset.FullID, asset.Data)) |
323 | cacheLayerDecode = null; | 499 | m_log.WarnFormat( |
500 | "[DYNAMIC TEXTURE MODULE]: Decoding of dynamically generated asset {0} for {1} in {2} failed", | ||
501 | asset.ID, part.Name, part.ParentGroup.Scene.Name); | ||
324 | } | 502 | } |
325 | 503 | ||
326 | UUID oldID = UUID.Zero; | 504 | UUID oldID = UpdatePart(part, asset.FullID); |
327 | |||
328 | lock (part) | ||
329 | { | ||
330 | // mostly keep the values from before | ||
331 | Primitive.TextureEntry tmptex = part.Shape.Textures; | ||
332 | |||
333 | // remove the old asset from the cache | ||
334 | oldID = tmptex.DefaultTexture.TextureID; | ||
335 | |||
336 | if (Face == ALL_SIDES) | ||
337 | { | ||
338 | tmptex.DefaultTexture.TextureID = asset.FullID; | ||
339 | } | ||
340 | else | ||
341 | { | ||
342 | try | ||
343 | { | ||
344 | Primitive.TextureEntryFace texface = tmptex.CreateFace((uint)Face); | ||
345 | texface.TextureID = asset.FullID; | ||
346 | tmptex.FaceTextures[Face] = texface; | ||
347 | } | ||
348 | catch (Exception) | ||
349 | { | ||
350 | tmptex.DefaultTexture.TextureID = asset.FullID; | ||
351 | } | ||
352 | } | ||
353 | |||
354 | // I'm pretty sure we always want to force this to true | ||
355 | // I'm pretty sure noone whats to set fullbright true if it wasn't true before. | ||
356 | // tmptex.DefaultTexture.Fullbright = true; | ||
357 | |||
358 | part.UpdateTextureEntry(tmptex.GetBytes()); | ||
359 | } | ||
360 | 505 | ||
361 | if (oldID != UUID.Zero && ((Disp & DISP_EXPIRE) != 0)) | 506 | if (oldID != UUID.Zero && ((Disp & DISP_EXPIRE) != 0)) |
362 | { | 507 | { |
363 | if (oldAsset == null) oldAsset = scene.AssetService.Get(oldID.ToString()); | 508 | if (oldAsset == null) |
509 | oldAsset = scene.AssetService.Get(oldID.ToString()); | ||
510 | |||
364 | if (oldAsset != null) | 511 | if (oldAsset != null) |
365 | { | 512 | { |
366 | if (oldAsset.Temporary == true) | 513 | if (oldAsset.Temporary) |
367 | { | 514 | { |
368 | scene.AssetService.Delete(oldID.ToString()); | 515 | scene.AssetService.Delete(oldID.ToString()); |
369 | } | 516 | } |
370 | } | 517 | } |
371 | } | 518 | } |
519 | |||
520 | return asset.FullID; | ||
372 | } | 521 | } |
373 | 522 | ||
374 | private byte[] BlendTextures(byte[] frontImage, byte[] backImage, bool setNewAlpha, byte newAlpha) | 523 | private byte[] BlendTextures(byte[] frontImage, byte[] backImage, bool setNewAlpha, byte newAlpha) |
diff --git a/OpenSim/Region/CoreModules/Scripting/LSLHttp/UrlModule.cs b/OpenSim/Region/CoreModules/Scripting/LSLHttp/UrlModule.cs index 13762f7..99ffbe7 100644 --- a/OpenSim/Region/CoreModules/Scripting/LSLHttp/UrlModule.cs +++ b/OpenSim/Region/CoreModules/Scripting/LSLHttp/UrlModule.cs | |||
@@ -58,6 +58,7 @@ namespace OpenSim.Region.CoreModules.Scripting.LSLHttp | |||
58 | public string body; | 58 | public string body; |
59 | public int responseCode; | 59 | public int responseCode; |
60 | public string responseBody; | 60 | public string responseBody; |
61 | public string responseType = "text/plain"; | ||
61 | //public ManualResetEvent ev; | 62 | //public ManualResetEvent ev; |
62 | public bool requestDone; | 63 | public bool requestDone; |
63 | public int startTime; | 64 | public int startTime; |
@@ -270,6 +271,22 @@ namespace OpenSim.Region.CoreModules.Scripting.LSLHttp | |||
270 | } | 271 | } |
271 | } | 272 | } |
272 | 273 | ||
274 | public void HttpContentType(UUID request, string type) | ||
275 | { | ||
276 | lock (m_UrlMap) | ||
277 | { | ||
278 | if (m_RequestMap.ContainsKey(request)) | ||
279 | { | ||
280 | UrlData urlData = m_RequestMap[request]; | ||
281 | urlData.requests[request].responseType = type; | ||
282 | } | ||
283 | else | ||
284 | { | ||
285 | m_log.Info("[HttpRequestHandler] There is no http-in request with id " + request.ToString()); | ||
286 | } | ||
287 | } | ||
288 | } | ||
289 | |||
273 | public void HttpResponse(UUID request, int status, string body) | 290 | public void HttpResponse(UUID request, int status, string body) |
274 | { | 291 | { |
275 | lock (m_RequestMap) | 292 | lock (m_RequestMap) |
diff --git a/OpenSim/Region/CoreModules/Scripting/LoadImageURL/LoadImageURLModule.cs b/OpenSim/Region/CoreModules/Scripting/LoadImageURL/LoadImageURLModule.cs index 6f83948..45e6527 100644 --- a/OpenSim/Region/CoreModules/Scripting/LoadImageURL/LoadImageURLModule.cs +++ b/OpenSim/Region/CoreModules/Scripting/LoadImageURL/LoadImageURLModule.cs | |||
@@ -32,6 +32,7 @@ using System.Net; | |||
32 | using Nini.Config; | 32 | using Nini.Config; |
33 | using OpenMetaverse; | 33 | using OpenMetaverse; |
34 | using OpenMetaverse.Imaging; | 34 | using OpenMetaverse.Imaging; |
35 | using OpenSim.Region.CoreModules.Scripting.DynamicTexture; | ||
35 | using OpenSim.Region.Framework.Interfaces; | 36 | using OpenSim.Region.Framework.Interfaces; |
36 | using OpenSim.Region.Framework.Scenes; | 37 | using OpenSim.Region.Framework.Scenes; |
37 | using log4net; | 38 | using log4net; |
@@ -67,12 +68,18 @@ namespace OpenSim.Region.CoreModules.Scripting.LoadImageURL | |||
67 | return true; | 68 | return true; |
68 | } | 69 | } |
69 | 70 | ||
70 | public byte[] ConvertUrl(string url, string extraParams) | 71 | // public bool AlwaysIdenticalConversion(string bodyData, string extraParams) |
72 | // { | ||
73 | // // We don't support conversion of body data. | ||
74 | // return false; | ||
75 | // } | ||
76 | |||
77 | public IDynamicTexture ConvertUrl(string url, string extraParams) | ||
71 | { | 78 | { |
72 | return null; | 79 | return null; |
73 | } | 80 | } |
74 | 81 | ||
75 | public byte[] ConvertStream(Stream data, string extraParams) | 82 | public IDynamicTexture ConvertData(string bodyData, string extraParams) |
76 | { | 83 | { |
77 | return null; | 84 | return null; |
78 | } | 85 | } |
@@ -165,11 +172,11 @@ namespace OpenSim.Region.CoreModules.Scripting.LoadImageURL | |||
165 | 172 | ||
166 | private void HttpRequestReturn(IAsyncResult result) | 173 | private void HttpRequestReturn(IAsyncResult result) |
167 | { | 174 | { |
168 | |||
169 | RequestState state = (RequestState) result.AsyncState; | 175 | RequestState state = (RequestState) result.AsyncState; |
170 | WebRequest request = (WebRequest) state.Request; | 176 | WebRequest request = (WebRequest) state.Request; |
171 | Stream stream = null; | 177 | Stream stream = null; |
172 | byte[] imageJ2000 = new byte[0]; | 178 | byte[] imageJ2000 = new byte[0]; |
179 | Size newSize = new Size(0, 0); | ||
173 | 180 | ||
174 | try | 181 | try |
175 | { | 182 | { |
@@ -182,37 +189,43 @@ namespace OpenSim.Region.CoreModules.Scripting.LoadImageURL | |||
182 | try | 189 | try |
183 | { | 190 | { |
184 | Bitmap image = new Bitmap(stream); | 191 | Bitmap image = new Bitmap(stream); |
185 | Size newsize; | ||
186 | 192 | ||
187 | // TODO: make this a bit less hard coded | 193 | // TODO: make this a bit less hard coded |
188 | if ((image.Height < 64) && (image.Width < 64)) | 194 | if ((image.Height < 64) && (image.Width < 64)) |
189 | { | 195 | { |
190 | newsize = new Size(32, 32); | 196 | newSize.Width = 32; |
197 | newSize.Height = 32; | ||
191 | } | 198 | } |
192 | else if ((image.Height < 128) && (image.Width < 128)) | 199 | else if ((image.Height < 128) && (image.Width < 128)) |
193 | { | 200 | { |
194 | newsize = new Size(64, 64); | 201 | newSize.Width = 64; |
202 | newSize.Height = 64; | ||
195 | } | 203 | } |
196 | else if ((image.Height < 256) && (image.Width < 256)) | 204 | else if ((image.Height < 256) && (image.Width < 256)) |
197 | { | 205 | { |
198 | newsize = new Size(128, 128); | 206 | newSize.Width = 128; |
207 | newSize.Height = 128; | ||
199 | } | 208 | } |
200 | else if ((image.Height < 512 && image.Width < 512)) | 209 | else if ((image.Height < 512 && image.Width < 512)) |
201 | { | 210 | { |
202 | newsize = new Size(256, 256); | 211 | newSize.Width = 256; |
212 | newSize.Height = 256; | ||
203 | } | 213 | } |
204 | else if ((image.Height < 1024 && image.Width < 1024)) | 214 | else if ((image.Height < 1024 && image.Width < 1024)) |
205 | { | 215 | { |
206 | newsize = new Size(512, 512); | 216 | newSize.Width = 512; |
217 | newSize.Height = 512; | ||
207 | } | 218 | } |
208 | else | 219 | else |
209 | { | 220 | { |
210 | newsize = new Size(1024, 1024); | 221 | newSize.Width = 1024; |
222 | newSize.Height = 1024; | ||
211 | } | 223 | } |
212 | 224 | ||
213 | Bitmap resize = new Bitmap(image, newsize); | 225 | using (Bitmap resize = new Bitmap(image, newSize)) |
214 | 226 | { | |
215 | imageJ2000 = OpenJPEG.EncodeFromImage(resize, true); | 227 | imageJ2000 = OpenJPEG.EncodeFromImage(resize, true); |
228 | } | ||
216 | } | 229 | } |
217 | catch (Exception) | 230 | catch (Exception) |
218 | { | 231 | { |
@@ -227,7 +240,6 @@ namespace OpenSim.Region.CoreModules.Scripting.LoadImageURL | |||
227 | } | 240 | } |
228 | catch (WebException) | 241 | catch (WebException) |
229 | { | 242 | { |
230 | |||
231 | } | 243 | } |
232 | finally | 244 | finally |
233 | { | 245 | { |
@@ -236,9 +248,14 @@ namespace OpenSim.Region.CoreModules.Scripting.LoadImageURL | |||
236 | stream.Close(); | 248 | stream.Close(); |
237 | } | 249 | } |
238 | } | 250 | } |
239 | m_log.DebugFormat("[LOADIMAGEURLMODULE] Returning {0} bytes of image data for request {1}", | 251 | |
252 | m_log.DebugFormat("[LOADIMAGEURLMODULE]: Returning {0} bytes of image data for request {1}", | ||
240 | imageJ2000.Length, state.RequestID); | 253 | imageJ2000.Length, state.RequestID); |
241 | m_textureManager.ReturnData(state.RequestID, imageJ2000); | 254 | |
255 | m_textureManager.ReturnData( | ||
256 | state.RequestID, | ||
257 | new OpenSim.Region.CoreModules.Scripting.DynamicTexture.DynamicTexture( | ||
258 | request.RequestUri, null, imageJ2000, newSize, false)); | ||
242 | } | 259 | } |
243 | 260 | ||
244 | #region Nested type: RequestState | 261 | #region Nested type: RequestState |
diff --git a/OpenSim/Region/CoreModules/Scripting/VectorRender/Tests/VectorRenderModuleTests.cs b/OpenSim/Region/CoreModules/Scripting/VectorRender/Tests/VectorRenderModuleTests.cs index 9787c8c..41baccc 100644 --- a/OpenSim/Region/CoreModules/Scripting/VectorRender/Tests/VectorRenderModuleTests.cs +++ b/OpenSim/Region/CoreModules/Scripting/VectorRender/Tests/VectorRenderModuleTests.cs | |||
@@ -45,31 +45,292 @@ using OpenSim.Tests.Common.Mock; | |||
45 | namespace OpenSim.Region.CoreModules.Scripting.VectorRender.Tests | 45 | namespace OpenSim.Region.CoreModules.Scripting.VectorRender.Tests |
46 | { | 46 | { |
47 | [TestFixture] | 47 | [TestFixture] |
48 | public class VectorRenderModuleTests | 48 | public class VectorRenderModuleTests : OpenSimTestCase |
49 | { | 49 | { |
50 | Scene m_scene; | ||
51 | DynamicTextureModule m_dtm; | ||
52 | VectorRenderModule m_vrm; | ||
53 | |||
54 | private void SetupScene(bool reuseTextures) | ||
55 | { | ||
56 | m_scene = new SceneHelpers().SetupScene(); | ||
57 | |||
58 | m_dtm = new DynamicTextureModule(); | ||
59 | m_dtm.ReuseTextures = reuseTextures; | ||
60 | // m_dtm.ReuseLowDataTextures = reuseTextures; | ||
61 | |||
62 | m_vrm = new VectorRenderModule(); | ||
63 | |||
64 | SceneHelpers.SetupSceneModules(m_scene, m_dtm, m_vrm); | ||
65 | } | ||
66 | |||
50 | [Test] | 67 | [Test] |
51 | public void TestDraw() | 68 | public void TestDraw() |
52 | { | 69 | { |
53 | TestHelpers.InMethod(); | 70 | TestHelpers.InMethod(); |
54 | 71 | ||
55 | Scene scene = new SceneHelpers().SetupScene(); | 72 | SetupScene(false); |
56 | DynamicTextureModule dtm = new DynamicTextureModule(); | 73 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); |
57 | VectorRenderModule vrm = new VectorRenderModule(); | ||
58 | SceneHelpers.SetupSceneModules(scene, dtm, vrm); | ||
59 | |||
60 | SceneObjectGroup so = SceneHelpers.AddSceneObject(scene); | ||
61 | UUID originalTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | 74 | UUID originalTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; |
62 | 75 | ||
63 | dtm.AddDynamicTextureData( | 76 | m_dtm.AddDynamicTextureData( |
64 | scene.RegionInfo.RegionID, | 77 | m_scene.RegionInfo.RegionID, |
65 | so.UUID, | 78 | so.UUID, |
66 | vrm.GetContentType(), | 79 | m_vrm.GetContentType(), |
67 | "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World;", | 80 | "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World;", |
68 | "", | 81 | "", |
69 | 0); | 82 | 0); |
70 | 83 | ||
84 | Assert.That(originalTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | ||
85 | } | ||
86 | |||
87 | [Test] | ||
88 | public void TestRepeatSameDraw() | ||
89 | { | ||
90 | TestHelpers.InMethod(); | ||
91 | |||
92 | string dtText = "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World;"; | ||
93 | |||
94 | SetupScene(false); | ||
95 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); | ||
96 | |||
97 | m_dtm.AddDynamicTextureData( | ||
98 | m_scene.RegionInfo.RegionID, | ||
99 | so.UUID, | ||
100 | m_vrm.GetContentType(), | ||
101 | dtText, | ||
102 | "", | ||
103 | 0); | ||
104 | |||
105 | UUID firstDynamicTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | ||
106 | |||
107 | m_dtm.AddDynamicTextureData( | ||
108 | m_scene.RegionInfo.RegionID, | ||
109 | so.UUID, | ||
110 | m_vrm.GetContentType(), | ||
111 | dtText, | ||
112 | "", | ||
113 | 0); | ||
114 | |||
115 | Assert.That(firstDynamicTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | ||
116 | } | ||
117 | |||
118 | [Test] | ||
119 | public void TestRepeatSameDrawDifferentExtraParams() | ||
120 | { | ||
121 | TestHelpers.InMethod(); | ||
122 | |||
123 | string dtText = "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World;"; | ||
124 | |||
125 | SetupScene(false); | ||
126 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); | ||
127 | |||
128 | m_dtm.AddDynamicTextureData( | ||
129 | m_scene.RegionInfo.RegionID, | ||
130 | so.UUID, | ||
131 | m_vrm.GetContentType(), | ||
132 | dtText, | ||
133 | "", | ||
134 | 0); | ||
135 | |||
136 | UUID firstDynamicTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | ||
137 | |||
138 | m_dtm.AddDynamicTextureData( | ||
139 | m_scene.RegionInfo.RegionID, | ||
140 | so.UUID, | ||
141 | m_vrm.GetContentType(), | ||
142 | dtText, | ||
143 | "alpha:250", | ||
144 | 0); | ||
145 | |||
146 | Assert.That(firstDynamicTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | ||
147 | } | ||
148 | |||
149 | [Test] | ||
150 | public void TestRepeatSameDrawContainingImage() | ||
151 | { | ||
152 | TestHelpers.InMethod(); | ||
153 | |||
154 | string dtText | ||
155 | = "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World; Image http://localhost/shouldnotexist.png"; | ||
156 | |||
157 | SetupScene(false); | ||
158 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); | ||
159 | |||
160 | m_dtm.AddDynamicTextureData( | ||
161 | m_scene.RegionInfo.RegionID, | ||
162 | so.UUID, | ||
163 | m_vrm.GetContentType(), | ||
164 | dtText, | ||
165 | "", | ||
166 | 0); | ||
167 | |||
168 | UUID firstDynamicTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | ||
169 | |||
170 | m_dtm.AddDynamicTextureData( | ||
171 | m_scene.RegionInfo.RegionID, | ||
172 | so.UUID, | ||
173 | m_vrm.GetContentType(), | ||
174 | dtText, | ||
175 | "", | ||
176 | 0); | ||
177 | |||
178 | Assert.That(firstDynamicTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | ||
179 | } | ||
180 | |||
181 | [Test] | ||
182 | public void TestDrawReusingTexture() | ||
183 | { | ||
184 | TestHelpers.InMethod(); | ||
185 | |||
186 | SetupScene(true); | ||
187 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); | ||
188 | UUID originalTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | ||
189 | |||
190 | m_dtm.AddDynamicTextureData( | ||
191 | m_scene.RegionInfo.RegionID, | ||
192 | so.UUID, | ||
193 | m_vrm.GetContentType(), | ||
194 | "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World;", | ||
195 | "", | ||
196 | 0); | ||
71 | 197 | ||
72 | Assert.That(originalTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | 198 | Assert.That(originalTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); |
73 | } | 199 | } |
200 | |||
201 | [Test] | ||
202 | public void TestRepeatSameDrawReusingTexture() | ||
203 | { | ||
204 | TestHelpers.InMethod(); | ||
205 | // TestHelpers.EnableLogging(); | ||
206 | |||
207 | string dtText = "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World;"; | ||
208 | |||
209 | SetupScene(true); | ||
210 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); | ||
211 | |||
212 | m_dtm.AddDynamicTextureData( | ||
213 | m_scene.RegionInfo.RegionID, | ||
214 | so.UUID, | ||
215 | m_vrm.GetContentType(), | ||
216 | dtText, | ||
217 | "", | ||
218 | 0); | ||
219 | |||
220 | UUID firstDynamicTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | ||
221 | |||
222 | m_dtm.AddDynamicTextureData( | ||
223 | m_scene.RegionInfo.RegionID, | ||
224 | so.UUID, | ||
225 | m_vrm.GetContentType(), | ||
226 | dtText, | ||
227 | "", | ||
228 | 0); | ||
229 | |||
230 | Assert.That(firstDynamicTextureID, Is.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | ||
231 | } | ||
232 | |||
233 | /// <summary> | ||
234 | /// Test a low data dynamically generated texture such that it is treated as a low data texture that causes | ||
235 | /// problems for current viewers. | ||
236 | /// </summary> | ||
237 | /// <remarks> | ||
238 | /// As we do not set DynamicTextureModule.ReuseLowDataTextures = true in this test, it should not reuse the | ||
239 | /// texture | ||
240 | /// </remarks> | ||
241 | [Test] | ||
242 | public void TestRepeatSameDrawLowDataTexture() | ||
243 | { | ||
244 | TestHelpers.InMethod(); | ||
245 | // TestHelpers.EnableLogging(); | ||
246 | |||
247 | string dtText = "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World;"; | ||
248 | |||
249 | SetupScene(true); | ||
250 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); | ||
251 | |||
252 | m_dtm.AddDynamicTextureData( | ||
253 | m_scene.RegionInfo.RegionID, | ||
254 | so.UUID, | ||
255 | m_vrm.GetContentType(), | ||
256 | dtText, | ||
257 | "1024", | ||
258 | 0); | ||
259 | |||
260 | UUID firstDynamicTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | ||
261 | |||
262 | m_dtm.AddDynamicTextureData( | ||
263 | m_scene.RegionInfo.RegionID, | ||
264 | so.UUID, | ||
265 | m_vrm.GetContentType(), | ||
266 | dtText, | ||
267 | "1024", | ||
268 | 0); | ||
269 | |||
270 | Assert.That(firstDynamicTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | ||
271 | } | ||
272 | |||
273 | [Test] | ||
274 | public void TestRepeatSameDrawDifferentExtraParamsReusingTexture() | ||
275 | { | ||
276 | TestHelpers.InMethod(); | ||
277 | |||
278 | string dtText = "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World;"; | ||
279 | |||
280 | SetupScene(true); | ||
281 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); | ||
282 | |||
283 | m_dtm.AddDynamicTextureData( | ||
284 | m_scene.RegionInfo.RegionID, | ||
285 | so.UUID, | ||
286 | m_vrm.GetContentType(), | ||
287 | dtText, | ||
288 | "", | ||
289 | 0); | ||
290 | |||
291 | UUID firstDynamicTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | ||
292 | |||
293 | m_dtm.AddDynamicTextureData( | ||
294 | m_scene.RegionInfo.RegionID, | ||
295 | so.UUID, | ||
296 | m_vrm.GetContentType(), | ||
297 | dtText, | ||
298 | "alpha:250", | ||
299 | 0); | ||
300 | |||
301 | Assert.That(firstDynamicTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | ||
302 | } | ||
303 | |||
304 | [Test] | ||
305 | public void TestRepeatSameDrawContainingImageReusingTexture() | ||
306 | { | ||
307 | TestHelpers.InMethod(); | ||
308 | |||
309 | string dtText | ||
310 | = "PenColour BLACK; MoveTo 40,220; FontSize 32; Text Hello World; Image http://localhost/shouldnotexist.png"; | ||
311 | |||
312 | SetupScene(true); | ||
313 | SceneObjectGroup so = SceneHelpers.AddSceneObject(m_scene); | ||
314 | |||
315 | m_dtm.AddDynamicTextureData( | ||
316 | m_scene.RegionInfo.RegionID, | ||
317 | so.UUID, | ||
318 | m_vrm.GetContentType(), | ||
319 | dtText, | ||
320 | "", | ||
321 | 0); | ||
322 | |||
323 | UUID firstDynamicTextureID = so.RootPart.Shape.Textures.GetFace(0).TextureID; | ||
324 | |||
325 | m_dtm.AddDynamicTextureData( | ||
326 | m_scene.RegionInfo.RegionID, | ||
327 | so.UUID, | ||
328 | m_vrm.GetContentType(), | ||
329 | dtText, | ||
330 | "", | ||
331 | 0); | ||
332 | |||
333 | Assert.That(firstDynamicTextureID, Is.Not.EqualTo(so.RootPart.Shape.Textures.GetFace(0).TextureID)); | ||
334 | } | ||
74 | } | 335 | } |
75 | } \ No newline at end of file | 336 | } \ No newline at end of file |
diff --git a/OpenSim/Region/CoreModules/Scripting/VectorRender/VectorRenderModule.cs b/OpenSim/Region/CoreModules/Scripting/VectorRender/VectorRenderModule.cs index 8b2f2f8..1e17b02 100644 --- a/OpenSim/Region/CoreModules/Scripting/VectorRender/VectorRenderModule.cs +++ b/OpenSim/Region/CoreModules/Scripting/VectorRender/VectorRenderModule.cs | |||
@@ -30,10 +30,12 @@ using System.Drawing; | |||
30 | using System.Drawing.Imaging; | 30 | using System.Drawing.Imaging; |
31 | using System.Globalization; | 31 | using System.Globalization; |
32 | using System.IO; | 32 | using System.IO; |
33 | using System.Linq; | ||
33 | using System.Net; | 34 | using System.Net; |
34 | using Nini.Config; | 35 | using Nini.Config; |
35 | using OpenMetaverse; | 36 | using OpenMetaverse; |
36 | using OpenMetaverse.Imaging; | 37 | using OpenMetaverse.Imaging; |
38 | using OpenSim.Region.CoreModules.Scripting.DynamicTexture; | ||
37 | using OpenSim.Region.Framework.Interfaces; | 39 | using OpenSim.Region.Framework.Interfaces; |
38 | using OpenSim.Region.Framework.Scenes; | 40 | using OpenSim.Region.Framework.Scenes; |
39 | using log4net; | 41 | using log4net; |
@@ -45,9 +47,13 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
45 | { | 47 | { |
46 | public class VectorRenderModule : IRegionModule, IDynamicTextureRender | 48 | public class VectorRenderModule : IRegionModule, IDynamicTextureRender |
47 | { | 49 | { |
50 | // These fields exist for testing purposes, please do not remove. | ||
51 | // private static bool s_flipper; | ||
52 | // private static byte[] s_asset1Data; | ||
53 | // private static byte[] s_asset2Data; | ||
54 | |||
48 | private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); | 55 | private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); |
49 | 56 | ||
50 | private string m_name = "VectorRenderModule"; | ||
51 | private Scene m_scene; | 57 | private Scene m_scene; |
52 | private IDynamicTextureManager m_textureManager; | 58 | private IDynamicTextureManager m_textureManager; |
53 | private Graphics m_graph; | 59 | private Graphics m_graph; |
@@ -61,12 +67,12 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
61 | 67 | ||
62 | public string GetContentType() | 68 | public string GetContentType() |
63 | { | 69 | { |
64 | return ("vector"); | 70 | return "vector"; |
65 | } | 71 | } |
66 | 72 | ||
67 | public string GetName() | 73 | public string GetName() |
68 | { | 74 | { |
69 | return m_name; | 75 | return Name; |
70 | } | 76 | } |
71 | 77 | ||
72 | public bool SupportsAsynchronous() | 78 | public bool SupportsAsynchronous() |
@@ -74,14 +80,20 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
74 | return true; | 80 | return true; |
75 | } | 81 | } |
76 | 82 | ||
77 | public byte[] ConvertUrl(string url, string extraParams) | 83 | // public bool AlwaysIdenticalConversion(string bodyData, string extraParams) |
84 | // { | ||
85 | // string[] lines = GetLines(bodyData); | ||
86 | // return lines.Any((str, r) => str.StartsWith("Image")); | ||
87 | // } | ||
88 | |||
89 | public IDynamicTexture ConvertUrl(string url, string extraParams) | ||
78 | { | 90 | { |
79 | return null; | 91 | return null; |
80 | } | 92 | } |
81 | 93 | ||
82 | public byte[] ConvertStream(Stream data, string extraParams) | 94 | public IDynamicTexture ConvertData(string bodyData, string extraParams) |
83 | { | 95 | { |
84 | return null; | 96 | return Draw(bodyData, extraParams); |
85 | } | 97 | } |
86 | 98 | ||
87 | public bool AsyncConvertUrl(UUID id, string url, string extraParams) | 99 | public bool AsyncConvertUrl(UUID id, string url, string extraParams) |
@@ -91,7 +103,9 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
91 | 103 | ||
92 | public bool AsyncConvertData(UUID id, string bodyData, string extraParams) | 104 | public bool AsyncConvertData(UUID id, string bodyData, string extraParams) |
93 | { | 105 | { |
94 | Draw(bodyData, id, extraParams); | 106 | // XXX: This isn't actually being done asynchronously! |
107 | m_textureManager.ReturnData(id, ConvertData(bodyData, extraParams)); | ||
108 | |||
95 | return true; | 109 | return true; |
96 | } | 110 | } |
97 | 111 | ||
@@ -144,6 +158,13 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
144 | { | 158 | { |
145 | m_textureManager.RegisterRender(GetContentType(), this); | 159 | m_textureManager.RegisterRender(GetContentType(), this); |
146 | } | 160 | } |
161 | |||
162 | // This code exists for testing purposes, please do not remove. | ||
163 | // s_asset1Data = m_scene.AssetService.Get("00000000-0000-1111-9999-000000000001").Data; | ||
164 | // s_asset1Data = m_scene.AssetService.Get("9f4acf0d-1841-4e15-bdb8-3a12efc9dd8f").Data; | ||
165 | |||
166 | // Terrain dirt - smallest bin/assets file (6004 bytes) | ||
167 | // s_asset2Data = m_scene.AssetService.Get("b8d3965a-ad78-bf43-699b-bff8eca6c975").Data; | ||
147 | } | 168 | } |
148 | 169 | ||
149 | public void Close() | 170 | public void Close() |
@@ -152,7 +173,7 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
152 | 173 | ||
153 | public string Name | 174 | public string Name |
154 | { | 175 | { |
155 | get { return m_name; } | 176 | get { return "VectorRenderModule"; } |
156 | } | 177 | } |
157 | 178 | ||
158 | public bool IsSharedModule | 179 | public bool IsSharedModule |
@@ -162,7 +183,7 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
162 | 183 | ||
163 | #endregion | 184 | #endregion |
164 | 185 | ||
165 | private void Draw(string data, UUID id, string extraParams) | 186 | private IDynamicTexture Draw(string data, string extraParams) |
166 | { | 187 | { |
167 | // We need to cater for old scripts that didnt use extraParams neatly, they use either an integer size which represents both width and height, or setalpha | 188 | // We need to cater for old scripts that didnt use extraParams neatly, they use either an integer size which represents both width and height, or setalpha |
168 | // we will now support multiple comma seperated params in the form width:256,height:512,alpha:255 | 189 | // we will now support multiple comma seperated params in the form width:256,height:512,alpha:255 |
@@ -305,40 +326,57 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
305 | 326 | ||
306 | Bitmap bitmap = null; | 327 | Bitmap bitmap = null; |
307 | Graphics graph = null; | 328 | Graphics graph = null; |
329 | bool reuseable = false; | ||
308 | 330 | ||
309 | try | 331 | try |
310 | { | 332 | { |
311 | if (alpha == 256) | 333 | // XXX: In testing, it appears that if multiple threads dispose of separate GDI+ objects simultaneously, |
312 | bitmap = new Bitmap(width, height, PixelFormat.Format32bppRgb); | 334 | // the native malloc heap can become corrupted, possibly due to a double free(). This may be due to |
313 | else | 335 | // bugs in the underlying libcairo used by mono's libgdiplus.dll on Linux/OSX. These problems were |
314 | bitmap = new Bitmap(width, height, PixelFormat.Format32bppArgb); | 336 | // seen with both libcario 1.10.2-6.1ubuntu3 and 1.8.10-2ubuntu1. They go away if disposal is perfomed |
315 | 337 | // under lock. | |
316 | graph = Graphics.FromImage(bitmap); | 338 | lock (this) |
317 | |||
318 | // this is really just to save people filling the | ||
319 | // background color in their scripts, only do when fully opaque | ||
320 | if (alpha >= 255) | ||
321 | { | 339 | { |
322 | using (SolidBrush bgFillBrush = new SolidBrush(bgColor)) | 340 | if (alpha == 256) |
341 | bitmap = new Bitmap(width, height, PixelFormat.Format32bppRgb); | ||
342 | else | ||
343 | bitmap = new Bitmap(width, height, PixelFormat.Format32bppArgb); | ||
344 | |||
345 | graph = Graphics.FromImage(bitmap); | ||
346 | |||
347 | // this is really just to save people filling the | ||
348 | // background color in their scripts, only do when fully opaque | ||
349 | if (alpha >= 255) | ||
323 | { | 350 | { |
324 | graph.FillRectangle(bgFillBrush, 0, 0, width, height); | 351 | using (SolidBrush bgFillBrush = new SolidBrush(bgColor)) |
352 | { | ||
353 | graph.FillRectangle(bgFillBrush, 0, 0, width, height); | ||
354 | } | ||
325 | } | 355 | } |
326 | } | 356 | |
327 | 357 | for (int w = 0; w < bitmap.Width; w++) | |
328 | for (int w = 0; w < bitmap.Width; w++) | ||
329 | { | ||
330 | if (alpha <= 255) | ||
331 | { | 358 | { |
332 | for (int h = 0; h < bitmap.Height; h++) | 359 | if (alpha <= 255) |
333 | { | 360 | { |
334 | bitmap.SetPixel(w, h, Color.FromArgb(alpha, bitmap.GetPixel(w, h))); | 361 | for (int h = 0; h < bitmap.Height; h++) |
362 | { | ||
363 | bitmap.SetPixel(w, h, Color.FromArgb(alpha, bitmap.GetPixel(w, h))); | ||
364 | } | ||
335 | } | 365 | } |
336 | } | 366 | } |
367 | |||
368 | GDIDraw(data, graph, altDataDelim, out reuseable); | ||
337 | } | 369 | } |
338 | 370 | ||
339 | GDIDraw(data, graph, altDataDelim); | ||
340 | |||
341 | byte[] imageJ2000 = new byte[0]; | 371 | byte[] imageJ2000 = new byte[0]; |
372 | |||
373 | // This code exists for testing purposes, please do not remove. | ||
374 | // if (s_flipper) | ||
375 | // imageJ2000 = s_asset1Data; | ||
376 | // else | ||
377 | // imageJ2000 = s_asset2Data; | ||
378 | // | ||
379 | // s_flipper = !s_flipper; | ||
342 | 380 | ||
343 | try | 381 | try |
344 | { | 382 | { |
@@ -351,15 +389,24 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
351 | e.Message, e.StackTrace); | 389 | e.Message, e.StackTrace); |
352 | } | 390 | } |
353 | 391 | ||
354 | m_textureManager.ReturnData(id, imageJ2000); | 392 | return new OpenSim.Region.CoreModules.Scripting.DynamicTexture.DynamicTexture( |
393 | data, extraParams, imageJ2000, new Size(width, height), reuseable); | ||
355 | } | 394 | } |
356 | finally | 395 | finally |
357 | { | 396 | { |
358 | if (graph != null) | 397 | // XXX: In testing, it appears that if multiple threads dispose of separate GDI+ objects simultaneously, |
359 | graph.Dispose(); | 398 | // the native malloc heap can become corrupted, possibly due to a double free(). This may be due to |
360 | 399 | // bugs in the underlying libcairo used by mono's libgdiplus.dll on Linux/OSX. These problems were | |
361 | if (bitmap != null) | 400 | // seen with both libcario 1.10.2-6.1ubuntu3 and 1.8.10-2ubuntu1. They go away if disposal is perfomed |
362 | bitmap.Dispose(); | 401 | // under lock. |
402 | lock (this) | ||
403 | { | ||
404 | if (graph != null) | ||
405 | graph.Dispose(); | ||
406 | |||
407 | if (bitmap != null) | ||
408 | bitmap.Dispose(); | ||
409 | } | ||
363 | } | 410 | } |
364 | } | 411 | } |
365 | 412 | ||
@@ -418,8 +465,21 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
418 | } | 465 | } |
419 | */ | 466 | */ |
420 | 467 | ||
421 | private void GDIDraw(string data, Graphics graph, char dataDelim) | 468 | /// <summary> |
469 | /// Split input data into discrete command lines. | ||
470 | /// </summary> | ||
471 | /// <returns></returns> | ||
472 | /// <param name='data'></param> | ||
473 | /// <param name='dataDelim'></param> | ||
474 | private string[] GetLines(string data, char dataDelim) | ||
475 | { | ||
476 | char[] lineDelimiter = { dataDelim }; | ||
477 | return data.Split(lineDelimiter); | ||
478 | } | ||
479 | |||
480 | private void GDIDraw(string data, Graphics graph, char dataDelim, out bool reuseable) | ||
422 | { | 481 | { |
482 | reuseable = true; | ||
423 | Point startPoint = new Point(0, 0); | 483 | Point startPoint = new Point(0, 0); |
424 | Point endPoint = new Point(0, 0); | 484 | Point endPoint = new Point(0, 0); |
425 | Pen drawPen = null; | 485 | Pen drawPen = null; |
@@ -434,11 +494,9 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
434 | myFont = new Font(fontName, fontSize); | 494 | myFont = new Font(fontName, fontSize); |
435 | myBrush = new SolidBrush(Color.Black); | 495 | myBrush = new SolidBrush(Color.Black); |
436 | 496 | ||
437 | char[] lineDelimiter = {dataDelim}; | ||
438 | char[] partsDelimiter = {','}; | 497 | char[] partsDelimiter = {','}; |
439 | string[] lines = data.Split(lineDelimiter); | ||
440 | 498 | ||
441 | foreach (string line in lines) | 499 | foreach (string line in GetLines(data, dataDelim)) |
442 | { | 500 | { |
443 | string nextLine = line.Trim(); | 501 | string nextLine = line.Trim(); |
444 | //replace with switch, or even better, do some proper parsing | 502 | //replace with switch, or even better, do some proper parsing |
@@ -469,6 +527,10 @@ namespace OpenSim.Region.CoreModules.Scripting.VectorRender | |||
469 | } | 527 | } |
470 | else if (nextLine.StartsWith("Image")) | 528 | else if (nextLine.StartsWith("Image")) |
471 | { | 529 | { |
530 | // We cannot reuse any generated texture involving fetching an image via HTTP since that image | ||
531 | // can change. | ||
532 | reuseable = false; | ||
533 | |||
472 | float x = 0; | 534 | float x = 0; |
473 | float y = 0; | 535 | float y = 0; |
474 | GetParams(partsDelimiter, ref nextLine, 5, ref x, ref y); | 536 | GetParams(partsDelimiter, ref nextLine, 5, ref x, ref y); |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsOut/Grid/Tests/GridConnectorsTests.cs b/OpenSim/Region/CoreModules/ServiceConnectorsOut/Grid/Tests/GridConnectorsTests.cs index b286d17..57ae549 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsOut/Grid/Tests/GridConnectorsTests.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsOut/Grid/Tests/GridConnectorsTests.cs | |||
@@ -43,7 +43,7 @@ using OpenSim.Tests.Common; | |||
43 | namespace OpenSim.Region.CoreModules.ServiceConnectorsOut.Grid.Tests | 43 | namespace OpenSim.Region.CoreModules.ServiceConnectorsOut.Grid.Tests |
44 | { | 44 | { |
45 | [TestFixture] | 45 | [TestFixture] |
46 | public class GridConnectorsTests | 46 | public class GridConnectorsTests : OpenSimTestCase |
47 | { | 47 | { |
48 | LocalGridServicesConnector m_LocalConnector; | 48 | LocalGridServicesConnector m_LocalConnector; |
49 | private void SetUp() | 49 | private void SetUp() |
diff --git a/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/LocalSimulationConnector.cs b/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/LocalSimulationConnector.cs index 6eb99ea..8ed1833 100644 --- a/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/LocalSimulationConnector.cs +++ b/OpenSim/Region/CoreModules/ServiceConnectorsOut/Simulation/LocalSimulationConnector.cs | |||
@@ -313,7 +313,11 @@ namespace OpenSim.Region.CoreModules.ServiceConnectorsOut.Simulation | |||
313 | 313 | ||
314 | if (m_scenes.ContainsKey(destination.RegionID)) | 314 | if (m_scenes.ContainsKey(destination.RegionID)) |
315 | { | 315 | { |
316 | Util.FireAndForget(delegate { m_scenes[destination.RegionID].IncomingCloseAgent(id); }); | 316 | // m_log.DebugFormat( |
317 | // "[LOCAL SIMULATION CONNECTOR]: Found region {0} {1} to send AgentUpdate", | ||
318 | // s.RegionInfo.RegionName, destination.RegionHandle); | ||
319 | |||
320 | Util.FireAndForget(delegate { m_scenes[destination.RegionID].IncomingCloseAgent(id, false); }); | ||
317 | return true; | 321 | return true; |
318 | } | 322 | } |
319 | //m_log.Debug("[LOCAL COMMS]: region not found in SendCloseAgent"); | 323 | //m_log.Debug("[LOCAL COMMS]: region not found in SendCloseAgent"); |
diff --git a/OpenSim/Region/CoreModules/World/Archiver/ArchiveReadRequest.cs b/OpenSim/Region/CoreModules/World/Archiver/ArchiveReadRequest.cs index 619550c..142567b 100644 --- a/OpenSim/Region/CoreModules/World/Archiver/ArchiveReadRequest.cs +++ b/OpenSim/Region/CoreModules/World/Archiver/ArchiveReadRequest.cs | |||
@@ -97,6 +97,13 @@ namespace OpenSim.Region.CoreModules.World.Archiver | |||
97 | } | 97 | } |
98 | } | 98 | } |
99 | 99 | ||
100 | /// <summary> | ||
101 | /// Used to cache lookups for valid groups. | ||
102 | /// </summary> | ||
103 | private IDictionary<UUID, bool> m_validGroupUuids = new Dictionary<UUID, bool>(); | ||
104 | |||
105 | private IGroupsModule m_groupsModule; | ||
106 | |||
100 | public ArchiveReadRequest(Scene scene, string loadPath, bool merge, bool skipAssets, Guid requestId) | 107 | public ArchiveReadRequest(Scene scene, string loadPath, bool merge, bool skipAssets, Guid requestId) |
101 | { | 108 | { |
102 | m_scene = scene; | 109 | m_scene = scene; |
@@ -120,6 +127,8 @@ namespace OpenSim.Region.CoreModules.World.Archiver | |||
120 | 127 | ||
121 | // Zero can never be a valid user id | 128 | // Zero can never be a valid user id |
122 | m_validUserUuids[UUID.Zero] = false; | 129 | m_validUserUuids[UUID.Zero] = false; |
130 | |||
131 | m_groupsModule = m_scene.RequestModuleInterface<IGroupsModule>(); | ||
123 | } | 132 | } |
124 | 133 | ||
125 | public ArchiveReadRequest(Scene scene, Stream loadStream, bool merge, bool skipAssets, Guid requestId) | 134 | public ArchiveReadRequest(Scene scene, Stream loadStream, bool merge, bool skipAssets, Guid requestId) |
@@ -132,6 +141,8 @@ namespace OpenSim.Region.CoreModules.World.Archiver | |||
132 | 141 | ||
133 | // Zero can never be a valid user id | 142 | // Zero can never be a valid user id |
134 | m_validUserUuids[UUID.Zero] = false; | 143 | m_validUserUuids[UUID.Zero] = false; |
144 | |||
145 | m_groupsModule = m_scene.RequestModuleInterface<IGroupsModule>(); | ||
135 | } | 146 | } |
136 | 147 | ||
137 | /// <summary> | 148 | /// <summary> |
@@ -302,6 +313,9 @@ namespace OpenSim.Region.CoreModules.World.Archiver | |||
302 | if (!ResolveUserUuid(part.LastOwnerID)) | 313 | if (!ResolveUserUuid(part.LastOwnerID)) |
303 | part.LastOwnerID = m_scene.RegionInfo.EstateSettings.EstateOwner; | 314 | part.LastOwnerID = m_scene.RegionInfo.EstateSettings.EstateOwner; |
304 | 315 | ||
316 | if (!ResolveGroupUuid(part.GroupID)) | ||
317 | part.GroupID = UUID.Zero; | ||
318 | |||
305 | // And zap any troublesome sit target information | 319 | // And zap any troublesome sit target information |
306 | // part.SitTargetOrientation = new Quaternion(0, 0, 0, 1); | 320 | // part.SitTargetOrientation = new Quaternion(0, 0, 0, 1); |
307 | // part.SitTargetPosition = new Vector3(0, 0, 0); | 321 | // part.SitTargetPosition = new Vector3(0, 0, 0); |
@@ -335,13 +349,18 @@ namespace OpenSim.Region.CoreModules.World.Archiver | |||
335 | { | 349 | { |
336 | kvp.Value.OwnerID = m_scene.RegionInfo.EstateSettings.EstateOwner; | 350 | kvp.Value.OwnerID = m_scene.RegionInfo.EstateSettings.EstateOwner; |
337 | } | 351 | } |
352 | |||
338 | if (kvp.Value.CreatorData == null || kvp.Value.CreatorData == string.Empty) | 353 | if (kvp.Value.CreatorData == null || kvp.Value.CreatorData == string.Empty) |
339 | { | 354 | { |
340 | if (!ResolveUserUuid(kvp.Value.CreatorID)) | 355 | if (!ResolveUserUuid(kvp.Value.CreatorID)) |
341 | kvp.Value.CreatorID = m_scene.RegionInfo.EstateSettings.EstateOwner; | 356 | kvp.Value.CreatorID = m_scene.RegionInfo.EstateSettings.EstateOwner; |
342 | } | 357 | } |
358 | |||
343 | if (UserManager != null) | 359 | if (UserManager != null) |
344 | UserManager.AddUser(kvp.Value.CreatorID, kvp.Value.CreatorData); | 360 | UserManager.AddUser(kvp.Value.CreatorID, kvp.Value.CreatorData); |
361 | |||
362 | if (!ResolveGroupUuid(kvp.Value.GroupID)) | ||
363 | kvp.Value.GroupID = UUID.Zero; | ||
345 | } | 364 | } |
346 | part.TaskInventory.LockItemsForRead(false); | 365 | part.TaskInventory.LockItemsForRead(false); |
347 | } | 366 | } |
@@ -382,9 +401,27 @@ namespace OpenSim.Region.CoreModules.World.Archiver | |||
382 | foreach (string serialisedParcel in serialisedParcels) | 401 | foreach (string serialisedParcel in serialisedParcels) |
383 | { | 402 | { |
384 | LandData parcel = LandDataSerializer.Deserialize(serialisedParcel); | 403 | LandData parcel = LandDataSerializer.Deserialize(serialisedParcel); |
404 | |||
405 | // Validate User and Group UUID's | ||
406 | |||
385 | if (!ResolveUserUuid(parcel.OwnerID)) | 407 | if (!ResolveUserUuid(parcel.OwnerID)) |
386 | parcel.OwnerID = m_scene.RegionInfo.EstateSettings.EstateOwner; | 408 | parcel.OwnerID = m_scene.RegionInfo.EstateSettings.EstateOwner; |
387 | 409 | ||
410 | if (!ResolveGroupUuid(parcel.GroupID)) | ||
411 | { | ||
412 | parcel.GroupID = UUID.Zero; | ||
413 | parcel.IsGroupOwned = false; | ||
414 | } | ||
415 | |||
416 | List<LandAccessEntry> accessList = new List<LandAccessEntry>(); | ||
417 | foreach (LandAccessEntry entry in parcel.ParcelAccessList) | ||
418 | { | ||
419 | if (ResolveUserUuid(entry.AgentID)) | ||
420 | accessList.Add(entry); | ||
421 | // else, drop this access rule | ||
422 | } | ||
423 | parcel.ParcelAccessList = accessList; | ||
424 | |||
388 | // m_log.DebugFormat( | 425 | // m_log.DebugFormat( |
389 | // "[ARCHIVER]: Adding parcel {0}, local id {1}, area {2}", | 426 | // "[ARCHIVER]: Adding parcel {0}, local id {1}, area {2}", |
390 | // parcel.Name, parcel.LocalID, parcel.Area); | 427 | // parcel.Name, parcel.LocalID, parcel.Area); |
@@ -419,6 +456,30 @@ namespace OpenSim.Region.CoreModules.World.Archiver | |||
419 | } | 456 | } |
420 | 457 | ||
421 | /// <summary> | 458 | /// <summary> |
459 | /// Look up the given group id to check whether it's one that is valid for this grid. | ||
460 | /// </summary> | ||
461 | /// <param name="uuid"></param> | ||
462 | /// <returns></returns> | ||
463 | private bool ResolveGroupUuid(UUID uuid) | ||
464 | { | ||
465 | if (uuid == UUID.Zero) | ||
466 | return true; // this means the object has no group | ||
467 | |||
468 | if (!m_validGroupUuids.ContainsKey(uuid)) | ||
469 | { | ||
470 | bool exists; | ||
471 | |||
472 | if (m_groupsModule == null) | ||
473 | exists = false; | ||
474 | else | ||
475 | exists = (m_groupsModule.GetGroupRecord(uuid) != null); | ||
476 | |||
477 | m_validGroupUuids.Add(uuid, exists); | ||
478 | } | ||
479 | |||
480 | return m_validGroupUuids[uuid]; | ||
481 | } | ||
482 | |||
422 | /// Load an asset | 483 | /// Load an asset |
423 | /// </summary> | 484 | /// </summary> |
424 | /// <param name="assetFilename"></param> | 485 | /// <param name="assetFilename"></param> |
diff --git a/OpenSim/Region/CoreModules/World/Archiver/Tests/ArchiverTests.cs b/OpenSim/Region/CoreModules/World/Archiver/Tests/ArchiverTests.cs index 5deaf52..904110e 100644 --- a/OpenSim/Region/CoreModules/World/Archiver/Tests/ArchiverTests.cs +++ b/OpenSim/Region/CoreModules/World/Archiver/Tests/ArchiverTests.cs | |||
@@ -51,7 +51,7 @@ using RegionSettings = OpenSim.Framework.RegionSettings; | |||
51 | namespace OpenSim.Region.CoreModules.World.Archiver.Tests | 51 | namespace OpenSim.Region.CoreModules.World.Archiver.Tests |
52 | { | 52 | { |
53 | [TestFixture] | 53 | [TestFixture] |
54 | public class ArchiverTests | 54 | public class ArchiverTests : OpenSimTestCase |
55 | { | 55 | { |
56 | private Guid m_lastRequestId; | 56 | private Guid m_lastRequestId; |
57 | private string m_lastErrorMessage; | 57 | private string m_lastErrorMessage; |
diff --git a/OpenSim/Region/CoreModules/World/Sound/SoundModule.cs b/OpenSim/Region/CoreModules/World/Sound/SoundModule.cs index 14c1a39..a2f0950 100644 --- a/OpenSim/Region/CoreModules/World/Sound/SoundModule.cs +++ b/OpenSim/Region/CoreModules/World/Sound/SoundModule.cs | |||
@@ -85,13 +85,15 @@ namespace OpenSim.Region.CoreModules.World.Sound | |||
85 | dis = 0; | 85 | dis = 0; |
86 | } | 86 | } |
87 | 87 | ||
88 | float thisSpGain; | ||
89 | |||
88 | // Scale by distance | 90 | // Scale by distance |
89 | if (radius == 0) | 91 | if (radius == 0) |
90 | gain = (float)((double)gain * ((100.0 - dis) / 100.0)); | 92 | thisSpGain = (float)((double)gain * ((100.0 - dis) / 100.0)); |
91 | else | 93 | else |
92 | gain = (float)((double)gain * ((radius - dis) / radius)); | 94 | thisSpGain = (float)((double)gain * ((radius - dis) / radius)); |
93 | 95 | ||
94 | sp.ControllingClient.SendPlayAttachedSound(soundID, objectID, ownerID, (float)gain, flags); | 96 | sp.ControllingClient.SendPlayAttachedSound(soundID, objectID, ownerID, thisSpGain, flags); |
95 | }); | 97 | }); |
96 | } | 98 | } |
97 | 99 | ||
diff --git a/OpenSim/Region/CoreModules/World/Terrain/TerrainModule.cs b/OpenSim/Region/CoreModules/World/Terrain/TerrainModule.cs index 402b9fb..d99567c 100644 --- a/OpenSim/Region/CoreModules/World/Terrain/TerrainModule.cs +++ b/OpenSim/Region/CoreModules/World/Terrain/TerrainModule.cs | |||
@@ -414,6 +414,7 @@ namespace OpenSim.Region.CoreModules.World.Terrain | |||
414 | private void LoadPlugins() | 414 | private void LoadPlugins() |
415 | { | 415 | { |
416 | m_plugineffects = new Dictionary<string, ITerrainEffect>(); | 416 | m_plugineffects = new Dictionary<string, ITerrainEffect>(); |
417 | LoadPlugins(Assembly.GetCallingAssembly()); | ||
417 | string plugineffectsPath = "Terrain"; | 418 | string plugineffectsPath = "Terrain"; |
418 | 419 | ||
419 | // Load the files in the Terrain/ dir | 420 | // Load the files in the Terrain/ dir |
@@ -427,34 +428,39 @@ namespace OpenSim.Region.CoreModules.World.Terrain | |||
427 | try | 428 | try |
428 | { | 429 | { |
429 | Assembly library = Assembly.LoadFrom(file); | 430 | Assembly library = Assembly.LoadFrom(file); |
430 | foreach (Type pluginType in library.GetTypes()) | 431 | LoadPlugins(library); |
431 | { | 432 | } |
432 | try | 433 | catch (BadImageFormatException) |
433 | { | 434 | { |
434 | if (pluginType.IsAbstract || pluginType.IsNotPublic) | 435 | } |
435 | continue; | 436 | } |
437 | } | ||
436 | 438 | ||
437 | string typeName = pluginType.Name; | 439 | private void LoadPlugins(Assembly library) |
440 | { | ||
441 | foreach (Type pluginType in library.GetTypes()) | ||
442 | { | ||
443 | try | ||
444 | { | ||
445 | if (pluginType.IsAbstract || pluginType.IsNotPublic) | ||
446 | continue; | ||
438 | 447 | ||
439 | if (pluginType.GetInterface("ITerrainEffect", false) != null) | 448 | string typeName = pluginType.Name; |
440 | { | ||
441 | ITerrainEffect terEffect = (ITerrainEffect) Activator.CreateInstance(library.GetType(pluginType.ToString())); | ||
442 | 449 | ||
443 | InstallPlugin(typeName, terEffect); | 450 | if (pluginType.GetInterface("ITerrainEffect", false) != null) |
444 | } | 451 | { |
445 | else if (pluginType.GetInterface("ITerrainLoader", false) != null) | 452 | ITerrainEffect terEffect = (ITerrainEffect)Activator.CreateInstance(library.GetType(pluginType.ToString())); |
446 | { | 453 | |
447 | ITerrainLoader terLoader = (ITerrainLoader) Activator.CreateInstance(library.GetType(pluginType.ToString())); | 454 | InstallPlugin(typeName, terEffect); |
448 | m_loaders[terLoader.FileExtension] = terLoader; | 455 | } |
449 | m_log.Info("L ... " + typeName); | 456 | else if (pluginType.GetInterface("ITerrainLoader", false) != null) |
450 | } | 457 | { |
451 | } | 458 | ITerrainLoader terLoader = (ITerrainLoader)Activator.CreateInstance(library.GetType(pluginType.ToString())); |
452 | catch (AmbiguousMatchException) | 459 | m_loaders[terLoader.FileExtension] = terLoader; |
453 | { | 460 | m_log.Info("L ... " + typeName); |
454 | } | ||
455 | } | 461 | } |
456 | } | 462 | } |
457 | catch (BadImageFormatException) | 463 | catch (AmbiguousMatchException) |
458 | { | 464 | { |
459 | } | 465 | } |
460 | } | 466 | } |
@@ -1178,7 +1184,8 @@ namespace OpenSim.Region.CoreModules.World.Terrain | |||
1178 | 1184 | ||
1179 | private void InterfaceRunPluginEffect(Object[] args) | 1185 | private void InterfaceRunPluginEffect(Object[] args) |
1180 | { | 1186 | { |
1181 | if ((string) args[0] == "list") | 1187 | string firstArg = (string)args[0]; |
1188 | if (firstArg == "list") | ||
1182 | { | 1189 | { |
1183 | m_log.Info("List of loaded plugins"); | 1190 | m_log.Info("List of loaded plugins"); |
1184 | foreach (KeyValuePair<string, ITerrainEffect> kvp in m_plugineffects) | 1191 | foreach (KeyValuePair<string, ITerrainEffect> kvp in m_plugineffects) |
@@ -1187,14 +1194,14 @@ namespace OpenSim.Region.CoreModules.World.Terrain | |||
1187 | } | 1194 | } |
1188 | return; | 1195 | return; |
1189 | } | 1196 | } |
1190 | if ((string) args[0] == "reload") | 1197 | if (firstArg == "reload") |
1191 | { | 1198 | { |
1192 | LoadPlugins(); | 1199 | LoadPlugins(); |
1193 | return; | 1200 | return; |
1194 | } | 1201 | } |
1195 | if (m_plugineffects.ContainsKey((string) args[0])) | 1202 | if (m_plugineffects.ContainsKey(firstArg)) |
1196 | { | 1203 | { |
1197 | m_plugineffects[(string) args[0]].RunEffect(m_channel); | 1204 | m_plugineffects[firstArg].RunEffect(m_channel); |
1198 | CheckForTerrainUpdates(); | 1205 | CheckForTerrainUpdates(); |
1199 | } | 1206 | } |
1200 | else | 1207 | else |
diff --git a/OpenSim/Region/CoreModules/World/Warp3DMap/Warp3DImageModule.cs b/OpenSim/Region/CoreModules/World/Warp3DMap/Warp3DImageModule.cs index 3c48d07..33f6c3f 100644 --- a/OpenSim/Region/CoreModules/World/Warp3DMap/Warp3DImageModule.cs +++ b/OpenSim/Region/CoreModules/World/Warp3DMap/Warp3DImageModule.cs | |||
@@ -222,6 +222,13 @@ namespace OpenSim.Region.CoreModules.World.Warp3DMap | |||
222 | bitmap = ImageUtils.ResizeImage(origBitmap, viewport.Width, viewport.Height); | 222 | bitmap = ImageUtils.ResizeImage(origBitmap, viewport.Width, viewport.Height); |
223 | } | 223 | } |
224 | 224 | ||
225 | // XXX: It shouldn't really be necesary to force a GC here as one should occur anyway pretty shortly | ||
226 | // afterwards. It's generally regarded as a bad idea to manually GC. If Warp3D is using lots of memory | ||
227 | // then this may be some issue with the Warp3D code itself, though it's also quite possible that generating | ||
228 | // this map tile simply takes a lot of memory. | ||
229 | GC.Collect(); | ||
230 | m_log.Debug("[WARP 3D IMAGE MODULE]: GC.Collect()"); | ||
231 | |||
225 | return bitmap; | 232 | return bitmap; |
226 | } | 233 | } |
227 | 234 | ||