Merge pull request #11 from Archinamon/fix_abs_path

calculating abs path only once on init step

Author: Archinamon

.github/workflows/check-build.yml

@@ -23,6 +23,10 @@ jobs:
       - uses: fwilhe2/setup-kotlin@main
         with:
           install-native: true
+
+      - name: Select Xcode 14
+        if: ${{ startsWith(matrix.os, 'macos') }}
+        run: sudo xcode-select -s /Applications/Xcode_14.3.1.app/Contents/Developer
 
       - name: Cache Build files
         uses: actions/cache@v2

.github/workflows/gradle-publish.yml

@@ -34,6 +34,10 @@ jobs:
         restore-keys: |
           ${{ runner.os }}-gradle-
 
+    - name: Select Xcode 14
+      if: ${{ startsWith(matrix.os, 'macos') }}
+      run: sudo xcode-select -s /Applications/Xcode_14.3.1.app/Contents/Developer
+
     - name: Build with Gradle [Macos]
       uses: eskatos/gradle-command-action@v1
       if: ${{ startsWith(matrix.os, 'macos') }}

.github/workflows/tests.yml

@@ -25,6 +25,9 @@ jobs:
             ~/.konan
             ~/.gradle
           key: ${{ runner.os }}-${{ hashFiles('gradle.properties') }}
+      - name: Select Xcode 14
+        if: ${{ startsWith(matrix.os, 'macos') }}
+        run: sudo xcode-select -s /Applications/Xcode_14.3.1.app/Contents/Developer
       - uses: eskatos/gradle-command-action@v1
         name: Test Windows Target
         if: ${{ startsWith(matrix.os, 'windows') }}

src/posixMain/kotlin/File.kt

@@ -51,6 +51,13 @@ actual class File actual constructor(
     private val modeAppend = "a"
     private val modeRewrite = "w"
 
+    private val absPath: String
+
+    init {
+        // not lazy but preserve on obj init
+        absPath = getRealAbsolutePath()
+    }
+
     actual fun getParent(): String? {
         val path = getAbsolutePath()
         val idx = path.lastIndexOf(filePathSeparator)
@@ -76,6 +83,14 @@ actual class File actual constructor(
     actual fun getPath(): String = pathname
 
     actual fun getAbsolutePath(): String {
+        if (absPath.isNotBlank()) {
+            return absPath
+        }
+
+        return getRealAbsolutePath()
+    }
+
+    private fun getRealAbsolutePath(): String {
         return memScoped {
             val path = if (pathname.first() != filePathSeparator || pathname.first() == '.') {
                 getcwd(allocArray(FILENAME_MAX), FILENAME_MAX.convert())
@@ -309,7 +324,7 @@ actual fun File.readBytes(): ByteArray {
 }
 
 actual fun File.writeBytes(bytes: ByteArray) {
-    // no need to use pinning or memscope, cause it's inside the method already does
+    // no need to use pinning or memscope, 'cause it's inside the method already does
     writeBytes(bytes, O_RDWR, bytes.size.convert(), Byte.SIZE_BYTES.convert())
 }