]> err.no Git - sope/commitdiff
minor code cleanups
authorhelge <helge@e4a50df8-12e2-0310-a44c-efbce7f8a7e3>
Fri, 19 Nov 2004 16:59:17 +0000 (16:59 +0000)
committerhelge <helge@e4a50df8-12e2-0310-a44c-efbce7f8a7e3>
Fri, 19 Nov 2004 16:59:17 +0000 (16:59 +0000)
git-svn-id: http://svn.opengroupware.org/SOPE/trunk@403 e4a50df8-12e2-0310-a44c-efbce7f8a7e3

sope-core/NGStreams/NGStreams/NGActiveSocket.h
sope-core/NGStreams/NGStreams/NGSocketProtocols.h
sope-core/NGStreams/NGStreams/NGStream.h
sope-core/NGStreams/NGStreams/NGTextStream.h
sope-mime/ChangeLog
sope-mime/NGImap4/ChangeLog
sope-mime/NGImap4/NGImap4Client.m
sope-mime/NGImap4/NGSieveClient.m
sope-mime/Version

index b80808a594473d582fa33b6a79d2da28b9c3f398..c7dcbae314e8cf5ae6b05d122307578a51b26e26 100644 (file)
@@ -1,7 +1,7 @@
 /*
-  Copyright (C) 2000-2003 SKYRIX Software AG
+  Copyright (C) 2000-2004 SKYRIX Software AG
 
-  This file is part of OGo
+  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
@@ -18,7 +18,6 @@
   Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
   02111-1307, USA.
 */
-// $Id$
 
 #ifndef __NGNet_NGActiveSocket_H__
 #define __NGNet_NGActiveSocket_H__
 @class NSData, NSFileHandle;
 
 /*
-  Represents an active STREAM socket based on the standard Unix sockets library.
+  Represents an active STREAM socket based on the standard Unix sockets
+  library.
 
   An active socket can be either a socket gained by calling accept with an
-  passive socket or by explicitly connecting one to an address (a client socket).
+  passive socket or by explicitly connecting one to an address (a client
+  socket).
   Therefore an active socket has two addresses, the local and the remote one.
 
   There are three methods to perform a close, this is rooted in the fact that
index e3f9796a37005a11437af19d1ede9c421299e8e6..56d77e595cebe973eef57ea1fe62a5b8322b35b5 100644 (file)
@@ -1,7 +1,7 @@
 /*
-  Copyright (C) 2000-2003 SKYRIX Software AG
+  Copyright (C) 2000-2004 SKYRIX Software AG
 
-  This file is part of OGo
+  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
@@ -18,7 +18,6 @@
   Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
   02111-1307, USA.
 */
-// $Id$
 
 #ifndef __NGNet_NGSocketProtocols_H__
 #define __NGNet_NGSocketProtocols_H__
index b4afec5585f0c728254d032ce37c0bee423e45ad..fbf3a4aaf29506ae67a0f4ba65d0964ae35006d5 100644 (file)
@@ -1,7 +1,7 @@
 /*
-  Copyright (C) 2000-2003 SKYRIX Software AG
+  Copyright (C) 2000-2004 SKYRIX Software AG
 
-  This file is part of OGo
+  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
@@ -18,7 +18,6 @@
   Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
   02111-1307, USA.
 */
-// $Id$
 
 #ifndef __NGStreams_NGStream_H__
 #define __NGStreams_NGStream_H__
