[Skim-app-commit] SF.net SVN: skim-app:[14933] trunk/PDFSelection_SKE xtensions.m

[hofman--- via Skim-app-commit]

Revision: 14933
          http://sourceforge.net/p/skim-app/code/14933
Author:   hofman
Date:     2025-03-02 16:16:25 +0000 (Sun, 02 Mar 2025)
Log Message:
-----------
don't use extra inline function

Modified Paths:
--------------
    trunk/PDFSelection_SKExtensions.m

Modified: trunk/PDFSelection_SKExtensions.m
===================================================================
--- trunk/PDFSelection_SKExtensions.m   2025-03-02 10:26:04 UTC (rev 14932)
+++ trunk/PDFSelection_SKExtensions.m   2025-03-02 16:16:25 UTC (rev 14933)
@@ -606,26 +606,23 @@
     }
 }
 
-static inline NSRange 
addSpecifierWithCharacterRangeAndPageOrAppendRange(NSMutableArray *ranges, 
NSRange nextRange, NSRange range, PDFPage *page) {
-    if (nextRange.length == 0) {
-    } else if (range.length == 0) {
-        range = nextRange;
-    } else if (NSMaxRange(range) == nextRange.location) {
-        range.length += nextRange.length;
-    } else {
-        addSpecifierWithCharacterRangeAndPage(ranges, range, page);
-        range = nextRange;
-    }
-    return range;
-}
-
 - (id)objectSpecifiers {
     NSMutableArray *ranges = [NSMutableArray array];
     for (PDFPage *page in [self pages]) {
         NSRange range = NSMakeRange(0, 0);
         NSInteger i, iMax = [self numberOfTextRangesOnPage:page];
-        for (i = 0; i < iMax; i++)
-            range = addSpecifierWithCharacterRangeAndPageOrAppendRange(ranges, 
[self rangeAtIndex:i onPage:page], range, page);
+        for (i = 0; i < iMax; i++) {
+            NSRange nextRange = [self rangeAtIndex:i onPage:page];
+            if (nextRange.length == 0) {
+            } else if (range.length == 0) {
+                range = nextRange;
+            } else if (NSMaxRange(range) == nextRange.location) {
+                range.length += nextRange.length;
+            } else {
+                addSpecifierWithCharacterRangeAndPage(ranges, range, page);
+                range = nextRange;
+            }
+        }
         if (range.length)
             addSpecifierWithCharacterRangeAndPage(ranges, range, page);
     }

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.



_______________________________________________
Skim-app-commit mailing list
Skim-app-commit@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/skim-app-commit