]> err.no Git - sope/commitdiff
cleaned up logging subproject
authorhelge <helge@e4a50df8-12e2-0310-a44c-efbce7f8a7e3>
Fri, 12 Nov 2004 22:08:13 +0000 (22:08 +0000)
committerhelge <helge@e4a50df8-12e2-0310-a44c-efbce7f8a7e3>
Fri, 12 Nov 2004 22:08:13 +0000 (22:08 +0000)
git-svn-id: http://svn.opengroupware.org/SOPE/trunk@367 e4a50df8-12e2-0310-a44c-efbce7f8a7e3

17 files changed:
sope-core/NGExtensions/ChangeLog
sope-core/NGExtensions/NGExtensions/NGLogAppender.h
sope-core/NGExtensions/NGExtensions/NGLogConsoleAppender.h
sope-core/NGExtensions/NGExtensions/NGLogEvent.h
sope-core/NGExtensions/NGExtensions/NGLogSyslogAppender.h
sope-core/NGExtensions/NGExtensions/NGLogger.h
sope-core/NGExtensions/NGExtensions/NGLoggerManager.h
sope-core/NGExtensions/NGExtensions/NSObject+ExtendedLogging.h
sope-core/NGExtensions/NGLogging.subproj/ChangeLog
sope-core/NGExtensions/NGLogging.subproj/GNUmakefile
sope-core/NGExtensions/NGLogging.subproj/NGLogAppender.m
sope-core/NGExtensions/NGLogging.subproj/NGLogEvent.m
sope-core/NGExtensions/NGLogging.subproj/NGLogSyslogAppender.m
sope-core/NGExtensions/NGLogging.subproj/NGLogger.m
sope-core/NGExtensions/NGLogging.subproj/NGLoggerManager.m
sope-core/NGExtensions/NGLogging.subproj/NSObject+ExtendedLogging.m
sope-core/NGExtensions/Version

index 3fa31299d813733194e2cea219c2f4e85cc4b860..901c9b842ed682a3506108c580ff1c56eac84872 100644 (file)
@@ -1,6 +1,10 @@
+2004-11-12  Helge Hess  <helge.hess@opengroupware.org>
+
+       * NGLogging: code cleanup (v4.5.128)
+
 2004-11-12  Marcus Mueller  <znek@mulle-kybernetik.com>
 
-       * GNUmakefile: added NGLogging.subproj (v4.3.127)
+       * GNUmakefile: added NGLogging.subproj (v4.5.127)
 
 2004-10-21  Helge Hess  <helge.hess@skyrix.com>
 
index 975a50c4e533eca7841b264b70c0aacb617f7caf..3de602acb859080bfe8b4451304443f1c9573a91 100644 (file)
@@ -1,5 +1,5 @@
 /*
-  Copyright (C) 2000-2004 SKYRIX Software AG
+  Copyright (C) 2004 SKYRIX Software AG
 
   This file is part of OpenGroupware.org.
 
 #define        __NGExtensions_NGLogAppender_H_
 
 /*
-  Abstract superclass for all appenders.
+  Abstract superclass for all log appenders.
 */
 
-#import  <Foundation/Foundation.h>
-#include <NGExtensions/NSObject+ExtendedLogging.h>
+#import <Foundation/NSObject.h>
+#include <NGExtensions/NSObject+ExtendedLogging.h> /* for LogLevel */
 
 @class NGLogEvent;
 
index 90fc9b044908b29efa11d07a106248044482cc08..ec8c8b747e118c0ccfaecfe22b34e35a060cdbdd 100644 (file)
@@ -1,5 +1,5 @@
 /*
-  Copyright (C) 2000-2004 SKYRIX Software AG
+  Copyright (C) 2004 SKYRIX Software AG
 
   This file is part of OpenGroupware.org.
 
index 8d541bc80ab2ffc40e5cb3b3c38ed5ef2f010e9c..b507c96fdab25a0ff65376d5d119aae4fb7a135e 100644 (file)
@@ -1,5 +1,5 @@
 /*
-  Copyright (C) 2000-2004 SKYRIX Software AG
+  Copyright (C) 2004 SKYRIX Software AG
 
   This file is part of OpenGroupware.org.
 
 #ifndef        __NGExtensions_NGLogEvent_H_
 #define        __NGExtensions_NGLogEvent_H_
 
-#import  <Foundation/Foundation.h>
-#include <NGExtensions/NSObject+ExtendedLogging.h>
+#import <Foundation/NSObject.h>
+#import <Foundation/NSDate.h>
 
+#include <NGExtensions/NSObject+ExtendedLogging.h> /* for NGLogLevel */
+
+@class NSString;
 
 @interface NGLogEvent : NSObject
 {
-  NSString *msg;
-  NGLogLevel level;
+  NSString       *msg;
+  NGLogLevel     level;
   NSTimeInterval date;
 }
 
 - (id)initWithLevel:(NGLogLevel)_level message:(NSString *)_msg;
 
