From 87b337ebf9f36ca3b2af94631ec9a235434877ac Mon Sep 17 00:00:00 2001
From: MW
Date: Fri, 7 Dec 2007 22:47:11 +0000
Subject: Enabled the TextureDownloadModule, so that hopefully I might get some
 feedback, as to if it makes the texture problem better or worse. As I
 plan/hope to work on texture/asset downloading this weekend.

---
 OpenSim/Region/ClientStack/ClientView.cs           | 26 +++++++++---------
 .../Environment/Modules/AvatarFactoryModule.cs     |  2 +-
 .../Environment/Modules/TextureDownloadModule.cs   | 31 +++++++++++-----------
 .../Region/Environment/Scenes/AvatarAppearance.cs  |  5 ----
 4 files changed, 30 insertions(+), 34 deletions(-)

(limited to 'OpenSim/Region')

diff --git a/OpenSim/Region/ClientStack/ClientView.cs b/OpenSim/Region/ClientStack/ClientView.cs
index 2ec4fef..dc229f3 100644
--- a/OpenSim/Region/ClientStack/ClientView.cs
+++ b/OpenSim/Region/ClientStack/ClientView.cs
@@ -2728,19 +2728,19 @@ namespace OpenSim.Region.ClientStack
                         for (int i = 0; i < imageRequest.RequestImage.Length; i++)
                         {
                             // still working on the Texture download module so for now using old method
-                            //  TextureRequestArgs args = new TextureRequestArgs();
-                            //  args.RequestedAssetID = imageRequest.RequestImage[i].Image;
-                            //  args.DiscardLevel = imageRequest.RequestImage[i].DiscardLevel;
-                            //  args.PacketNumber = imageRequest.RequestImage[i].Packet;
-
-                            //  if (OnRequestTexture != null)
-                            //  {
-                            //      OnRequestTexture(this, args);
-                            //  }
-
-                            m_assetCache.AddTextureRequest(this, imageRequest.RequestImage[i].Image,
-                                                           imageRequest.RequestImage[i].Packet,
-                                                           imageRequest.RequestImage[i].DiscardLevel);
+                            if (OnRequestTexture != null)
+                            {
+                                TextureRequestArgs args = new TextureRequestArgs();
+                                args.RequestedAssetID = imageRequest.RequestImage[i].Image;
+                                args.DiscardLevel = imageRequest.RequestImage[i].DiscardLevel;
+                                args.PacketNumber = imageRequest.RequestImage[i].Packet;
+
+                                OnRequestTexture(this, args);
+                            }
+
+                           // m_assetCache.AddTextureRequest(this, imageRequest.RequestImage[i].Image,
+                            //                               imageRequest.RequestImage[i].Packet,
+                            //                               imageRequest.RequestImage[i].DiscardLevel);
                         }
                         break;
                     case PacketType.TransferRequest:
