diff options
author | John Hurliman | 2009-10-02 13:08:15 -0700 |
---|---|---|
committer | John Hurliman | 2009-10-02 13:08:15 -0700 |
commit | bdc13455011c9de2877813138a49d83fc52d3142 (patch) | |
tree | 860e7015731f3e3148c158817af1b6ce4642ab88 /OpenSim/Region/ClientStack/LindenUDP/LLImageManager.cs | |
parent | * Changed the flush logic to drop packets in non-transactional streams, and t... (diff) | |
download | opensim-SC-bdc13455011c9de2877813138a49d83fc52d3142.zip opensim-SC-bdc13455011c9de2877813138a49d83fc52d3142.tar.gz opensim-SC-bdc13455011c9de2877813138a49d83fc52d3142.tar.bz2 opensim-SC-bdc13455011c9de2877813138a49d83fc52d3142.tar.xz |
* Removed even more unnecessary state variables from J2KImage
* Fixed the incorrect "out of memory?" exception messages and replaced them with useful information
* Cleaned up J2KImage and LLImageManager to conform to coding conventions
Diffstat (limited to '')
-rw-r--r-- | OpenSim/Region/ClientStack/LindenUDP/LLImageManager.cs | 46 |
1 files changed, 23 insertions, 23 deletions
diff --git a/OpenSim/Region/ClientStack/LindenUDP/LLImageManager.cs b/OpenSim/Region/ClientStack/LindenUDP/LLImageManager.cs index a484fdf..d641b6c 100644 --- a/OpenSim/Region/ClientStack/LindenUDP/LLImageManager.cs +++ b/OpenSim/Region/ClientStack/LindenUDP/LLImageManager.cs | |||
@@ -45,7 +45,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
45 | { | 45 | { |
46 | public int Compare(J2KImage x, J2KImage y) | 46 | public int Compare(J2KImage x, J2KImage y) |
47 | { | 47 | { |
48 | return x.m_requestedPriority.CompareTo(y.m_requestedPriority); | 48 | return x.Priority.CompareTo(y.Priority); |
49 | } | 49 | } |
50 | } | 50 | } |
51 | 51 | ||
@@ -94,7 +94,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
94 | 94 | ||
95 | // Do a linear search for this texture download | 95 | // Do a linear search for this texture download |
96 | lock (m_priorityQueue) | 96 | lock (m_priorityQueue) |
97 | m_priorityQueue.Find(delegate(J2KImage img) { return img.m_requestedUUID == newRequest.RequestedAssetID; }, out imgrequest); | 97 | m_priorityQueue.Find(delegate(J2KImage img) { return img.TextureID == newRequest.RequestedAssetID; }, out imgrequest); |
98 | 98 | ||
99 | if (imgrequest != null) | 99 | if (imgrequest != null) |
100 | { | 100 | { |
@@ -105,7 +105,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
105 | try | 105 | try |
106 | { | 106 | { |
107 | lock (m_priorityQueue) | 107 | lock (m_priorityQueue) |
108 | m_priorityQueue.Delete(imgrequest.m_priorityQueueHandle); | 108 | m_priorityQueue.Delete(imgrequest.PriorityQueueHandle); |
109 | } | 109 | } |
110 | catch (Exception) { } | 110 | catch (Exception) { } |
111 | } | 111 | } |
@@ -116,29 +116,29 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
116 | 116 | ||
117 | //Check the packet sequence to make sure this isn't older than | 117 | //Check the packet sequence to make sure this isn't older than |
118 | //one we've already received | 118 | //one we've already received |
119 | if (newRequest.requestSequence > imgrequest.m_lastSequence) | 119 | if (newRequest.requestSequence > imgrequest.LastSequence) |
120 | { | 120 | { |
121 | //Update the sequence number of the last RequestImage packet | 121 | //Update the sequence number of the last RequestImage packet |
122 | imgrequest.m_lastSequence = newRequest.requestSequence; | 122 | imgrequest.LastSequence = newRequest.requestSequence; |
123 | 123 | ||
124 | //Update the requested discard level | 124 | //Update the requested discard level |
125 | imgrequest.m_requestedDiscardLevel = newRequest.DiscardLevel; | 125 | imgrequest.DiscardLevel = newRequest.DiscardLevel; |
126 | 126 | ||
127 | //Update the requested packet number | 127 | //Update the requested packet number |
128 | imgrequest.m_requestedPacketNumber = newRequest.PacketNumber; | 128 | imgrequest.StartPacket = newRequest.PacketNumber; |
129 | 129 | ||
130 | //Update the requested priority | 130 | //Update the requested priority |
131 | imgrequest.m_requestedPriority = newRequest.Priority; | 131 | imgrequest.Priority = newRequest.Priority; |
132 | try | 132 | try |
133 | { | 133 | { |
134 | lock (m_priorityQueue) | 134 | lock (m_priorityQueue) |
135 | m_priorityQueue.Replace(imgrequest.m_priorityQueueHandle, imgrequest); | 135 | m_priorityQueue.Replace(imgrequest.PriorityQueueHandle, imgrequest); |
136 | } | 136 | } |
137 | catch (Exception) | 137 | catch (Exception) |
138 | { | 138 | { |
139 | imgrequest.m_priorityQueueHandle = null; | 139 | imgrequest.PriorityQueueHandle = null; |
140 | lock (m_priorityQueue) | 140 | lock (m_priorityQueue) |
141 | m_priorityQueue.Add(ref imgrequest.m_priorityQueueHandle, imgrequest); | 141 | m_priorityQueue.Add(ref imgrequest.PriorityQueueHandle, imgrequest); |
142 | } | 142 | } |
143 | 143 | ||
144 | //Run an update | 144 | //Run an update |
@@ -161,29 +161,29 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
161 | imgrequest = new J2KImage(this); | 161 | imgrequest = new J2KImage(this); |
162 | 162 | ||
163 | //Assign our decoder module | 163 | //Assign our decoder module |
164 | imgrequest.m_j2kDecodeModule = m_j2kDecodeModule; | 164 | imgrequest.J2KDecoder = m_j2kDecodeModule; |
165 | 165 | ||
166 | //Assign our asset cache module | 166 | //Assign our asset cache module |
167 | imgrequest.m_assetCache = m_assetCache; | 167 | imgrequest.AssetService = m_assetCache; |
168 | 168 | ||
169 | //Assign the requested discard level | 169 | //Assign the requested discard level |
170 | imgrequest.m_requestedDiscardLevel = newRequest.DiscardLevel; | 170 | imgrequest.DiscardLevel = newRequest.DiscardLevel; |
171 | 171 | ||
172 | //Assign the requested packet number | 172 | //Assign the requested packet number |
173 | imgrequest.m_requestedPacketNumber = newRequest.PacketNumber; | 173 | imgrequest.StartPacket = newRequest.PacketNumber; |
174 | 174 | ||
175 | //Assign the requested priority | 175 | //Assign the requested priority |
176 | imgrequest.m_requestedPriority = newRequest.Priority; | 176 | imgrequest.Priority = newRequest.Priority; |
177 | 177 | ||
178 | //Assign the asset uuid | 178 | //Assign the asset uuid |
179 | imgrequest.m_requestedUUID = newRequest.RequestedAssetID; | 179 | imgrequest.TextureID = newRequest.RequestedAssetID; |
180 | 180 | ||
181 | //Assign the requested priority | 181 | //Assign the requested priority |
182 | imgrequest.m_requestedPriority = newRequest.Priority; | 182 | imgrequest.Priority = newRequest.Priority; |
183 | 183 | ||
184 | //Add this download to the priority queue | 184 | //Add this download to the priority queue |
185 | lock (m_priorityQueue) | 185 | lock (m_priorityQueue) |
186 | m_priorityQueue.Add(ref imgrequest.m_priorityQueueHandle, imgrequest); | 186 | m_priorityQueue.Add(ref imgrequest.PriorityQueueHandle, imgrequest); |
187 | 187 | ||
188 | //Run an update | 188 | //Run an update |
189 | imgrequest.RunUpdate(); | 189 | imgrequest.RunUpdate(); |
@@ -249,12 +249,12 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
249 | lock (m_priorityQueue) | 249 | lock (m_priorityQueue) |
250 | imagereq = m_priorityQueue.FindMax(); | 250 | imagereq = m_priorityQueue.FindMax(); |
251 | 251 | ||
252 | if (imagereq.m_decoded == true) | 252 | if (imagereq.IsDecoded == true) |
253 | { | 253 | { |
254 | // we need to test this here now that we are dropping assets | 254 | // we need to test this here now that we are dropping assets |
255 | if (!imagereq.m_hasasset) | 255 | if (!imagereq.HasAsset) |
256 | { | 256 | { |
257 | m_log.WarnFormat("[LLIMAGE MANAGER]: Re-requesting the image asset {0}", imagereq.m_requestedUUID); | 257 | m_log.WarnFormat("[LLIMAGE MANAGER]: Re-requesting the image asset {0}", imagereq.TextureID); |
258 | imagereq.RunUpdate(); | 258 | imagereq.RunUpdate(); |
259 | continue; | 259 | continue; |
260 | } | 260 | } |
@@ -268,7 +268,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP | |||
268 | try | 268 | try |
269 | { | 269 | { |
270 | lock (m_priorityQueue) | 270 | lock (m_priorityQueue) |
271 | m_priorityQueue.Delete(imagereq.m_priorityQueueHandle); | 271 | m_priorityQueue.Delete(imagereq.PriorityQueueHandle); |
272 | } | 272 | } |
273 | catch (Exception) { } | 273 | catch (Exception) { } |
274 | } | 274 | } |