+/* accessors */
+
 - (NGLogLevel)level;
 - (NSString *)message;
 - (NSDate *)date;
index 4e3b99f7c3a6f7f17fec966e56d0b286e495f1bc..2539e7b6d3d6c64c9195a920810307e71c730687 100644 (file)
 
 #include <NGExtensions/NGLogAppender.h>
 
+/*
+  NGLogSyslogAppender
+  
+  An appender which writes to the systems syslog facility.
+*/
+
+@class NSString;
+
 @interface NGLogSyslogAppender : NGLogAppender
 {
 }
index 1e9ddfccc3e486893198e4ae6f763d3271e39285..d5dbda1a72c8acf9ffbb4add775b7ebcdcd19cc6 100644 (file)
@@ -1,5 +1,5 @@
 /*
-  Copyright (C) 2000-2004 SKYRIX Software AG
+  Copyright (C) 2004 SKYRIX Software AG
 
   This file is part of OpenGroupware.org.
 
 #define        __NGExtensions_NGLogger_H_
 
 /*
-  The logger, modeled closely after log4j.
- */
+  NGLogger
+  
+  A logger class, modeled closely after Log4J.
+  
+  TODO: explain more.
+*/
 
-#import  <Foundation/Foundation.h>
+#import <Foundation/NSObject.h>
 #include <NGExtensions/NSObject+ExtendedLogging.h>
 
-
 @interface NGLogger : NSObject
 {
   NGLogLevel logLevel;
-  id _appender; // going away as soon as we have a config
+  id         _appender; // going away as soon as we have a config
 }
 
 - (id)initWithLogLevel:(NGLogLevel)_level;
 
-- (void)logLevel:(NGLogLevel)_level message:(NSString *)_msg;
-- (void)logLevel:(NGLogLevel)_level withFormat:(NSString *)_fmt, ...;
+/* accessors */
 
 - (void)setLogLevel:(NGLogLevel)_level;
 - (NGLogLevel)logLevel;
 
+/* logging */
+
+- (void)logLevel:(NGLogLevel)_level message:(NSString *)_msg;
+- (void)logLevel:(NGLogLevel)_level withFormat:(NSString *)_fmt, ...;
+
 @end
 
 #endif /* __NGExtensions_NGLogger_H_ */
index 26214f5c8ac063fd0c94f1356975097c67901b45..b83c1398ceb74293b92fa99b373e5dd1b2b6b942 100644 (file)
@@ -1,5 +1,5 @@
 /*
-  Copyright (C) 2000-2004 SKYRIX Software AG
+  Copyright (C) 2004 SKYRIX Software AG
 
   This file is part of OpenGroupware.org.
 
@@ -22,8 +22,9 @@
 #ifndef        __NGExtensions_NGLoggerManager_H_
 #define        __NGExtensions_NGLoggerManager_H_
 
-#import <Foundation/Foundation.h>
+#import <Foundation/NSObject.h>
 
+@class NSString, NSMutableDictionary;
 @class NGLogger;
 
 @interface NGLoggerManager : NSObject
@@ -31,7 +32,9 @@
   NSMutableDictionary *loggerMap;
 }
 
-+ (id)defaultManager;
++ (id)defaultLoggerManager;
+
+/* operations */
 
 - (NGLogger *)loggerForDefaultKey:(NSString *)_defaultKey;
 
index 155244147e0c81daf973621311ce0b5d66cb46a6..548f07789cf5bc141e967aeac2a23948881a86aa 100644 (file)
@@ -22,7 +22,7 @@
 #ifndef        __NSObject_ExtendedLogging_H_
 #define        __NSObject_ExtendedLogging_H_
 