diff --git a/OpenSim/Region/Environment/Modules/AvatarFactoryModule.cs b/OpenSim/Region/Environment/Modules/AvatarFactoryModule.cs
index 4f9aa2a..9ed8c0b 100644
--- a/OpenSim/Region/Environment/Modules/AvatarFactoryModule.cs
+++ b/OpenSim/Region/Environment/Modules/AvatarFactoryModule.cs
@@ -80,7 +80,7 @@ namespace OpenSim.Region.Environment.Modules
             {
                 if (profile.RootFolder != null)
                 {
-                    //Todo look up the assetid from the inventory cache (or something) for each itemId that is in AvatarWearingArgs
+                    //Todo look up the assetid from the inventory cache for each itemId that is in AvatarWearingArgs
                     // then store assetid and itemId and wearable type in a database
                     foreach (AvatarWearingArgs.Wearable wear in e.NowWearing)
                     {
diff --git a/OpenSim/Region/Environment/Modules/TextureDownloadModule.cs b/OpenSim/Region/Environment/Modules/TextureDownloadModule.cs
index 902b669..66cdec7 100644
--- a/OpenSim/Region/Environment/Modules/TextureDownloadModule.cs
+++ b/OpenSim/Region/Environment/Modules/TextureDownloadModule.cs
@@ -27,6 +27,7 @@
 */
 using System;
 using System.Collections.Generic;
+using System.Threading;
 using libsecondlife;
 using libsecondlife.Packets;
 using Nini.Config;
@@ -50,13 +51,13 @@ namespace OpenSim.Region.Environment.Modules
 
         private BlockingQueue<TextureSender> QueueSenders = new BlockingQueue<TextureSender>();
         private Dictionary<LLUUID, List<LLUUID>> InProcess = new Dictionary<LLUUID, List<LLUUID>>();
-        // private Thread m_thread;
+        private Thread m_thread;
 
         public TextureDownloadModule()
         {
-            //  m_thread = new Thread(new ThreadStart(ProcessTextureSenders));
-            //  m_thread.IsBackground = true;
-            //  m_thread.Start();
+            m_thread = new Thread(new ThreadStart(ProcessTextureSenders));
+            m_thread.IsBackground = true;
+            m_thread.Start();
         }
 
         public void Initialise(Scene scene, IConfigSource config)
@@ -89,7 +90,7 @@ namespace OpenSim.Region.Environment.Modules
 
         public void NewClient(IClientAPI client)
         {
-            /* lock (ClientRequests)
+            lock (ClientRequests)
             {
                 if (!ClientRequests.ContainsKey(client.AgentId))
                 {
@@ -98,7 +99,7 @@ namespace OpenSim.Region.Environment.Modules
                 }
             }
             client.OnRequestTexture += TextureRequest;
-            */
+
         }
 
         public void TextureCallback(LLUUID textureID, AssetBase asset)
@@ -124,7 +125,7 @@ namespace OpenSim.Region.Environment.Modules
 
         public void TextureRequest(Object sender, TextureRequestArgs e)
         {
-            IClientAPI client = (IClientAPI) sender;
+            IClientAPI client = (IClientAPI)sender;
             if (!ClientRequests[client.AgentId].ContainsKey(e.RequestedAssetID))
             {
                 lock (ClientRequests)
@@ -177,14 +178,14 @@ namespace OpenSim.Region.Environment.Modules
 
                 if (asset.Data.LongLength > 600)
                 {
-                    NumPackets = 2 + (int) (asset.Data.Length - 601)/1000;
+                    NumPackets = 2 + (int)(asset.Data.Length - 601) / 1000;
                 }
                 else
                 {
                     NumPackets = 1;
                 }
 
-                PacketCounter = (int) req.PacketNumber;
+                PacketCounter = (int)req.PacketNumber;
             }
 
             public bool SendTexture()
@@ -209,7 +210,7 @@ namespace OpenSim.Region.Environment.Modules
                         im.Header.Reliable = false;
                         im.ImageID.Packets = 1;
                         im.ImageID.ID = m_asset.FullID;
-                        im.ImageID.Size = (uint) m_asset.Data.Length;
+                        im.ImageID.Size = (uint)m_asset.Data.Length;
                         im.ImageData.Data = m_asset.Data;
                         im.ImageID.Codec = 2;
                         req.RequestUser.OutPacket(im, ThrottleOutPacketType.Texture);
@@ -219,9 +220,9 @@ namespace OpenSim.Region.Environment.Modules
                     {
                         ImageDataPacket im = new ImageDataPacket();
                         im.Header.Reliable = false;
-                        im.ImageID.Packets = (ushort) (NumPackets);
+                        im.ImageID.Packets = (ushort)(NumPackets);
                         im.ImageID.ID = m_asset.FullID;
-                        im.ImageID.Size = (uint) m_asset.Data.Length;
+                        im.ImageID.Size = (uint)m_asset.Data.Length;
                         im.ImageData.Data = new byte[600];
                         Array.Copy(m_asset.Data, 0, im.ImageData.Data, 0, 600);
                         im.ImageID.Codec = 2;
@@ -233,12 +234,12 @@ namespace OpenSim.Region.Environment.Modules
                 {
                     ImagePacketPacket im = new ImagePacketPacket();
                     im.Header.Reliable = false;
-                    im.ImageID.Packet = (ushort) (PacketCounter);
+                    im.ImageID.Packet = (ushort)(PacketCounter);
                     im.ImageID.ID = m_asset.FullID;
-                    int size = m_asset.Data.Length - 600 - (1000*(PacketCounter - 1));
+                    int size = m_asset.Data.Length - 600 - (1000 * (PacketCounter - 1));
                     if (size > 1000) size = 1000;
                     im.ImageData.Data = new byte[size];
-                    Array.Copy(m_asset.Data, 600 + (1000*(PacketCounter - 1)), im.ImageData.Data, 0, size);
+                    Array.Copy(m_asset.Data, 600 + (1000 * (PacketCounter - 1)), im.ImageData.Data, 0, size);
                     req.RequestUser.OutPacket(im, ThrottleOutPacketType.Texture);
                     PacketCounter++;
                 }
diff --git a/OpenSim/Region/Environment/Scenes/AvatarAppearance.cs b/OpenSim/Region/Environment/Scenes/AvatarAppearance.cs
index d8e69e6..84f635d 100644
--- a/OpenSim/Region/Environment/Scenes/AvatarAppearance.cs
+++ b/OpenSim/Region/Environment/Scenes/AvatarAppearance.cs
@@ -61,11 +61,6 @@ namespace OpenSim.Region.Environment.Scenes
         }
 
 
-        public void SetID(LLUUID avatarID)
-        {
-            m_scenePresenceID = avatarID;
-        }
-
         /// <summary>
         /// 
         /// </summary>
-- 
cgit v1.1