- turned off _debug framework support in tests since we now capture a lot more

  in log validation.
- Added GTM_SUPPORT_GC for controlling the inclusion of GC related code.
- If you are using GTMUnitTestDevLog, it also tries to capture logs from
  NSAssert.
- Added GTM_FOREACH_OBJECT/GTM_FOREACH_KEY that uses NSEnumerator and
  objectEnumerator/keyEnumerator on 10.4, but on 10.5+/iPhone uses
  FastEnumeration.
- GTMNSWorkspace+Running gives a variety of ways of determining the attributes
  of running processes.

1 files changed, 12 insertions, 8 deletions

diff --git a/AppKit/GTMNSImage+Scaling.m b/AppKit/GTMNSImage+Scaling.m
index 5f31190..8283b29 100644
--- a/AppKit/GTMNSImage+Scaling.m
+++ b/AppKit/GTMNSImage+Scaling.m
@@ -36,8 +36,7 @@
   CGFloat repDistance = CGFLOAT_MAX;
   
   NSImageRep *thisRep;
-  NSEnumerator *repEnum = [reps objectEnumerator];
-  while ((thisRep = [repEnum nextObject])) {
+  GTM_FOREACH_OBJECT(thisRep, reps) {
     CGFloat thisDistance;
     thisDistance = MIN(size.width - [thisRep size].width,
                        size.height - [thisRep size].height);  
@@ -62,8 +61,7 @@
   NSArray *reps = [self representations];
   
   NSImageRep *thisRep;
-  NSEnumerator *repEnum = [reps objectEnumerator];
-  while ((thisRep = [repEnum nextObject])) {
+  GTM_FOREACH_OBJECT(thisRep, reps) {
     if (NSEqualSizes([thisRep size], size)) {
       return thisRep;
     }
@@ -155,12 +153,18 @@
 }
 
 - (void)gtm_removeRepresentationsLargerThanSize:(NSSize)size {
-  NSEnumerator *e = [[self representations] reverseObjectEnumerator];
+  NSMutableArray *repsToRemove = [NSMutableArray array];
   NSImageRep *thisRep;
-  while((thisRep = [e nextObject]) ) {
+  // Remove them in a second loop so we don't change things will doing the
+  // initial loop.
+  GTM_FOREACH_OBJECT(thisRep, [self representations]) {
     if ([thisRep size].width > size.width
-        && [thisRep size].height > size.height)
-      [self removeRepresentation:thisRep];
+        && [thisRep size].height > size.height) {
+      [repsToRemove addObject:thisRep];
+    }
+  }
+  GTM_FOREACH_OBJECT(thisRep, repsToRemove) {
+    [self removeRepresentation:thisRep];
   }
 }