Implement (and document) a MacOS-X compatibility behavior

git-svn-id: svn+ssh://svn.gna.org/svn/gnustep/libs/base/trunk@19909 72102866-910b-0410-8b05-ffd578937521

ChangeLog

@@ -2,6 +2,9 @@
 
 	* Source/NSPropertyList.m: Always write date using GMT timezone for
 	consistency.  Make XML representation MacOS-X compatible.
+	* Source/NSIndexSet.m: ([getIndexes:maxCount:inIndexRange:]) accept
+	null pointer for index range.  Consistent with undocumented MacOS-X
+	feature (but we document it).
 
 2004-08-23 Richard Frith-Macdonald <rfm@gnu.org>
 

Headers/Foundation/NSIndexSet.h

@@ -94,6 +94,8 @@
  * Only copies index values present in aRange and copies them in order.<br />
  * Returns the number of index values placed in aVuffer.<br />
  * Modifies aRange to start after the last index value copied.<br />
+ * If aRange is a null pointer, this method attempts to get <em>all</em>
+ * index values from the set (and of course no range can be returned in it).
  */
 - (unsigned int) getIndexes: (unsigned int*)aBuffer
 		   maxCount: (unsigned int)aCount

Source/NSIndexSet.m

@@ -280,14 +280,20 @@ static unsigned posForIndex(GSIArray array, unsigned index)
   unsigned	pos;
   unsigned	i = 0;
   NSRange	r;
+  NSRange	fullRange;
 
-  if (aBuffer == 0 || aRange == 0)
+  if (aBuffer == 0)
     {
       [NSException raise: NSInvalidArgumentException
 		  format: @"[%@-%@]: nul pointer argument",
         NSStringFromClass([self class]), NSStringFromSelector(_cmd)];
     }
-  if (NSNotFound - aRange->length < aRange->location)
+  if (aRange == 0)
+    {
+      fullRange = (NSRange){0, NSNotFound};
+      aRange = &fullRange;
+    }
+  else if (NSNotFound - aRange->length < aRange->location)
     {
       [NSException raise: NSInvalidArgumentException
 		  format: @"[%@-%@]: Bad range",