dr scofield's warning safari:

* commented out [Obsolete(....)] attributes where no replacement feature
  was available: if we want to attribute code that we think needs to be 
  reworked, we should define a new attribute and use that instead 
  (together with a little tool to retrieve all the attributed code then)
* commenting out unused variables

18 files changed, 64 insertions, 62 deletions

diff --git a/OpenSim/Data/Migration.cs b/OpenSim/Data/Migration.cs
index 70c7a46..6857c2c 100644
--- a/OpenSim/Data/Migration.cs
+++ b/OpenSim/Data/Migration.cs
@@ -74,13 +74,13 @@ namespace OpenSim.Data
 
         private string _type;
         private DbConnection _conn;
-        private string _subtype;
+        // private string _subtype;
         private Assembly _assem;
         private Regex _match;
         
         private static readonly string _migrations_create = "create table migrations(name varchar(100), version int)";
-        private static readonly string _migrations_init = "insert into migrations values('migrations', 1)";
-        private static readonly string _migrations_find = "select version from migrations where name='migrations'";
+        // private static readonly string _migrations_init = "insert into migrations values('migrations', 1)";
+        // private static readonly string _migrations_find = "select version from migrations where name='migrations'";
         
         public Migration(DbConnection conn, Assembly assem, string type)
         {
@@ -150,23 +150,23 @@ namespace OpenSim.Data
             }
         }
 
-        private int MaxVersion()
-        {
-            int max = 0;
-            string[] names = _assem.GetManifestResourceNames();
+        // private int MaxVersion()
+        // {
+        //     int max = 0;
+        //     string[] names = _assem.GetManifestResourceNames();
 
-            foreach (string s in names)
-            {
-                Match m = _match.Match(s);
-                if (m.Success) 
-                {
-                    int MigrationVersion = int.Parse(m.Groups[1].ToString());
-                    if (MigrationVersion > max)
-                        max = MigrationVersion;
-                }
-            }
-            return max;
-        }
+        //     foreach (string s in names)
+        //     {
+        //         Match m = _match.Match(s);
+        //         if (m.Success) 
+        //         {
+        //             int MigrationVersion = int.Parse(m.Groups[1].ToString());
+        //             if (MigrationVersion > max)
+        //                 max = MigrationVersion;
+        //         }
+        //     }
+        //     return max;
+        // }
 
         public int Version 
         {
@@ -222,10 +222,10 @@ namespace OpenSim.Data
             cmd.ExecuteNonQuery();
         }
         