-#import <Foundation/Foundation.h>
+#import <Foundation/NSObject.h>
 
 typedef enum {
   NGLogLevelOff   = 0,
@@ -34,22 +34,28 @@ typedef enum {
   NGLogLevelAll   = 6
 } NGLogLevel;
 
-@interface NSObject (NGExtendedLogging)
+@interface NSObject(NGExtendedLogging)
 
 - (id)logger;
 
+/* convenience methods */
+
 - (void)logDebugWithFormat:(NSString *)_fmt, ...;
 - (void)logInfoWithFormat:(NSString *)_fmt, ...;
 - (void)logWarnWithFormat:(NSString *)_fmt, ...;
 - (void)logErrorWithFormat:(NSString *)_fmt, ...;
 - (void)logFatalWithFormat:(NSString *)_fmt, ...;
 
+/* conditions (try to avoid these, slow!) */
+
 - (BOOL)isLogDebugEnabled;
 - (BOOL)isLogInfoEnabled;
 - (BOOL)isLogWarnEnabled;
 - (BOOL)isLogErrorEnabled;
 - (BOOL)isLogFatalEnabled;
 
+/* generic methods */
+
 - (void)logLevel:(NGLogLevel)_level message:(NSString *)_msg;
 - (void)logLevel:(NGLogLevel)_level withFormat:(NSString *)_fmt, ...;
 
index 2171483682593b176ede0e48bcf669e7c935fbf8..3996480a452fd23f421302b2d021c026fc3e822a 100644 (file)
@@ -1,3 +1,9 @@
+2004-11-12  Helge Hess  <helge.hess@opengroupware.org>
+       
+       * deprecated -defaultManager in favor of -defaultLoggerManager
+
+       * code cleanups for SOPE styleguides
+
 2004-11-12  Marcus Mueller  <znek@mulle-kybernetik.com>
 
        * NSObject+ExtendedLogging.[hm]: -(id)logger queries NGLoggerManager
index 9f3bcf118eb54ea8d0bda65594cc3b5c104301fb..90d83e252bc34684ea51aa8fd2e5c8f630a0f784 100644 (file)
@@ -5,15 +5,18 @@ include ../../common.make
 
 SUBPROJECT_NAME = NGLogging
 
-NGLogging_OBJC_FILES = \
+NGLogging_OBJC_FILES += \
        NGLogger.m                      \
        NGLoggerManager.m               \
        NGLogEvent.m                    \
        NGLogAppender.m                 \
        NGLogConsoleAppender.m          \
-       NGLogSyslogAppender.m           \
        NSObject+ExtendedLogging.m      \
 
+# TODO: disable on Windows
+NGLogging_OBJC_FILES += \
+       NGLogSyslogAppender.m
+
 ADDITIONAL_INCLUDE_DIRS += -I. -I../NGExtensions/ -I.. -I../..
 
 -include GNUmakefile.preamble
index 86211367e3a07715e5dd19b4328b3e1d9a35d1ab..38e5adb3e19b86e27cf7598581b36a187a5862c9 100644 (file)
@@ -1,23 +1,23 @@
 /*
Copyright (C) 2000-2004 SKYRIX Software AG
- This file is part of OpenGroupware.org.
- OGo is free software; you can redistribute it and/or modify it under
- the terms of the GNU Lesser General Public License as published by the
- Free Software Foundation; either version 2, or (at your option) any
- later version.
- OGo is distributed in the hope that it will be useful, but WITHOUT ANY
- WARRANTY; without even the implied warranty of MERCHANTABILITY or
- FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
- License for more details.
- You should have received a copy of the GNU Lesser General Public
- License along with OGo; see the file COPYING.  If not, write to the
- Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
- 02111-1307, USA.
- */
 Copyright (C) 2004 SKYRIX Software AG
+  
 This file is part of OpenGroupware.org.
+  
 OGo is free software; you can redistribute it and/or modify it under
 the terms of the GNU Lesser General Public License as published by the
 Free Software Foundation; either version 2, or (at your option) any
 later version.
+  
 OGo is distributed in the hope that it will be useful, but WITHOUT ANY
 WARRANTY; without even the implied warranty of MERCHANTABILITY or
 FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
 License for more details.
+  
 You should have received a copy of the GNU Lesser General Public
 License along with OGo; see the file COPYING.  If not, write to the
 Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 02111-1307, USA.
+*/
 
 #include "NGLogAppender.h"
 #include "NSObject+ExtendedLogging.h"
@@ -61,4 +61,4 @@
   return name;
 }
 
