moe consistent use of white space to match coding standard etc

git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/base/trunk@34809 72102866-910b-0410-8b05-ffd578937521
This commit is contained in:
rfm 2012-02-23 17:57:50 +00:00
parent 2bd8c521dc
commit c0585bf7b2
4 changed files with 128 additions and 126 deletions

View file

@ -66,7 +66,8 @@ static const NSMatchingOptions NSMatchingWithTransparentBounds = 1<<3;
static const NSMatchingOptions NSMatchingWithoutAnchoringBounds = 1<<4; static const NSMatchingOptions NSMatchingWithoutAnchoringBounds = 1<<4;
DEFINE_BLOCK_TYPE(GSRegexBlock, void, NSTextCheckingResult*, NSMatchingFlags, BOOL*); DEFINE_BLOCK_TYPE(GSRegexBlock, void, NSTextCheckingResult*,
NSMatchingFlags, BOOL*);
#ifndef GSREGEXTYPE #ifndef GSREGEXTYPE
# define GSREGEXTYPE void # define GSREGEXTYPE void
@ -93,54 +94,54 @@ DEFINE_BLOCK_TYPE(GSRegexBlock, void, NSTextCheckingResult*, NSMatchingFlags, BO
// implementation. If you have configured GNUstep without this support then it // implementation. If you have configured GNUstep without this support then it
// will not work, so these methods are hidden. // will not work, so these methods are hidden.
#if GS_USE_ICU || GS_UNSAFE_REGEX #if GS_USE_ICU || GS_UNSAFE_REGEX
+ (NSRegularExpression*)regularExpressionWithPattern: (NSString*)aPattern + (NSRegularExpression*) regularExpressionWithPattern: (NSString*)aPattern
options: (NSRegularExpressionOptions)opts options: (NSRegularExpressionOptions)opts
error: (NSError**)e; error: (NSError**)e;
- initWithPattern: (NSString*)aPattern - (id) initWithPattern: (NSString*)aPattern
options: (NSRegularExpressionOptions)opts options: (NSRegularExpressionOptions)opts
error: (NSError**)e; error: (NSError**)e;
+ (NSRegularExpression*)regularExpressionWithPattern: (NSString*)aPattern + (NSRegularExpression*) regularExpressionWithPattern: (NSString*)aPattern
options: (NSRegularExpressionOptions)opts options: (NSRegularExpressionOptions)opts
error: (NSError**)e; error: (NSError**)e;
- initWithPattern: (NSString*)aPattern - (id) initWithPattern: (NSString*)aPattern
options: (NSRegularExpressionOptions)opts options: (NSRegularExpressionOptions)opts
error: (NSError**)e; error: (NSError**)e;
- (NSString*)pattern; - (NSString*) pattern;
- (void)enumerateMatchesInString: (NSString*)string - (void) enumerateMatchesInString: (NSString*)string
options: (NSMatchingOptions)options options: (NSMatchingOptions)options
range: (NSRange)range range: (NSRange)range
usingBlock: (GSRegexBlock)block; usingBlock: (GSRegexBlock)block;
- (NSUInteger)numberOfMatchesInString: (NSString*)string - (NSUInteger) numberOfMatchesInString: (NSString*)string
options: (NSMatchingOptions)options
range: (NSRange)range;
- (NSTextCheckingResult*) firstMatchInString: (NSString*)string
options: (NSMatchingOptions)options
range: (NSRange)range;
- (NSArray*) matchesInString: (NSString*)string
options: (NSMatchingOptions)options
range: (NSRange)range;
- (NSRange) rangeOfFirstMatchInString: (NSString*)string
options: (NSMatchingOptions)options options: (NSMatchingOptions)options
range: (NSRange)range; range: (NSRange)range;
- (NSUInteger) replaceMatchesInString: (NSMutableString*)string
- (NSTextCheckingResult*)firstMatchInString: (NSString*)string options: (NSMatchingOptions)options
options: (NSMatchingOptions)options range: (NSRange)range
range: (NSRange)range; withTemplate: (NSString*)templat;
- (NSArray*)matchesInString: (NSString*)string - (NSString*) stringByReplacingMatchesInString: (NSString*)string
options: (NSMatchingOptions)options options: (NSMatchingOptions)options
range: (NSRange)range; range: (NSRange)range
- (NSRange)rangeOfFirstMatchInString: (NSString*)string withTemplate: (NSString*)templat;
options: (NSMatchingOptions)options - (NSString*) replacementStringForResult: (NSTextCheckingResult*)result
range: (NSRange)range; inString: (NSString*)string
- (NSUInteger)replaceMatchesInString: (NSMutableString*)string offset: (NSInteger)offset
options: (NSMatchingOptions)options template: (NSString*)templat;
range: (NSRange)range
withTemplate: (NSString*)templat;
- (NSString*)stringByReplacingMatchesInString: (NSString*)string
options: (NSMatchingOptions)options
range: (NSRange)range
withTemplate: (NSString*)templat;
- (NSString*)replacementStringForResult: (NSTextCheckingResult*)result
inString: (NSString*)string
offset: (NSInteger)offset
template: (NSString*)templat;
#if GS_HAS_DECLARED_PROPERTIES #if GS_HAS_DECLARED_PROPERTIES
@property (readonly) NSRegularExpressionOptions options; @property (readonly) NSRegularExpressionOptions options;
@property (readonly) NSUInteger numberOfCaptureGroups; @property (readonly) NSUInteger numberOfCaptureGroups;
#else #else
- (NSRegularExpressionOptions)options; - (NSRegularExpressionOptions) options;
- (NSUInteger)numberOfCaptureGroups; - (NSUInteger) numberOfCaptureGroups;
#endif #endif
#endif // GS_USE_ICU #endif // GS_USE_ICU
@end @end

View file

@ -29,10 +29,11 @@
#if GS_USE_ICU == 1 #if GS_USE_ICU == 1
#include "unicode/uregex.h" #include "unicode/uregex.h"
// FIXME It would be nice to use autoconf for checking whether uregex_openUText /* FIXME It would be nice to use autoconf for checking whether uregex_openUText
// is defined. However the naive check using AC_CHECK_FUNCS(uregex_openUText) * is defined. However the naive check using AC_CHECK_FUNCS(uregex_openUText)
// wonn't work because libicu internally renames all entry points with some cpp * wonn't work because libicu internally renames all entry points with some cpp
// magic. * magic.
*/
#if (U_ICU_VERSION_MAJOR_NUM > 4 || (U_ICU_VERSION_MAJOR_NUM == 4 && U_ICU_VERSION_MINOR_NUM >= 4)) #if (U_ICU_VERSION_MAJOR_NUM > 4 || (U_ICU_VERSION_MAJOR_NUM == 4 && U_ICU_VERSION_MINOR_NUM >= 4))
#define HAVE_UREGEX_OPENUTEXT 1 #define HAVE_UREGEX_OPENUTEXT 1
#endif #endif
@ -69,7 +70,7 @@ NSRegularExpressionOptionsToURegexpFlags(NSRegularExpressionOptions opts)
} }
if (opts & NSRegularExpressionIgnoreMetacharacters) if (opts & NSRegularExpressionIgnoreMetacharacters)
{ {
flags |= UREGEX_LITERAL; flags |= UREGEX_LITERAL;
} }
if (opts & NSRegularExpressionDotMatchesLineSeparators) if (opts & NSRegularExpressionDotMatchesLineSeparators)
{ {

View file

@ -202,7 +202,7 @@ extern void clearPrivatePointers(xmlNodePtr aNode);
options |= XML_PARSE_NOBLANKS; options |= XML_PARSE_NOBLANKS;
//xmlKeepBlanksDefault(0); //xmlKeepBlanksDefault(0);
doc = xmlReadDoc((xmlChar *)str, url, encoding, options); doc = xmlReadDoc((xmlChar *)str, url, encoding, options);
if(doc == NULL) if (doc == NULL)
{ {
[self release]; [self release];
self = nil; self = nil;
@ -258,7 +258,7 @@ extern void clearPrivatePointers(xmlNodePtr aNode);
{ {
id oldElement = [self rootElement]; id oldElement = [self rootElement];
if(root == nil) if (root == nil)
{ {
return; return;
} }
@ -410,11 +410,11 @@ extern void clearPrivatePointers(xmlNodePtr aNode);
NSUInteger index = 0; NSUInteger index = 0;
// Iterate over the keys and put them into params... // Iterate over the keys and put them into params...
if(arguments != nil) if (arguments != nil)
{ {
int count = [[arguments allKeys] count]; int count = [[arguments allKeys] count];
*params = NSZoneCalloc([self zone],((count + 1) * 2),sizeof(xmlChar *)); *params = NSZoneCalloc([self zone],((count + 1) * 2),sizeof(xmlChar *));
while((key = [en nextObject]) != nil) while ((key = [en nextObject]) != nil)
{ {
params[index] = (xmlChar *)XMLSTRING(key); params[index] = (xmlChar *)XMLSTRING(key);
params[index+1] = (xmlChar *)XMLSTRING([arguments objectForKey: key]); params[index+1] = (xmlChar *)XMLSTRING([arguments objectForKey: key]);

View file

@ -193,9 +193,9 @@ BOOL isEqualTree(xmlNodePtr nodeA, xmlNodePtr nodeB)
- (void *) _node; - (void *) _node;
- (void) _setNode: (void *)_anode; - (void) _setNode: (void *)_anode;
+ (NSXMLNode *) _objectForNode: (xmlNodePtr)node; + (NSXMLNode *) _objectForNode: (xmlNodePtr)node;
- (void) _addSubNode:(NSXMLNode *)subNode; - (void) _addSubNode: (NSXMLNode *)subNode;
- (void) _removeSubNode:(NSXMLNode *)subNode; - (void) _removeSubNode: (NSXMLNode *)subNode;
- (id) _initWithNode:(xmlNodePtr)node kind:(NSXMLNodeKind)kind; - (id) _initWithNode: (xmlNodePtr)node kind: (NSXMLNodeKind)kind;
- (xmlNodePtr) _childNodeAtIndex: (NSUInteger)index; - (xmlNodePtr) _childNodeAtIndex: (NSUInteger)index;
- (void) _insertChild: (NSXMLNode*)child atIndex: (NSUInteger)index; - (void) _insertChild: (NSXMLNode*)child atIndex: (NSUInteger)index;
- (void) _updateExternalRetains; - (void) _updateExternalRetains;
@ -272,7 +272,7 @@ BOOL isEqualTree(xmlNodePtr nodeA, xmlNodePtr nodeB)
int extraRetains = ([self retainCount] > 1 ? 1 : 0); // start with 1 or 0 for ourself int extraRetains = ([self retainCount] > 1 ? 1 : 0); // start with 1 or 0 for ourself
int index; int index;
for (index = 0; index < [internal->subNodes count]; index++) for (index = 0; index < [internal->subNodes count]; index++)
extraRetains += [[internal->subNodes objectAtIndex:index] _externalRetains]; extraRetains += [[internal->subNodes objectAtIndex: index] _externalRetains];
return extraRetains; return extraRetains;
} }
@ -284,7 +284,7 @@ BOOL isEqualTree(xmlNodePtr nodeA, xmlNodePtr nodeB)
int extraRetains = ([self retainCount] > 1 ? 1 : 0); // start with 1 or 0 for ourself int extraRetains = ([self retainCount] > 1 ? 1 : 0); // start with 1 or 0 for ourself
int index; int index;
for (index = 0; index < [internal->subNodes count]; index++) for (index = 0; index < [internal->subNodes count]; index++)
extraRetains += [[internal->subNodes objectAtIndex:index] _externalRetains]; extraRetains += [[internal->subNodes objectAtIndex: index] _externalRetains];
internal->externalRetains = extraRetains; internal->externalRetains = extraRetains;
if (extraRetains != oldCount) if (extraRetains != oldCount)
{ {
@ -326,12 +326,12 @@ BOOL isEqualTree(xmlNodePtr nodeA, xmlNodePtr nodeB)
} }
} }
- (void) _passExternalRetainsTo:(NSXMLNode *)parent - (void) _passExternalRetainsTo: (NSXMLNode *)parent
{ {
// this object just became a subNode, so pass knowledge of external retains up the line // this object just became a subNode, so pass knowledge of external retains up the line
if (internal->externalRetains > 0) if (internal->externalRetains > 0)
{ {
//NSLog(@"_passExternalRetainsTo:%@ (%d,%d) from %@ Start:(%d,%d)", parent, [parent _externalRetains], [parent verifyExternalRetains], self, internal->externalRetains, [self verifyExternalRetains]); //NSLog(@"_passExternalRetainsTo: %@ (%d,%d) from %@ Start: (%d,%d)", parent, [parent _externalRetains], [parent verifyExternalRetains], self, internal->externalRetains, [self verifyExternalRetains]);
// if ([self retainCount] == 2) // if ([self retainCount] == 2)
// { // {
// internal->externalRetains--; // internal->externalRetains--;
@ -345,21 +345,21 @@ BOOL isEqualTree(xmlNodePtr nodeA, xmlNodePtr nodeB)
//NSLog(@"RELEASED SELF %@ (%d) in _passExternal...", self, internal->retainedSelf); //NSLog(@"RELEASED SELF %@ (%d) in _passExternal...", self, internal->retainedSelf);
} }
[parent _updateExternalRetains]; [parent _updateExternalRetains];
//NSLog(@"DID _passExternalRetainsTo:%@ (%d,%d) from %@ End:(%d,%d)", parent, [parent _externalRetains], [parent verifyExternalRetains], self, internal->externalRetains, [self verifyExternalRetains]); //NSLog(@"DID _passExternalRetainsTo: %@ (%d,%d) from %@ End: (%d,%d)", parent, [parent _externalRetains], [parent verifyExternalRetains], self, internal->externalRetains, [self verifyExternalRetains]);
} }
} }
- (void) _removeExternalRetainsFrom:(NSXMLNode *)parent - (void) _removeExternalRetainsFrom: (NSXMLNode *)parent
{ {
// this object is no longer a subNode, so pass removal of external retains up the line // this object is no longer a subNode, so pass removal of external retains up the line
if (internal->externalRetains > 0) if (internal->externalRetains > 0)
{ {
//NSLog(@"_removeExternalRetainsTo:%@ from %@ Start: %d", parent, self, internal->externalRetains); //NSLog(@"_removeExternalRetainsTo: %@ from %@ Start: %d", parent, self, internal->externalRetains);
/// [parent releaseExternalRetain:internal->externalRetains]; /// [parent releaseExternalRetain: internal->externalRetains];
if ([self retainCount] == 1) if ([self retainCount] == 1)
{ {
internal->externalRetains++; internal->externalRetains++;
//NSLog(@"ADDED TRICKY EXTRA COUNT WHILE REMOVING FROM PARENT in %@ now: %d subNodes(low):%d", self, internal->externalRetains, [self verifyExternalRetains]); //NSLog(@"ADDED TRICKY EXTRA COUNT WHILE REMOVING FROM PARENT in %@ now: %d subNodes(low): %d", self, internal->externalRetains, [self verifyExternalRetains]);
} }
[super retain]; // becoming detached, so retain ourself as the new root of our branch [super retain]; // becoming detached, so retain ourself as the new root of our branch
@ -373,18 +373,18 @@ BOOL isEqualTree(xmlNodePtr nodeA, xmlNodePtr nodeB)
{ {
if (!internal->subNodes) if (!internal->subNodes)
internal->subNodes = [[NSMutableArray alloc] init]; internal->subNodes = [[NSMutableArray alloc] init];
if ([internal->subNodes indexOfObjectIdenticalTo:subNode] == NSNotFound) if ([internal->subNodes indexOfObjectIdenticalTo: subNode] == NSNotFound)
{ {
[internal->subNodes addObject:subNode]; [internal->subNodes addObject: subNode];
[subNode _passExternalRetainsTo:self]; [subNode _passExternalRetainsTo: self];
} }
} }
- (void) _removeSubNode: (NSXMLNode *)subNode - (void) _removeSubNode: (NSXMLNode *)subNode
{ {
[subNode retain]; // retain temporarily so we can safely remove from our subNodes list first [subNode retain]; // retain temporarily so we can safely remove from our subNodes list first
[internal->subNodes removeObjectIdenticalTo:subNode]; [internal->subNodes removeObjectIdenticalTo: subNode];
[subNode _removeExternalRetainsFrom:self]; [subNode _removeExternalRetainsFrom: self];
[subNode release]; // release temporary hold [subNode release]; // release temporary hold
} }
@ -443,14 +443,14 @@ BOOL isEqualTree(xmlNodePtr nodeA, xmlNodePtr nodeB)
{ {
addedNode = xmlAddChild(parentNode, childNode); addedNode = xmlAddChild(parentNode, childNode);
} }
else //if(index < childCount) else //if (index < childCount)
{ {
addedNode = xmlAddPrevSibling(curNode, childNode); addedNode = xmlAddPrevSibling(curNode, childNode);
} }
if (addedNode != childNode) if (addedNode != childNode)
{ {
[child _setNode:NULL]; [child _setNode: NULL];
child = [NSXMLNode _objectForNode:addedNode]; child = [NSXMLNode _objectForNode: addedNode];
} }
} }
else else
@ -493,7 +493,7 @@ BOOL isEqualTree(xmlNodePtr nodeA, xmlNodePtr nodeB)
} }
} }
[self _addSubNode:child]; [self _addSubNode: child];
} }
- (void) _invalidate - (void) _invalidate
@ -537,8 +537,8 @@ int register_namespaces(xmlXPathContextPtr xpathCtx,
while (next != NULL) while (next != NULL)
{ {
/* skip spaces */ /* skip spaces */
while((*next) == ' ') next++; while ((*next) == ' ') next++;
if((*next) == '\0') break; if ((*next) == '\0') break;
/* find prefix */ /* find prefix */
prefix = next; prefix = next;
@ -605,7 +605,7 @@ NSArray *execute_xpath(NSXMLNode *node,
return nil; return nil;
} }
if (![xpath_exp hasPrefix:@"/"]) if (![xpath_exp hasPrefix: @"/"])
xpathCtx->node = (xmlNodePtr)doc; // provide a context for relative paths xpathCtx->node = (xmlNodePtr)doc; // provide a context for relative paths
/* Evaluate xpath expression */ /* Evaluate xpath expression */
@ -782,27 +782,27 @@ NSArray *execute_xpath(NSXMLNode *node,
if ([name isEqualToString: @"xml"]) if ([name isEqualToString: @"xml"])
{ {
return [self namespaceWithName: @"xml" return [self namespaceWithName: @"xml"
stringValue: @"http://www.w3.org/XML/1998/namespace"]; stringValue: @"http: //www.w3.org/XML/1998/namespace"];
} }
if ([name isEqualToString: @"xs"]) if ([name isEqualToString: @"xs"])
{ {
return [self namespaceWithName: @"xs" return [self namespaceWithName: @"xs"
stringValue: @"http://www.w3.org/2001/XMLSchema"]; stringValue: @"http: //www.w3.org/2001/XMLSchema"];
} }
if ([name isEqualToString: @"xsi"]) if ([name isEqualToString: @"xsi"])
{ {
return [self namespaceWithName: @"xsi" return [self namespaceWithName: @"xsi"
stringValue: @"http://www.w3.org/2001/XMLSchema-instance"]; stringValue: @"http: //www.w3.org/2001/XMLSchema-instance"];
} }
if ([name isEqualToString: @"fn"]) if ([name isEqualToString: @"fn"])
{ {
return [self namespaceWithName: @"fn" return [self namespaceWithName: @"fn"
stringValue: @"http://www.w3.org/2003/11/xpath-functions"]; stringValue: @"http: //www.w3.org/2003/11/xpath-functions"];
} }
if ([name isEqualToString: @"local"]) if ([name isEqualToString: @"local"])
{ {
return [self namespaceWithName: @"local" return [self namespaceWithName: @"local"
stringValue: @"http://www.w3.org/2003/11/xpath-local-functions"]; stringValue: @"http: //www.w3.org/2003/11/xpath-local-functions"];
} }
return nil; return nil;
@ -854,7 +854,7 @@ NSArray *execute_xpath(NSXMLNode *node,
- (NSXMLNode*) childAtIndex: (NSUInteger)index - (NSXMLNode*) childAtIndex: (NSUInteger)index
{ {
xmlNodePtr childNode = [self _childNodeAtIndex:index]; xmlNodePtr childNode = [self _childNodeAtIndex: index];
return [NSXMLNode _objectForNode: childNode]; return [NSXMLNode _objectForNode: childNode];
} }
@ -907,7 +907,7 @@ NSArray *execute_xpath(NSXMLNode *node,
clearPrivatePointers(newNode); clearPrivatePointers(newNode);
//c = [c initWithKind: internal->kind options: internal->options]; //c = [c initWithKind: internal->kind options: internal->options];
//[c _setNode:newNode]; //[c _setNode: newNode];
c = [c _initWithNode: newNode kind: internal->kind]; c = [c _initWithNode: newNode kind: internal->kind];
@ -978,17 +978,17 @@ NSArray *execute_xpath(NSXMLNode *node,
extraRetains = internal->externalRetains; //[self verifyExternalRetains]; extraRetains = internal->externalRetains; //[self verifyExternalRetains];
if (extraRetains) if (extraRetains)
{ {
/// [parent releaseExternalRetain:extraRetains]; /// [parent releaseExternalRetain: extraRetains];
if ([self retainCount] == 1) if ([self retainCount] == 1)
{ {
internal->externalRetains++; internal->externalRetains++;
//NSLog(@"ADDED TRICKY EXTRA COUNT WHILE DETACHING in %@ now: %d subNodes(low):%d", self, internal->externalRetains, [self verifyExternalRetains]); //NSLog(@"ADDED TRICKY EXTRA COUNT WHILE DETACHING in %@ now: %d subNodes(low): %d", self, internal->externalRetains, [self verifyExternalRetains]);
} }
[super retain]; //[self recordExternalRetain:extraRetains]; [super retain]; //[self recordExternalRetain: extraRetains];
internal->retainedSelf++; internal->retainedSelf++;
//NSLog(@"RETAINED SELF %@ (%d) in detach", self, internal->retainedSelf); //NSLog(@"RETAINED SELF %@ (%d) in detach", self, internal->retainedSelf);
} }
[parent _removeSubNode:self]; [parent _removeSubNode: self];
} }
//NSLog(@"DETACHED %@ from %@ and transferred extra retains: %d", self, parent, extraRetains); //NSLog(@"DETACHED %@ from %@ and transferred extra retains: %d", self, parent, extraRetains);
} }
@ -1301,7 +1301,7 @@ NSArray *execute_xpath(NSXMLNode *node,
- (void) setObjectValue: (id)value - (void) setObjectValue: (id)value
{ {
if(nil == value) if (nil == value)
{ {
ASSIGN(internal->objectValue, [NSString stringWithString: @""]); ASSIGN(internal->objectValue, [NSString stringWithString: @""]);
return; return;
@ -1356,7 +1356,7 @@ NSArray *execute_xpath(NSXMLNode *node,
- (NSString*) URI - (NSString*) URI
{ {
if(NSXMLInvalidKind == internal->kind) if (NSXMLInvalidKind == internal->kind)
{ {
return nil; return nil;
} }
@ -1400,7 +1400,7 @@ NSArray *execute_xpath(NSXMLNode *node,
- (NSArray*) nodesForXPath: (NSString*)anxpath error: (NSError**)error - (NSArray*) nodesForXPath: (NSString*)anxpath error: (NSError**)error
{ {
if(error != NULL) if (error != NULL)
{ {
*error = NULL; *error = NULL;
} }