-        private SortedList<int, string> GetAllMigrations()
-        {
-            return GetMigrationsAfter(0);
-        }
+        // private SortedList<int, string> GetAllMigrations()
+        // {
+        //     return GetMigrationsAfter(0);
+        // }
 
         private SortedList<int, string> GetMigrationsAfter(int after)
         {
diff --git a/OpenSim/Data/UserDataBase.cs b/OpenSim/Data/UserDataBase.cs
index 384dd19..eb21aee 100644
--- a/OpenSim/Data/UserDataBase.cs
+++ b/OpenSim/Data/UserDataBase.cs
@@ -35,9 +35,9 @@ namespace OpenSim.Data
 {
     public abstract class UserDataBase : IUserData
     {
-        private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
+        // private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
 
-        private Dictionary<LLUUID, AvatarAppearance> aplist = new Dictionary<LLUUID, AvatarAppearance>();
+        // private Dictionary<LLUUID, AvatarAppearance> aplist = new Dictionary<LLUUID, AvatarAppearance>();
 
         public abstract UserProfileData GetUserByUUID(LLUUID user);
         public abstract UserProfileData GetUserByName(string fname, string lname);
diff --git a/OpenSim/Framework/ClientManager.cs b/OpenSim/Framework/ClientManager.cs
index b557ec1..85dafef 100644
--- a/OpenSim/Framework/ClientManager.cs
+++ b/OpenSim/Framework/ClientManager.cs
@@ -138,7 +138,7 @@ namespace OpenSim.Framework
             }
         }
 
-        [Obsolete("Using Obsolete to drive development is invalid.  Obsolete presumes that something new has already been created to replace this.")]
+        // [Obsolete("Using Obsolete to drive development is invalid.  Obsolete presumes that something new has already been created to replace this.")]
         public uint[] GetAllCircuits(LLUUID agentId)
         {
             List<uint> circuits = new List<uint>();
diff --git a/OpenSim/Framework/Communications/Capabilities/Caps.cs b/OpenSim/Framework/Communications/Capabilities/Caps.cs
index 5249aa2..5aba086 100644
--- a/OpenSim/Framework/Communications/Capabilities/Caps.cs
+++ b/OpenSim/Framework/Communications/Capabilities/Caps.cs
@@ -79,17 +79,17 @@ namespace OpenSim.Framework.Communications.Capabilities
         private CapsHandlers m_capsHandlers;
 
         private static readonly string m_requestPath = "0000/";
-        private static readonly string m_mapLayerPath = "0001/";
+        // private static readonly string m_mapLayerPath = "0001/";
         private static readonly string m_newInventory = "0002/";
         //private static readonly string m_requestTexture = "0003/";
         private static readonly string m_notecardUpdatePath = "0004/";
         private static readonly string m_notecardTaskUpdatePath = "0005/";
-        private static readonly string m_fetchInventoryPath = "0006/";
+        // private static readonly string m_fetchInventoryPath = "0006/";
 
         // The following two entries are in a module, however, there also here so that we don't re-assign
         // the path to another cap by mistake.
-        private static readonly string m_parcelVoiceInfoRequestPath = "0007/"; // This is in a module.
-        private static readonly string m_provisionVoiceAccountRequestPath = "0008/";// This is in a module.
+        // private static readonly string m_parcelVoiceInfoRequestPath = "0007/"; // This is in a module.
+        // private static readonly string m_provisionVoiceAccountRequestPath = "0008/";// This is in a module.
 
         //private string eventQueue = "0100/";
         private BaseHttpServer m_httpListener;
@@ -246,7 +246,7 @@ namespace OpenSim.Framework.Communications.Capabilities
         // multiple fetch-folder maps are allowed within the larger folders map.
         public string FetchInventoryRequest(string request, string path, string param)
         {
-            string unmodifiedRequest = request.ToString();
+            // string unmodifiedRequest = request.ToString();
 
             //m_log.DebugFormat("[AGENT INVENTORY]: Received CAPS fetch inventory request {0}", unmodifiedRequest);
             m_log.Debug("[CAPS]: Inventory Request in region: " + m_regionName);
diff --git a/OpenSim/Framework/Communications/CommunicationsManager.cs b/OpenSim/Framework/Communications/CommunicationsManager.cs
index e8353b4..5de0c71 100644
--- a/OpenSim/Framework/Communications/CommunicationsManager.cs
+++ b/OpenSim/Framework/Communications/CommunicationsManager.cs
@@ -282,7 +282,7 @@ namespace OpenSim.Framework.Communications
                 if (profileData != null)
                 {
                     returnstring = new string[2];
-                    LLUUID profileId = profileData.ID;
+                    // LLUUID profileId = profileData.ID;
                     returnstring[0] = profileData.FirstName;
                     returnstring[1] = profileData.SurName;
                     lock (m_nameRequestCache)
diff --git a/OpenSim/Framework/Communications/LoginResponse.cs b/OpenSim/Framework/Communications/LoginResponse.cs
index 6fdd06a..113e3f0 100644
--- a/OpenSim/Framework/Communications/LoginResponse.cs
+++ b/OpenSim/Framework/Communications/LoginResponse.cs
@@ -95,7 +95,7 @@ namespace OpenSim.Framework.Communications
 
         // Response
         private XmlRpcResponse xmlRpcResponse;
-        private XmlRpcResponse defaultXmlRpcResponse;
+        // private XmlRpcResponse defaultXmlRpcResponse;
 
         private string welcomeMessage;
         private string startLocation;
@@ -117,7 +117,7 @@ namespace OpenSim.Framework.Communications
             loginError = new Hashtable();
             uiConfigHash = new Hashtable();
 
-            defaultXmlRpcResponse = new XmlRpcResponse();
+            // defaultXmlRpcResponse = new XmlRpcResponse();
             userProfile = new UserInfo();
             inventoryRoot = new ArrayList();
             initialOutfit = new ArrayList();
@@ -126,7 +126,7 @@ namespace OpenSim.Framework.Communications
             inventoryLibraryOwner = new ArrayList();
 
             xmlRpcResponse = new XmlRpcResponse();
-            defaultXmlRpcResponse = new XmlRpcResponse();
+            // defaultXmlRpcResponse = new XmlRpcResponse();
 
             SetDefaultValues();
         }
diff --git a/OpenSim/Framework/Communications/RestClient.cs b/OpenSim/Framework/Communications/RestClient.cs
index f6c33a8..b71a590 100644
--- a/OpenSim/Framework/Communications/RestClient.cs
+++ b/OpenSim/Framework/Communications/RestClient.cs
@@ -58,7 +58,7 @@ namespace OpenSim.Framework.Communications
     {
         private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
 
-        private string realuri;
+        // private string realuri;
 
         #region member variables
 
@@ -229,7 +229,7 @@ namespace OpenSim.Framework.Communications
                     sb.Append(kv.Value);
                 }
             }
-            realuri = sb.ToString();
+            // realuri = sb.ToString();
             //m_log.InfoFormat("[REST CLIENT]: RestURL: {0}", realuri);
             return new Uri(sb.ToString());
         }
@@ -250,8 +250,9 @@ namespace OpenSim.Framework.Communications
                 if (read > 0)
                 {
                     _resource.Write(_readbuf, 0, read);
-                    IAsyncResult asynchronousResult =
-                        s.BeginRead(_readbuf, 0, BufferSize, new AsyncCallback(StreamIsReadyDelegate), s);
+                    // IAsyncResult asynchronousResult =
+                    //     s.BeginRead(_readbuf, 0, BufferSize, new AsyncCallback(StreamIsReadyDelegate), s);
+                    s.BeginRead(_readbuf, 0, BufferSize, new AsyncCallback(StreamIsReadyDelegate), s);
 
                     // TODO! Implement timeout, without killing the server
                     //ThreadPool.RegisterWaitForSingleObject(asynchronousResult.AsyncWaitHandle, new WaitOrTimerCallback(TimeoutCallback), _request, DefaultTimeout, true);
diff --git a/OpenSim/Framework/Communications/XMPP/XmppSerializer.cs b/OpenSim/Framework/Communications/XMPP/XmppSerializer.cs
index 30a9eac..eb9dabe 100644
--- a/OpenSim/Framework/Communications/XMPP/XmppSerializer.cs
+++ b/OpenSim/Framework/Communications/XMPP/XmppSerializer.cs
@@ -38,8 +38,8 @@ namespace OpenSim.Framework.Communications.XMPP
 {
     public class XmppSerializer
     {
-        private static readonly ILog _log = 
-            LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
+        // private static readonly ILog _log = 
+        //     LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
 
         // need to do it this way, as XmlSerializer(type, extratypes)
         // does not work on mono (at least).
diff --git a/OpenSim/Framework/IClientAPI.cs b/OpenSim/Framework/IClientAPI.cs
index 5b0726b..7541c32 100644
--- a/OpenSim/Framework/IClientAPI.cs
+++ b/OpenSim/Framework/IClientAPI.cs
@@ -664,7 +664,7 @@ namespace OpenSim.Framework
         
         string LastName { get; }
 
-        [Obsolete("LLClientView Specific - Replace with ???")]
+        // [Obsolete("LLClientView Specific - Replace with ???")]
         int NextAnimationSequenceNumber { get; }
 
         /// <summary>
@@ -680,22 +680,22 @@ namespace OpenSim.Framework
             set;
         }
 
-        [Obsolete("LLClientView Specific - Circuits are unique to LLClientView")]
+        // [Obsolete("LLClientView Specific - Circuits are unique to LLClientView")]
         uint CircuitCode { get; }
-        [Obsolete("LLClientView Specific - Replace with more bare-bones arguments.")]
+        // [Obsolete("LLClientView Specific - Replace with more bare-bones arguments.")]
         event ImprovedInstantMessage OnInstantMessage;
-        [Obsolete("LLClientView Specific - Replace with more bare-bones arguments. Rename OnChat.")]
+        // [Obsolete("LLClientView Specific - Replace with more bare-bones arguments. Rename OnChat.")]
         event ChatFromViewer OnChatFromViewer;
-        [Obsolete("LLClientView Specific - Replace with more bare-bones arguments.")]
+        // [Obsolete("LLClientView Specific - Replace with more bare-bones arguments.")]
         event TextureRequest OnRequestTexture;
-        [Obsolete("LLClientView Specific - Remove bitbuckets. Adam, can you be more specific here..  as I don't see any bit buckets.")]
+        // [Obsolete("LLClientView Specific - Remove bitbuckets. Adam, can you be more specific here..  as I don't see any bit buckets.")]
         event RezObject OnRezObject;
-        [Obsolete("LLClientView Specific - Replace with more suitable arguments.")]
+        // [Obsolete("LLClientView Specific - Replace with more suitable arguments.")]
         event ModifyTerrain OnModifyTerrain;
         event BakeTerrain OnBakeTerrain;
-        [Obsolete("LLClientView Specific.")]
+        // [Obsolete("LLClientView Specific.")]
         event SetAppearance OnSetAppearance;
-        [Obsolete("LLClientView Specific - Replace and rename OnAvatarUpdate. Difference from SetAppearance?")]
+        // [Obsolete("LLClientView Specific - Replace and rename OnAvatarUpdate. Difference from SetAppearance?")]
         event AvatarNowWearing OnAvatarNowWearing;
         event RezSingleAttachmentFromInv OnRezSingleAttachmentFromInv;
         event UUIDNameRequest OnDetachAttachmentIntoInv;
@@ -841,7 +841,7 @@ namespace OpenSim.Framework
         event ScriptReset OnScriptReset;
         event UpdateVector OnAutoPilotGo;
 
-        [Obsolete("IClientAPI.OutPacket SHOULD NOT EXIST outside of LLClientView please refactor appropriately.")]
+        // [Obsolete("IClientAPI.OutPacket SHOULD NOT EXIST outside of LLClientView please refactor appropriately.")]
         void OutPacket(Packet newPack, ThrottleOutPacketType packType);
         void SendWearables(AvatarWearable[] wearables, int serial);
         void SendAppearance(LLUUID agentID, byte[] visualParams, byte[] textureEntry);
diff --git a/OpenSim/Framework/PrimitiveBaseShape.cs b/OpenSim/Framework/PrimitiveBaseShape.cs
index 503830b..12ca688 100644
--- a/OpenSim/Framework/PrimitiveBaseShape.cs
+++ b/OpenSim/Framework/PrimitiveBaseShape.cs
@@ -341,7 +341,7 @@ namespace OpenSim.Framework
             byte[] returnbytes = new byte[TotalBytesLength];
 
 
-            uint paramlength = ExtraParamsNum;
+            // uint paramlength = ExtraParamsNum;
 
             // Stick in the number of parameters
             returnbytes[i++] = (byte)ExtraParamsNum;
@@ -462,7 +462,7 @@ namespace OpenSim.Framework
                 ushort epType = Helpers.BytesToUInt16(data, i);
 
                 i += 2;
-                uint paramLength = Helpers.BytesToUIntBig(data, i);
+                // uint paramLength = Helpers.BytesToUIntBig(data, i);
 
                 i += 4;
                 switch (epType)
diff --git a/OpenSim/Framework/Servers/RestObjectPosterResponse.cs b/OpenSim/Framework/Servers/RestObjectPosterResponse.cs
index 05bfaa1..cbc8be4 100644
--- a/OpenSim/Framework/Servers/RestObjectPosterResponse.cs
+++ b/OpenSim/Framework/Servers/RestObjectPosterResponse.cs
@@ -76,7 +76,8 @@ namespace OpenSim.Framework.Servers
 
             Stream requestStream = request.GetRequestStream();
             requestStream.Write(buffer.ToArray(), 0, length);
-            IAsyncResult result = request.BeginGetResponse(AsyncCallback, request);
+            // IAsyncResult result = request.BeginGetResponse(AsyncCallback, request);
+            request.BeginGetResponse(AsyncCallback, request);
         }
 
         private void AsyncCallback(IAsyncResult result)
diff --git a/OpenSim/Framework/Statistics/AssetStatsCollector.cs b/OpenSim/Framework/Statistics/AssetStatsCollector.cs
index f2b765f..526132a 100644
--- a/OpenSim/Framework/Statistics/AssetStatsCollector.cs
+++ b/OpenSim/Framework/Statistics/AssetStatsCollector.cs
@@ -86,7 +86,7 @@ namespace OpenSim.Framework.Statistics
         /// Report back collected statistical information.
         /// </summary>
         /// <returns></returns>
-        public string Report()
+        override public string Report()
         {
             double elapsedHours = (DateTime.Now - startTime).TotalHours;
             if (elapsedHours <= 0) { elapsedHours = 1; }  // prevent divide by zero
diff --git a/OpenSim/Framework/Statistics/UserStatsCollector.cs b/OpenSim/Framework/Statistics/UserStatsCollector.cs
index e094d91..9ff44ca 100644
--- a/OpenSim/Framework/Statistics/UserStatsCollector.cs
+++ b/OpenSim/Framework/Statistics/UserStatsCollector.cs
@@ -81,7 +81,7 @@ namespace OpenSim.Framework.Statistics
         /// Report back collected statistical information.
         /// </summary>
         /// <returns></returns>
-        public string Report()
+        override public string Report()
         {
             return string.Format(
 @"Successful logins total : {0}, today : {1}, yesterday : {2}
diff --git a/OpenSim/Framework/TaskInventoryItem.cs b/OpenSim/Framework/TaskInventoryItem.cs
index 1105242..a383242 100644
--- a/OpenSim/Framework/TaskInventoryItem.cs
+++ b/OpenSim/Framework/TaskInventoryItem.cs
@@ -44,7 +44,7 @@ namespace OpenSim.Framework
     public class TaskInventoryDictionary : Dictionary<LLUUID, TaskInventoryItem>,
                                            ICloneable, IXmlSerializable
     {
-        private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
+        // private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType);
 
         private static XmlSerializer tiiSerializer = new XmlSerializer(typeof (TaskInventoryItem));
 
diff --git a/OpenSim/Framework/Util.cs b/OpenSim/Framework/Util.cs
index 5b529da..d72e03e 100644
--- a/OpenSim/Framework/Util.cs
+++ b/OpenSim/Framework/Util.cs
@@ -637,7 +637,7 @@ namespace OpenSim.Framework
             }
 
             memory.Position = 0;
-            MemoryStream outStream = new MemoryStream();
+            // MemoryStream outStream = new MemoryStream();
 
             byte[] compressed = new byte[memory.Length];
             memory.Read(compressed, 0, compressed.Length);
diff --git a/OpenSim/Region/Communications/Local/LocalBackEndServices.cs b/OpenSim/Region/Communications/Local/LocalBackEndServices.cs
index 989f08a..47606c2 100644
--- a/OpenSim/Region/Communications/Local/LocalBackEndServices.cs
+++ b/OpenSim/Region/Communications/Local/LocalBackEndServices.cs
@@ -45,7 +45,7 @@ namespace OpenSim.Region.Communications.Local
         protected Dictionary<ulong, RegionCommsListener> m_regionListeners =
             new Dictionary<ulong, RegionCommsListener>();
 
-        private Dictionary<ulong, RegionInfo> m_remoteRegionInfoCache = new Dictionary<ulong, RegionInfo>();
+        // private Dictionary<ulong, RegionInfo> m_remoteRegionInfoCache = new Dictionary<ulong, RegionInfo>();
 
         private Dictionary<string, string> m_queuedGridSettings = new Dictionary<string, string>();
 
diff --git a/OpenSim/Region/Communications/Local/LocalLoginService.cs b/OpenSim/Region/Communications/Local/LocalLoginService.cs
index 966fdd4..5f4558d 100644
--- a/OpenSim/Region/Communications/Local/LocalLoginService.cs
+++ b/OpenSim/Region/Communications/Local/LocalLoginService.cs
@@ -209,7 +209,7 @@ namespace OpenSim.Region.Communications.Local
                 theUser.CurrentAgent.Region = reg.RegionID;
                 theUser.CurrentAgent.Handle = reg.RegionHandle;
 
-                LoginResponse.BuddyList buddyList = new LoginResponse.BuddyList();
+                // LoginResponse.BuddyList buddyList = new LoginResponse.BuddyList();
 
                 response.BuddList = ConvertFriendListItem(m_userManager.GetUserFriendList(theUser.ID));
 
diff --git a/OpenSim/Region/Communications/Local/LocalUserServices.cs b/OpenSim/Region/Communications/Local/LocalUserServices.cs
index 47f596d..798dd24 100644
--- a/OpenSim/Region/Communications/Local/LocalUserServices.cs
+++ b/OpenSim/Region/Communications/Local/LocalUserServices.cs
@@ -34,7 +34,7 @@ namespace OpenSim.Region.Communications.Local
 {
     public class LocalUserServices : UserManagerBase
     {
-        private readonly NetworkServersInfo m_serversInfo;
+        // private readonly NetworkServersInfo m_serversInfo;
         private readonly uint m_defaultHomeX;
         private readonly uint m_defaultHomeY;
         private IInventoryServices m_inventoryService;
@@ -50,7 +50,7 @@ namespace OpenSim.Region.Communications.Local
         public LocalUserServices(NetworkServersInfo serversInfo, uint defaultHomeLocX, uint defaultHomeLocY,
                                  IInventoryServices inventoryService)
         {
-            m_serversInfo = serversInfo;
+            // m_serversInfo = serversInfo;
 
             m_defaultHomeX = defaultHomeLocX;
             m_defaultHomeY = defaultHomeLocY;