-@end
+@end /* NGLogAppender */
index eb68b136d6fb33cd440f25c324538ef053323082..fc99bef14625a8c41a27e84c57c7dd84b2414be0 100644 (file)
@@ -1,34 +1,40 @@
 /*
- Copyright (C) 2000-2004 SKYRIX Software AG
- This file is part of OpenGroupware.org.
- OGo is free software; you can redistribute it and/or modify it under
- the terms of the GNU Lesser General Public License as published by the
- Free Software Foundation; either version 2, or (at your option) any
- later version.
- OGo is distributed in the hope that it will be useful, but WITHOUT ANY
- WARRANTY; without even the implied warranty of MERCHANTABILITY or
- FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
- License for more details.
- You should have received a copy of the GNU Lesser General Public
- License along with OGo; see the file COPYING.  If not, write to the
- Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
- 02111-1307, USA.
- */
 Copyright (C) 2000-2004 SKYRIX Software AG
+  
 This file is part of OpenGroupware.org.
+  
 OGo is free software; you can redistribute it and/or modify it under
 the terms of the GNU Lesser General Public License as published by the
 Free Software Foundation; either version 2, or (at your option) any
 later version.
+  
 OGo is distributed in the hope that it will be useful, but WITHOUT ANY
 WARRANTY; without even the implied warranty of MERCHANTABILITY or
 FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
 License for more details.
+  
 You should have received a copy of the GNU Lesser General Public
 License along with OGo; see the file COPYING.  If not, write to the
 Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 02111-1307, USA.
+*/
 
 #include "NGLogEvent.h"
 #include "common.h"
 
 @implementation NGLogEvent
 
+static Class DateClass = Nil;
+
++ (void)initialize {
+  DateClass = [NSDate class];
+}
+
 - (id)initWithLevel:(NGLogLevel)_level message:(NSString *)_msg {
   if((self = [super init])) {
-    self->date = [NSDate timeIntervalSinceReferenceDate];
+    self->date  = [DateClass timeIntervalSinceReferenceDate];
     self->level = _level;
-    ASSIGN(self->msg, _msg);
+    self->msg   = [_msg copy];
   }
   return self;
 }
@@ -38,6 +44,8 @@
   [super dealloc];
 }
 
+/* accessors */
+
 - (NGLogLevel)level {
   return self->level;
 }
@@ -47,7 +55,7 @@
 }
 
 - (NSDate *)date {
-  return [NSDate dateWithTimeIntervalSinceReferenceDate:self->date];
+  return [DateClass dateWithTimeIntervalSinceReferenceDate:self->date];
 }
 
-@end
+@end /* NGLogEvent */
index dc65fdd3faacb0e27773bae659f8f98419f326c5..430dadab79d6a52cf68be17637c83def832c622b 100644 (file)
@@ -1,23 +1,23 @@
 /*
- Copyright (C) 2000-2004 SKYRIX Software AG
- This file is part of OpenGroupware.org.
- OGo is free software; you can redistribute it and/or modify it under
- the terms of the GNU Lesser General Public License as published by the
- Free Software Foundation; either version 2, or (at your option) any
- later version.
- OGo is distributed in the hope that it will be useful, but WITHOUT ANY
- WARRANTY; without even the implied warranty of MERCHANTABILITY or
- FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
- License for more details.
- You should have received a copy of the GNU Lesser General Public
- License along with OGo; see the file COPYING.  If not, write to the
- Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
- 02111-1307, USA.
- */
 Copyright (C) 2000-2004 SKYRIX Software AG
+  
 This file is part of OpenGroupware.org.
+  
 OGo is free software; you can redistribute it and/or modify it under
 the terms of the GNU Lesser General Public License as published by the
 Free Software Foundation; either version 2, or (at your option) any
 later version.
