diff options
author | Justin Clark-Casey (justincc) | 2012-04-17 01:33:55 +0100 |
---|---|---|
committer | Justin Clark-Casey (justincc) | 2012-04-17 01:33:55 +0100 |
commit | 7e1ab216e3c8b7ade3f007d352451c9d9d742b5c (patch) | |
tree | ab3221f7da5fbadb60311ac4340ca4b5dfe72669 | |
parent | refactor: Rename EstateSettings.IsEstateManager() to EstateSettings.IsEstateM... (diff) | |
download | opensim-SC_OLD-7e1ab216e3c8b7ade3f007d352451c9d9d742b5c.zip opensim-SC_OLD-7e1ab216e3c8b7ade3f007d352451c9d9d742b5c.tar.gz opensim-SC_OLD-7e1ab216e3c8b7ade3f007d352451c9d9d742b5c.tar.bz2 opensim-SC_OLD-7e1ab216e3c8b7ade3f007d352451c9d9d742b5c.tar.xz |
Move some public methods on WebStatsModule to private to reduce some static analysis complexity.
There's no obvious reason for these methods to be public.
-rw-r--r-- | OpenSim/Region/UserStatistics/WebStatsModule.cs | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/OpenSim/Region/UserStatistics/WebStatsModule.cs b/OpenSim/Region/UserStatistics/WebStatsModule.cs index a00955a..4248035 100644 --- a/OpenSim/Region/UserStatistics/WebStatsModule.cs +++ b/OpenSim/Region/UserStatistics/WebStatsModule.cs | |||
@@ -131,7 +131,7 @@ namespace OpenSim.Region.UserStatistics | |||
131 | } | 131 | } |
132 | } | 132 | } |
133 | 133 | ||
134 | public void ReceiveClassicSimStatsPacket(SimStats stats) | 134 | private void ReceiveClassicSimStatsPacket(SimStats stats) |
135 | { | 135 | { |
136 | if (!enabled) | 136 | if (!enabled) |
137 | { | 137 | { |
@@ -163,7 +163,7 @@ namespace OpenSim.Region.UserStatistics | |||
163 | } | 163 | } |
164 | } | 164 | } |
165 | 165 | ||
166 | public Hashtable HandleUnknownCAPSRequest(Hashtable request) | 166 | private Hashtable HandleUnknownCAPSRequest(Hashtable request) |
167 | { | 167 | { |
168 | //string regpath = request["uri"].ToString(); | 168 | //string regpath = request["uri"].ToString(); |
169 | int response_code = 200; | 169 | int response_code = 200; |
@@ -178,7 +178,7 @@ namespace OpenSim.Region.UserStatistics | |||
178 | return responsedata; | 178 | return responsedata; |
179 | } | 179 | } |
180 | 180 | ||
181 | public Hashtable HandleStatsRequest(Hashtable request) | 181 | private Hashtable HandleStatsRequest(Hashtable request) |
182 | { | 182 | { |
183 | lastHit = System.Environment.TickCount; | 183 | lastHit = System.Environment.TickCount; |
184 | Hashtable responsedata = new Hashtable(); | 184 | Hashtable responsedata = new Hashtable(); |
@@ -238,7 +238,7 @@ namespace OpenSim.Region.UserStatistics | |||
238 | return responsedata; | 238 | return responsedata; |
239 | } | 239 | } |
240 | 240 | ||
241 | public void CreateTables(SqliteConnection db) | 241 | private void CreateTables(SqliteConnection db) |
242 | { | 242 | { |
243 | using (SqliteCommand createcmd = new SqliteCommand(SQL_STATS_TABLE_CREATE, db)) | 243 | using (SqliteCommand createcmd = new SqliteCommand(SQL_STATS_TABLE_CREATE, db)) |
244 | { | 244 | { |
@@ -277,7 +277,7 @@ namespace OpenSim.Region.UserStatistics | |||
277 | get { return true; } | 277 | get { return true; } |
278 | } | 278 | } |
279 | 279 | ||
280 | public void OnRegisterCaps(UUID agentID, Caps caps) | 280 | private void OnRegisterCaps(UUID agentID, Caps caps) |
281 | { | 281 | { |
282 | // m_log.DebugFormat("[WEB STATS MODULE]: OnRegisterCaps: agentID {0} caps {1}", agentID, caps); | 282 | // m_log.DebugFormat("[WEB STATS MODULE]: OnRegisterCaps: agentID {0} caps {1}", agentID, caps); |
283 | 283 | ||
@@ -292,7 +292,7 @@ namespace OpenSim.Region.UserStatistics | |||
292 | })); | 292 | })); |
293 | } | 293 | } |
294 | 294 | ||
295 | public void OnDeRegisterCaps(UUID agentID, Caps caps) | 295 | private void OnDeRegisterCaps(UUID agentID, Caps caps) |
296 | { | 296 | { |
297 | } | 297 | } |
298 | 298 | ||
@@ -312,7 +312,7 @@ namespace OpenSim.Region.UserStatistics | |||
312 | } | 312 | } |
313 | } | 313 | } |
314 | 314 | ||
315 | public void OnMakeRootAgent(ScenePresence agent) | 315 | private void OnMakeRootAgent(ScenePresence agent) |
316 | { | 316 | { |
317 | UUID regionUUID = GetRegionUUIDFromHandle(agent.RegionHandle); | 317 | UUID regionUUID = GetRegionUUIDFromHandle(agent.RegionHandle); |
318 | 318 | ||
@@ -341,11 +341,11 @@ namespace OpenSim.Region.UserStatistics | |||
341 | } | 341 | } |
342 | } | 342 | } |
343 | 343 | ||
344 | public void OnMakeChildAgent(ScenePresence agent) | 344 | private void OnMakeChildAgent(ScenePresence agent) |
345 | { | 345 | { |
346 | } | 346 | } |
347 | 347 | ||
348 | public void OnClientClosed(UUID agentID, Scene scene) | 348 | private void OnClientClosed(UUID agentID, Scene scene) |
349 | { | 349 | { |
350 | lock (m_sessions) | 350 | lock (m_sessions) |
351 | { | 351 | { |
@@ -356,7 +356,7 @@ namespace OpenSim.Region.UserStatistics | |||
356 | } | 356 | } |
357 | } | 357 | } |
358 | 358 | ||
359 | public string readLogLines(int amount) | 359 | private string readLogLines(int amount) |
360 | { | 360 | { |
361 | Encoding encoding = Encoding.ASCII; | 361 | Encoding encoding = Encoding.ASCII; |
362 | int sizeOfChar = encoding.GetByteCount("\n"); | 362 | int sizeOfChar = encoding.GetByteCount("\n"); |
@@ -394,7 +394,7 @@ namespace OpenSim.Region.UserStatistics | |||
394 | return encoding.GetString(buffer); | 394 | return encoding.GetString(buffer); |
395 | } | 395 | } |
396 | 396 | ||
397 | public UUID GetRegionUUIDFromHandle(ulong regionhandle) | 397 | private UUID GetRegionUUIDFromHandle(ulong regionhandle) |
398 | { | 398 | { |
399 | lock (m_scenes) | 399 | lock (m_scenes) |
400 | { | 400 | { |
@@ -417,7 +417,7 @@ namespace OpenSim.Region.UserStatistics | |||
417 | /// <param name="agentID"></param> | 417 | /// <param name="agentID"></param> |
418 | /// <param name="caps"></param> | 418 | /// <param name="caps"></param> |
419 | /// <returns></returns> | 419 | /// <returns></returns> |
420 | public string ViewerStatsReport(string request, string path, string param, | 420 | private string ViewerStatsReport(string request, string path, string param, |
421 | UUID agentID, Caps caps) | 421 | UUID agentID, Caps caps) |
422 | { | 422 | { |
423 | // m_log.DebugFormat("[WEB STATS MODULE]: Received viewer starts report from {0}", agentID); | 423 | // m_log.DebugFormat("[WEB STATS MODULE]: Received viewer starts report from {0}", agentID); |
@@ -427,7 +427,7 @@ namespace OpenSim.Region.UserStatistics | |||
427 | return String.Empty; | 427 | return String.Empty; |
428 | } | 428 | } |
429 | 429 | ||
430 | public UserSessionID ParseViewerStats(string request, UUID agentID) | 430 | private UserSessionID ParseViewerStats(string request, UUID agentID) |
431 | { | 431 | { |
432 | UserSessionID uid = new UserSessionID(); | 432 | UserSessionID uid = new UserSessionID(); |
433 | UserSessionData usd; | 433 | UserSessionData usd; |
@@ -568,7 +568,7 @@ namespace OpenSim.Region.UserStatistics | |||
568 | return uid; | 568 | return uid; |
569 | } | 569 | } |
570 | 570 | ||
571 | public void UpdateUserStats(UserSessionID uid, SqliteConnection db) | 571 | private void UpdateUserStats(UserSessionID uid, SqliteConnection db) |
572 | { | 572 | { |
573 | if (uid.session_id == UUID.Zero) | 573 | if (uid.session_id == UUID.Zero) |
574 | return; | 574 | return; |