diff options
author | Marck | 2010-09-19 13:26:06 +0200 |
---|---|---|
committer | Justin Clark-Casey (justincc) | 2010-10-09 00:47:39 +0100 |
commit | 211ea5d52126bf19d53e57af750a0a964a60b56e (patch) | |
tree | 3a6a00348dc6b1cf24f6a6ca8db05ca629b0534f | |
parent | Guard against locking a nullref to avoid build break (diff) | |
download | opensim-SC_OLD-211ea5d52126bf19d53e57af750a0a964a60b56e.zip opensim-SC_OLD-211ea5d52126bf19d53e57af750a0a964a60b56e.tar.gz opensim-SC_OLD-211ea5d52126bf19d53e57af750a0a964a60b56e.tar.bz2 opensim-SC_OLD-211ea5d52126bf19d53e57af750a0a964a60b56e.tar.xz |
Limit formatting of local console output to actual logging messages.
The formatting of lines with the help of a regular expression match will be done only for output with an explicitly given logging level.
This fixes the issue of colons being added to help texts on the local console.
-rw-r--r-- | OpenSim/Framework/Console/LocalConsole.cs | 30 |
1 files changed, 17 insertions, 13 deletions
diff --git a/OpenSim/Framework/Console/LocalConsole.cs b/OpenSim/Framework/Console/LocalConsole.cs index 6043094..5f2f404 100644 --- a/OpenSim/Framework/Console/LocalConsole.cs +++ b/OpenSim/Framework/Console/LocalConsole.cs | |||
@@ -44,6 +44,7 @@ namespace OpenSim.Framework.Console | |||
44 | // private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); | 44 | // private static readonly ILog m_log = LogManager.GetLogger(MethodBase.GetCurrentMethod().DeclaringType); |
45 | 45 | ||
46 | // private readonly object m_syncRoot = new object(); | 46 | // private readonly object m_syncRoot = new object(); |
47 | private const string LOGLEVEL_NONE = "(none)"; | ||
47 | 48 | ||
48 | private int y = -1; | 49 | private int y = -1; |
49 | private int cp = 0; | 50 | private int cp = 0; |
@@ -278,22 +279,25 @@ namespace OpenSim.Framework.Console | |||
278 | 279 | ||
279 | private void WriteLocalText(string text, string level) | 280 | private void WriteLocalText(string text, string level) |
280 | { | 281 | { |
281 | string regex = @"^(?<Front>.*?)\[(?<Category>[^\]]+)\]:?(?<End>.*)"; | ||
282 | |||
283 | Regex RE = new Regex(regex, RegexOptions.Multiline); | ||
284 | MatchCollection matches = RE.Matches(text); | ||
285 | |||
286 | string outText = text; | 282 | string outText = text; |
287 | 283 | ||
288 | if (matches.Count == 1) | 284 | if (level != LOGLEVEL_NONE) |
289 | { | 285 | { |
290 | outText = matches[0].Groups["End"].Value; | 286 | string regex = @"^(?<Front>.*?)\[(?<Category>[^\]]+)\]:?(?<End>.*)"; |
291 | System.Console.Write(matches[0].Groups["Front"].Value); | 287 | |
288 | Regex RE = new Regex(regex, RegexOptions.Multiline); | ||
289 | MatchCollection matches = RE.Matches(text); | ||
292 | 290 | ||
293 | System.Console.Write("["); | 291 | if (matches.Count == 1) |
294 | WriteColorText(DeriveColor(matches[0].Groups["Category"].Value), | 292 | { |
295 | matches[0].Groups["Category"].Value); | 293 | outText = matches[0].Groups["End"].Value; |
296 | System.Console.Write("]:"); | 294 | System.Console.Write(matches[0].Groups["Front"].Value); |
295 | |||
296 | System.Console.Write("["); | ||
297 | WriteColorText(DeriveColor(matches[0].Groups["Category"].Value), | ||
298 | matches[0].Groups["Category"].Value); | ||
299 | System.Console.Write("]:"); | ||
300 | } | ||
297 | } | 301 | } |
298 | 302 | ||
299 | if (level == "error") | 303 | if (level == "error") |
@@ -308,7 +312,7 @@ namespace OpenSim.Framework.Console | |||
308 | 312 | ||
309 | public override void Output(string text) | 313 | public override void Output(string text) |
310 | { | 314 | { |
311 | Output(text, "normal"); | 315 | Output(text, LOGLEVEL_NONE); |
312 | } | 316 | } |
313 | 317 | ||
314 | public override void Output(string text, string level) | 318 | public override void Output(string text, string level) |