+  
 OGo is distributed in the hope that it will be useful, but WITHOUT ANY
 WARRANTY; without even the implied warranty of MERCHANTABILITY or
 FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
 License for more details.
+  
 You should have received a copy of the GNU Lesser General Public
 License along with OGo; see the file COPYING.  If not, write to the
 Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 02111-1307, USA.
+*/
 
 #include "NGLogSyslogAppender.h"
 #include "NGLogEvent.h"
@@ -36,7 +36,7 @@ static NSString *defaultSyslogIdentifier = nil;
   NSUserDefaults *ud;
   static BOOL isInitialized = NO;
   
-  if(isInitialized)
+  if (isInitialized)
     return;
   
   ud = [NSUserDefaults standardUserDefaults];
@@ -48,7 +48,7 @@ static NSString *defaultSyslogIdentifier = nil;
 
 + (id)sharedAppender {
   static id sharedAppender = nil;
-  if(sharedAppender == nil) {
+  if (sharedAppender == nil) {
     sharedAppender = [[self alloc] init];
   }
   return sharedAppender;
@@ -59,8 +59,9 @@ static NSString *defaultSyslogIdentifier = nil;
 }
 
 - (id)initWithIdentifier:(NSString *)_ident {
-  if((self = [super init])) {
-#warning ** default flags?
+  if ((self = [super init])) {
+    // TODO: default flags?
+    // TODO: error code processing?
     openlog([_ident cString], LOG_PID | LOG_NDELAY, LOG_USER);
   }
   return self;
@@ -71,6 +72,8 @@ static NSString *defaultSyslogIdentifier = nil;
   [super dealloc];
 }
 
+/* operations */
+
 - (void)appendLogEvent:(NGLogEvent *)_event {
   NSString *formattedMsg;
   int level;
@@ -106,4 +109,4 @@ static NSString *defaultSyslogIdentifier = nil;
   return level;
 }
 
-@end
+@end /* NGLogSyslogAppender */
index 5cfcd907a83156bfad7854716275122ee9d5aea3..0799ac9a2e268d2a5fff997b7ecdc34c474fb3ac 100644 (file)
@@ -1,23 +1,23 @@
 /*
Copyright (C) 2000-2004 SKYRIX Software AG
- This file is part of OpenGroupware.org.
- OGo is free software; you can redistribute it and/or modify it under
- the terms of the GNU Lesser General Public License as published by the
- Free Software Foundation; either version 2, or (at your option) any
- later version.
- OGo is distributed in the hope that it will be useful, but WITHOUT ANY
- WARRANTY; without even the implied warranty of MERCHANTABILITY or
- FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
- License for more details.
- You should have received a copy of the GNU Lesser General Public
- License along with OGo; see the file COPYING.  If not, write to the
- Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
- 02111-1307, USA.
- */
 Copyright (C) 2004 SKYRIX Software AG
+  
 This file is part of OpenGroupware.org.
+  
 OGo is free software; you can redistribute it and/or modify it under
 the terms of the GNU Lesser General Public License as published by the
 Free Software Foundation; either version 2, or (at your option) any
 later version.
+  
 OGo is distributed in the hope that it will be useful, but WITHOUT ANY
 WARRANTY; without even the implied warranty of MERCHANTABILITY or
 FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
 License for more details.
+  
 You should have received a copy of the GNU Lesser General Public
 License along with OGo; see the file COPYING.  If not, write to the
 Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 02111-1307, USA.
+*/
 
 #include "NGLogger.h"
 #include "common.h"
 
 @implementation NGLogger
 
+static Class StrClass = Nil;
+
++ (void)initialize {
+  StrClass = [NSString class];
+}
+
 - (id)init {
-  self = [self initWithLogLevel:NGLogLevelAll];
-  return self;
+  return [self initWithLogLevel:NGLogLevelAll];
 }
 
 - (id)initWithLogLevel:(NGLogLevel)_level {
-  if((self = [super init])) {
+  if ((self = [super init])) {
     NSUserDefaults *ud;
     NSString *appenderClassName;
     
     [self setLogLevel:_level];
     
-#warning ** remove this as soon as we have a config
+    // TODO: remove this as soon as we have a config
     ud = [NSUserDefaults standardUserDefaults];
     appenderClassName = [ud stringForKey:@"NGLogDefaultAppenderClass"];
-    if(appenderClassName == nil)
+    if (appenderClassName == nil)
       appenderClassName = @"NGLogConsoleAppender";
     self->_appender = [[NSClassFromString(appenderClassName) alloc] init];
   }
@@ -53,6 +58,7 @@
   [super dealloc];
 }
 
+/* accessors */
 
 - (void)setLogLevel:(NGLogLevel)_level {
   self->logLevel = _level;
@@ -62,6 +68,8 @@
   return self->logLevel;
 }
 
+/* logging */
+
 - (void)logLevel:(NGLogLevel)_level withFormat:(NSString *)_fmt, ... {
   NSString *msg;
   va_list va;
@@ -70,7 +78,7 @@
     return;
   
   va_start(va, _fmt);
-  msg = [[NSString alloc] initWithFormat:_fmt arguments:va];
+  msg = [[StrClass alloc] initWithFormat:_fmt arguments:va];
   va_end(va);
   [self logLevel:_level message:msg];
   [msg release];
   NGLogEvent *event;
 
   event = [[NGLogEvent alloc] initWithLevel:_level message:_msg];
-
+  
   // iterate appenders
   // TODO: as soon as we have more appenders, we need to iterate on them
   [self->_appender appendLogEvent:event];
   [event release];
 }
 
+/* log conditions */
+
 - (BOOL)isLogDebugEnabled {
   return self->logLevel >= NGLogLevelDebug;
 }
   return self->logLevel >= NGLogLevelFatal;
 }
 
-@end
+@end /* NGLogger */
index d56129f2532e68d2c9ac0c10726a8ce86b973470..d598e8b02842e0de4f6f32c177e0eddb515b520e 100644 (file)
@@ -22,6 +22,7 @@
 #include "NGLoggerManager.h"
 #include "NSObject+ExtendedLogging.h"
 #include "NGLogger.h"
+#include "common.h"
 
 @interface NGLoggerManager (PrivateAPI)
 - (NGLogLevel)_logLevelForString:(NSString *)_defaultValue;
@@ -35,21 +36,25 @@ static NSNull          *sharedNull;
 + (void)initialize {
   static BOOL didInit = NO;
   
-  if(didInit)
+  if (didInit)
     return;
   
   didInit        = YES;
   sharedInstance = [[self alloc] init];
-  sharedNull     = [NSNull null];
+  sharedNull     = [[NSNull null] retain];
 }
 
-+ (id)defaultManager {
++ (id)defaultLoggerManager {
   return sharedInstance;
 }
++ (id)defaultManager {
+  NSLog(@"WARNING(%s): called deprecated method.", __PRETTY_FUNCTION__);
+  return [self defaultLoggerManager];
+}
 
 - (id)init {
   self = [super init];
-  if(self) {
+  if (self) {
     self->loggerMap = [[NSMutableDictionary alloc] initWithCapacity:50];
   }
   return self;
@@ -60,11 +65,13 @@ static NSNull          *sharedNull;
   [super dealloc];
 }
 
+/* operations */
+
 - (NGLogger *)loggerForDefaultKey:(NSString *)_defaultKey {
   id logger;
 
   logger = [self->loggerMap objectForKey:_defaultKey];
-  if(!logger) {
+  if (!logger) {
     NSUserDefaults *ud;
     NSString       *levelId;
     NGLogLevel     level;
@@ -72,7 +79,7 @@ static NSNull          *sharedNull;
     ud      = [NSUserDefaults standardUserDefaults];
     levelId = [[ud stringForKey:_defaultKey] uppercaseString];
     level   = [self _logLevelForString:levelId];
-    if(level == NGLogLevelOff) {
+    if (level == NGLogLevelOff) {
       [self->loggerMap setObject:sharedNull forKey:_defaultKey];
       logger = sharedNull;
     }
@@ -88,20 +95,23 @@ static NSNull          *sharedNull;
 /* Private */
 
 - (NGLogLevel)_logLevelForString:(NSString *)_defaultValue {
-  if(!_defaultValue || [_defaultValue isEqualToString:@"NO"])
+  if ([_defaultValue length] == 0 || [_defaultValue isEqualToString:@"NO"])
     return NGLogLevelOff;
-  else if([_defaultValue isEqualToString:@"YES"] ||
+  
+  if ([_defaultValue isEqualToString:@"YES"] ||
      [_defaultValue isEqualToString:@"DEBUG"])
     return NGLogLevelDebug;
-  else if([_defaultValue isEqualToString:@"INFO"])
+  
+  if ([_defaultValue isEqualToString:@"INFO"])
     return NGLogLevelInfo;
-  else if([_defaultValue isEqualToString:@"WARN"])
+  if ([_defaultValue isEqualToString:@"WARN"])
     return NGLogLevelWarn;
-  else if([_defaultValue isEqualToString:@"ERROR"])
+  if ([_defaultValue isEqualToString:@"ERROR"])
     return NGLogLevelError;
-  else if([_defaultValue isEqualToString:@"FATAL"])
+  if ([_defaultValue isEqualToString:@"FATAL"])
     return NGLogLevelFatal;
+  
   return NGLogLevelAll;
 }
 
-@end
+@end /* NGLoggerManager */
index 99ed679048bd446bc4ed0fc99a065afa7e1a10f2..0e28bf2d84d666d83c29b42100bfc52613002484 100644 (file)
@@ -1,43 +1,48 @@
 /*
Copyright (C) 2000-2004 SKYRIX Software AG
- This file is part of OpenGroupware.org.
- OGo is free software; you can redistribute it and/or modify it under
- the terms of the GNU Lesser General Public License as published by the
- Free Software Foundation; either version 2, or (at your option) any
- later version.
- OGo is distributed in the hope that it will be useful, but WITHOUT ANY
- WARRANTY; without even the implied warranty of MERCHANTABILITY or
- FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
- License for more details.
- You should have received a copy of the GNU Lesser General Public
- License along with OGo; see the file COPYING.  If not, write to the
- Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
- 02111-1307, USA.
- */
 Copyright (C) 2004 SKYRIX Software AG
+  
 This file is part of OpenGroupware.org.
+  
 OGo is free software; you can redistribute it and/or modify it under
 the terms of the GNU Lesser General Public License as published by the
 Free Software Foundation; either version 2, or (at your option) any
 later version.
+  
 OGo is distributed in the hope that it will be useful, but WITHOUT ANY
 WARRANTY; without even the implied warranty of MERCHANTABILITY or
 FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public
 License for more details.
+  
 You should have received a copy of the GNU Lesser General Public
 License along with OGo; see the file COPYING.  If not, write to the
 Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
 02111-1307, USA.
+*/
 
 #include "NSObject+ExtendedLogging.h"
 #include "NGLogger.h"
 #include "NGLoggerManager.h"
+#include "common.h"
 
-@implementation NSObject (NGExtendedLogging)
+@implementation NSObject(NGExtendedLogging)
 
 - (id)logger {
   static id   sharedLogger = nil;
   static BOOL shouldLog    = YES;
-  if(sharedLogger == nil && shouldLog) {
+  
+  if (sharedLogger == nil && shouldLog) {
     NGLoggerManager *lm;
-    lm           = [NGLoggerManager defaultManager];
+    
+    lm           = [NGLoggerManager defaultLoggerManager];
     sharedLogger = [lm loggerForDefaultKey:@"NGDefaultLogLevel"];
-    if(!sharedLogger)
+    if (!sharedLogger)
       shouldLog = NO;
   }
   return sharedLogger;
 }
 
+/* log methods */
+
 - (void)logDebugWithFormat:(NSString *)_fmt, ... {
   NSString *msg;
   va_list va;
   [msg release];
 }
 
+/* log conditions (do not use, slow ...) */
+
 - (BOOL)isLogDebugEnabled {
   return [[self logger] isLogDebugEnabled];
 }
   return [[self logger] isLogFatalEnabled];
 }
 
-@end
+@end /* NSObject(NGExtendedLogging) */
index 5ddb9af7aced8d70c192b15bd072e2331cd96c7c..195a3590a312de7a687f923be9e5873df4869960 100644 (file)
@@ -1,6 +1,6 @@
 # version
 
-SUBMINOR_VERSION:=127
+SUBMINOR_VERSION:=128
 
 # v4.3.115 requires libFoundation v1.0.59
 # v4.2.72  requires libEOControl  v4.2.39