@@ -57,6 +56,7 @@ static inline BOOL NGCanWriteInStreamMode(NGStreamMode _mode) {
 
 /* methods which read/write exactly _len bytes */
 
+// TODO: should return exception? (would change the API significantly)
 - (BOOL)safeReadBytes:(void *)_buf count:(unsigned)_len;
 - (BOOL)safeWriteBytes:(const void *)_buf count:(unsigned)_len;
 
index 7a5b4f2147c3af01857db361a0287669c9ff2188..2eb9ba861df551bc0e69331c78cb338e09ea9b1f 100644 (file)
@@ -1,7 +1,7 @@
 /*
-  Copyright (C) 2000-2003 SKYRIX Software AG
+  Copyright (C) 2000-2004 SKYRIX Software AG
 
-  This file is part of OGo
+  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
@@ -18,7 +18,6 @@
   Free Software Foundation, 59 Temple Place - Suite 330, Boston, MA
   02111-1307, USA.
 */
-// $Id$
 
 #ifndef __NGStreams_NGTextStream_H__
 #define __NGStreams_NGTextStream_H__
@@ -41,7 +40,7 @@
 - (void)setLastException:(NSException *)_exception;
 - (void)resetLastException;
 
-// NGExtendedTextInputStream
+/* NGExtendedTextInputStream */
 
 - (NSString *)readLineAsString; // inefficient
 - (unsigned)readCharacters:(unichar *)_chars count:(unsigned)_count;
index fa39c5c38d39f963a5203654ee185c688a8ab2cf..a1c8175a8878a081cf890783635205862d1fa16b 100644 (file)
@@ -1,3 +1,7 @@
+2004-11-19  Helge Hess  <helge.hess@skyrix.com>
+
+       * NGImap4: minor code cleanups (v4.5.200)
+
 2004-11-16  Helge Hess  <helge.hess@skyrix.com>
 
        * NGImap4: minor code cleanups in NGSieveClient (v4.5.199)
index bc32078bb49424c1780778e699db1203ea4a81f5..141136db151f8f18fee11a518733f58e065f3be6 100644 (file)
@@ -1,3 +1,7 @@
+2004-11-19  Helge Hess  <helge.hess@skyrix.com>
+
+       * NGSieveClient.m, NGImap4Client.m: minor code cleanups
+
 2004-11-16  Helge Hess  <helge.hess@skyrix.com>
 
        * NGSieveClient.m: some code cleanups, needs fixes for UTF-8 scripts
index 94df12f0dd16eb936cc378489674967eb7ef3529..d06f6eb64176699a38813439c646acd9e750b4ab 100644 (file)
@@ -1322,13 +1322,14 @@ static BOOL         ImapDebugEnabled   = NO;
   [self sendCommand:_command withTag:YES logText:_command];
 }
 
