From 97bc5263de990ce80fc0adfbdb632ba8c5cbb77b Mon Sep 17 00:00:00 2001
From: Robert Adams
Date: Wed, 16 Oct 2013 07:52:30 -0700
Subject: varregion: move the compressed heighmap compression factor from
 Constants into TerrainData. Save compression factor with the terrain blob in
 the database.

---
 OpenSim/Region/Framework/Scenes/TerrainChannel.cs  | 24 ++++++----------------
 .../Region/Framework/Scenes/TerrainCompressor.cs   |  2 +-
 2 files changed, 7 insertions(+), 19 deletions(-)

(limited to 'OpenSim/Region/Framework/Scenes')

diff --git a/OpenSim/Region/Framework/Scenes/TerrainChannel.cs b/OpenSim/Region/Framework/Scenes/TerrainChannel.cs
index 65e890f..6d245cb 100644
--- a/OpenSim/Region/Framework/Scenes/TerrainChannel.cs
+++ b/OpenSim/Region/Framework/Scenes/TerrainChannel.cs
@@ -183,21 +183,6 @@ namespace OpenSim.Region.Framework.Scenes
 
         #endregion
 
-        /*
-        // To save space (especially for large regions), keep the height as a short integer
-        //    that is coded as the float height times the compression factor (usually '100'
-        //    to make for two decimal points).
-        public static short ToCompressedHeight(double pHeight)
-        {
-            return (short)(pHeight * Constants.TerrainCompression);
-        }
-
-        public static float FromCompressedHeight(short pHeight)
-        {
-            return ((float)pHeight) / Constants.TerrainCompression;
-        }
-         */
-
         public TerrainChannel Copy()
         {
             TerrainChannel copy = new TerrainChannel();
@@ -280,13 +265,15 @@ namespace OpenSim.Region.Framework.Scenes
             public int SizeX;
             public int SizeY;
             public int SizeZ;
+            public float CompressionFactor;
             public short[] Map;
-            public TerrainChannelXMLPackage(int pX, int pY, int pZ, short[] pMap)
+            public TerrainChannelXMLPackage(int pX, int pY, int pZ, float pCompressionFactor, short[] pMap)
             {
                 Version = 1;
                 SizeX = pX;
                 SizeY = pY;
                 SizeZ = pZ;
+                CompressionFactor = pCompressionFactor;
                 Map = pMap;
             }
         }
@@ -294,7 +281,8 @@ namespace OpenSim.Region.Framework.Scenes
         // New terrain serialization format that includes the width and length.
         private void ToXml2(XmlWriter xmlWriter)
         {
-            TerrainChannelXMLPackage package = new TerrainChannelXMLPackage(Width, Height, Altitude, m_terrainData.GetCompressedMap());
+            TerrainChannelXMLPackage package = new TerrainChannelXMLPackage(Width, Height, Altitude, m_terrainData.CompressionFactor,
+                                            m_terrainData.GetCompressedMap());
             XmlSerializer serializer = new XmlSerializer(typeof(TerrainChannelXMLPackage));
             serializer.Serialize(xmlWriter, package);
         }
@@ -304,7 +292,7 @@ namespace OpenSim.Region.Framework.Scenes
         {
             XmlSerializer serializer = new XmlSerializer(typeof(TerrainChannelXMLPackage));
             TerrainChannelXMLPackage package = (TerrainChannelXMLPackage)serializer.Deserialize(xmlReader);
-            m_terrainData = new HeightmapTerrainData(package.Map, package.SizeX, package.SizeY, package.SizeZ);
+            m_terrainData = new HeightmapTerrainData(package.Map, package.CompressionFactor, package.SizeX, package.SizeY, package.SizeZ);
         }
 
         // Fill the heightmap with the center bump terrain
diff --git a/OpenSim/Region/Framework/Scenes/TerrainCompressor.cs b/OpenSim/Region/Framework/Scenes/TerrainCompressor.cs
index 511745d..e91c959 100644
--- a/OpenSim/Region/Framework/Scenes/TerrainCompressor.cs
+++ b/OpenSim/Region/Framework/Scenes/TerrainCompressor.cs
@@ -112,7 +112,7 @@ namespace OpenSim.Region.ClientStack.LindenUDP
         // This is an intermediate step in converting terrain into a variable sized heightmap. Some of the
         //     routines (like IClientAPI) only pass the float array of heights around. This entry
         //     converts that legacy representation into the more compact represenation used in
-        //     TerrainChannel. Someday fix the plumbing between here and the scene.
+        //     TerrainData. Someday fix the plumbing between here and the scene.
         public static LayerDataPacket CreateLandPacket(TerrainData terrData, int patchX, int patchY)
         {
             int[] xPieces = new int[1];
-- 
cgit v1.1