-static inline NSArray *_flags2ImapFlags(NGImap4Client *self, NSArray *_flags) {
+- (NSArray *)_flags2ImapFlags:(NSArray *)_flags {
+  /* adds backslashes in front of the flags */
   NSEnumerator *enumerator;
   NSArray      *result;
   id           obj;
   id           *objs;
   unsigned     cnt;
-
+  
   objs = calloc([_flags count] + 2, sizeof(id));
   cnt  = 0;
   enumerator = [_flags objectEnumerator];
@@ -1337,9 +1338,12 @@ static inline NSArray *_flags2ImapFlags(NGImap4Client *self, NSArray *_flags) {
     cnt++;
   }
   result = [NSArray arrayWithObjects:objs count:cnt];
-  if (objs) free(objs);
+  if (objs != NULL) free(objs);
   return result;
 }
+static inline NSArray *_flags2ImapFlags(NGImap4Client *self, NSArray *_flags) {
+  return [self _flags2ImapFlags:_flags];
+}
 
 - (NSString *)_folder2ImapFolder:(NSString *)_folder {
   NSArray *array;
index dfa78e1343a2fc1929d171235659f62d4d550e51..4d4a8d4387514a37394382a898d2f16ba2006df1 100644 (file)
 
 @end
 
-/*"
-**  An implementation of an Imap4 client
-**
-** A folder name always looks like an absolute filename (/inbox/doof) 
-**
-"*/
+/*
+  An implementation of an Imap4 client
+  
+  A folder name always looks like an absolute filename (/inbox/blah)
+  
+  NOTE: Sieve is just the filtering language ...
+  
+  This should be ACAP?
+    http://asg.web.cmu.edu/rfc/rfc2244.html
+
+  ---snip---
+"IMPLEMENTATION" "Cyrus timsieved v2.1.15-IPv6-Debian-2.1.15-0woody.1.0"
+"SASL" "PLAIN"
+"SIEVE" "fileinto reject envelope vacation imapflags notify subaddress relational regex"
+"STARTTLS"
+OK
+  ---snap---
+*/
 
 @implementation NGSieveClient
 
-static int      defaultSievePort = 143;
-static NSNumber *YesNumber = nil;
-static NSNumber *NoNumber  = nil;
+static int      defaultSievePort   = 2000;
+static NSNumber *YesNumber         = nil;
+static NSNumber *NoNumber          = nil;
 static BOOL     ProfileImapEnabled = NO;
 static BOOL     LOG_PASSWORD       = NO;
 static BOOL     debugImap4         = NO;
@@ -84,15 +96,12 @@ static BOOL     debugImap4         = NO;
 
 - (id)initWithHost:(id)_host {
   NGInternetSocketAddress *a;
+  
   a = [NGInternetSocketAddress addressWithPort:defaultSievePort onHost:_host];
   return [self initWithAddress:a];
 }
 
-/**"
- ** designated initializer
-"**/
-
-- (id)initWithAddress:(id<NGSocketAddress>)_address {
+- (id)initWithAddress:(id<NGSocketAddress>)_address { // di
   if ((self = [super init])) {
     self->address = [_address retain];
     self->debug = debugImap4;
@@ -138,9 +147,7 @@ static BOOL     debugImap4         = NO;
 
 /* connection */
 
-/*"
-** Opens a connection to given Host.
-"*/
+/* Opens a connection to given Host. */
 
 - (NSDictionary *)openConnection {
   struct timeval tv;
@@ -194,14 +201,12 @@ static BOOL     debugImap4         = NO;
   [self->parser release]; self->parser = nil;
 }
 
-/*"
-** login with plaintext password authenticating
-"*/
+/* login with plaintext password authenticating */
 
 - (NSDictionary *)login:(NSString *)_login password:(NSString *)_passwd {
   if ((_login == nil) || (_passwd == nil))
     return nil;
-
+  
   [self->login    release]; self->login    = nil;
   [self->password release]; self->password = nil;
   
@@ -224,8 +229,8 @@ static BOOL     debugImap4         = NO;
 
 
   logLen = [self->login cStringLength];
-  bufLen = (logLen*2) + [self->password cStringLength] +2;
-
+  bufLen = (logLen * 2) + [self->password cStringLength] +2;
+  
   buf = calloc(sizeof(char), bufLen);
 
   sprintf(buf, "%s %s %s", 
@@ -256,9 +261,7 @@ static BOOL     debugImap4         = NO;
   return [self normalizeResponse:map];
 }
 
-/*
-  logout from the connected host and close the connection
-*/
+/* logout from the connected host and close the connection */
 
 - (NSDictionary *)logout {
   NGHashMap *map;
@@ -472,9 +475,9 @@ static BOOL     debugImap4         = NO;
 - (void)sendCommand:(NSString *)_command logText:(NSString *)_txt {
   // TODO: should accept 'NSData' commands
   NSString *command = nil;
-
+  
   command = _command;
-
+  
   if (self->debug)
     fprintf(stderr, "C: %s\n", [_txt cString]);
   
index 268ab9f78bf23858d26fa88e4eb197631ea936ae..31c4d5bd3c7b3e5ecce7c574883487527bfa382d 100644 (file)
@@ -2,6 +2,6 @@
 
 MAJOR_VERSION:=4
 MINOR_VERSION:=5
-SUBMINOR_VERSION:=199
+SUBMINOR_VERSION:=200
 
 # v4.2.149 requires libNGStreams v4.2.34