From 0c08668ad7f9c32362ba15143d0276cdbedb8896 Mon Sep 17 00:00:00 2001
From: AlistairGrant <akgrant0710@gmail.com>
Date: Tue, 2 Oct 2018 08:46:48 +0200
Subject: [PATCH] FileAttributesPlugin 2.0.0: Fixes UTF8 on Windows

Restructure FileAttributesPlugin to move platform specific functionality
out to separate files. This is required as Windows requires the "wide"
versions of posix functions to be used, e.g. access() vs. _waccess(),
while other seem to require the Windows native functions, e.g. iterating
over directories, and interleaving #ifdef _WIN32 was becoming unwieldy.
---
 .../plugins/FileAttributesPlugin/faSupport.c  |  2 -
 .../FileAttributesPlugin.c                    | 75 ++-----------------
 2 files changed, 8 insertions(+), 69 deletions(-)

diff --git a/platforms/unix/plugins/FileAttributesPlugin/faSupport.c b/platforms/unix/plugins/FileAttributesPlugin/faSupport.c
index 469f77c4f4..8cfce07601 100644
--- a/platforms/unix/plugins/FileAttributesPlugin/faSupport.c
+++ b/platforms/unix/plugins/FileAttributesPlugin/faSupport.c
@@ -476,8 +476,6 @@ char	targetFile[FA_PATH_MAX];
  */
 sqInt faExists(fapath *aFaPath)
 {
-int		status;
-
 	if (access(faGetPlatPath(aFaPath), F_OK))
 		return interpreterProxy->falseObject();
 	else
diff --git a/src/plugins/FileAttributesPlugin/FileAttributesPlugin.c b/src/plugins/FileAttributesPlugin/FileAttributesPlugin.c
index 5c8393739e..6cb22b3bd0 100644
--- a/src/plugins/FileAttributesPlugin/FileAttributesPlugin.c
+++ b/src/plugins/FileAttributesPlugin/FileAttributesPlugin.c
@@ -1,9 +1,9 @@
 /* Automatically generated by
-	VMPluginCodeGenerator VMMaker.oscog-eem.2430 uuid: 4966a4b7-293a-4911-8f2d-396a7b97d82a
+	VMPluginCodeGenerator VMMaker.oscog-eem.2445 uuid: ecf80f10-9e24-4ff5-8a41-d65cb2690c94
    from
-	FileAttributesPlugin * FileAttributesPlugin.oscog-AlistairGrant.36 uuid: eb4e9dcc-d749-4b64-820b-b7fb8e9d9991
+	FileAttributesPlugin FileAttributesPlugin.oscog-AlistairGrant.38 uuid: f0fcdb0f-e642-4f87-8a9f-d35ffa0d05ff
  */
-static char __buildInfo[] = "FileAttributesPlugin * FileAttributesPlugin.oscog-AlistairGrant.36 uuid: eb4e9dcc-d749-4b64-820b-b7fb8e9d9991 " __DATE__ ;
+static char __buildInfo[] = "FileAttributesPlugin FileAttributesPlugin.oscog-AlistairGrant.38 uuid: f0fcdb0f-e642-4f87-8a9f-d35ffa0d05ff " __DATE__ ;
 
 
 
@@ -94,8 +94,6 @@ EXPORT(sqInt) primitiveStToPlatPath(void);
 EXPORT(sqInt) primitiveSymlinkChangeOwner(void);
 EXPORT(sqInt) primitiveVersionString(void);
 static sqInt processDirectory(fapath *faPath);
-static sqInt putLStatForintoBuffertargetName(char *cPathName, faStatStruct *statBufPointer, sqInt *fileNameOop);
-static sqInt putStatForintoBuffertargetName(char *cPathName, faStatStruct *statBufPointer, sqInt *fileNameOop);
 static sqInt readLinkintomaxLength(char *cPathName, char *cLinkPtr, size_t maxLength);
 EXPORT(sqInt) setInterpreter(struct VirtualMachine*anInterpreter);
 static sqInt squeakPathtoPlatformmaxLen(sqInt pathOop, char *cPathString, sqInt maxLength);
@@ -179,9 +177,9 @@ extern
 struct VirtualMachine* interpreterProxy;
 static const char *moduleName =
 #ifdef SQUEAK_BUILTIN_PLUGIN
-	"FileAttributesPlugin * FileAttributesPlugin.oscog-AlistairGrant.36 (i)"
+	"FileAttributesPlugin FileAttributesPlugin.oscog-AlistairGrant.38 (i)"
 #else
-	"FileAttributesPlugin * FileAttributesPlugin.oscog-AlistairGrant.36 (e)"
+	"FileAttributesPlugin FileAttributesPlugin.oscog-AlistairGrant.38 (e)"
 #endif
 ;
 static void * sCLPfn;
@@ -503,7 +501,7 @@ primitiveChangeMode(void)
 EXPORT(sqInt)
 primitiveChangeOwner(void)
 {
-    char cString[PATH_MAX+1];
+    char cString[FA_PATH_MAX];
     sqInt fileNameOop;
     sqInt groupId;
     sqInt ownerId;
@@ -518,7 +516,7 @@ primitiveChangeOwner(void)
 	}
 	
 #  if HAVE_CHOWN
-	squeakPathtoPlatformmaxLen(fileNameOop, cString, PATH_MAX);
+	squeakPathtoPlatformmaxLen(fileNameOop, cString, FA_PATH_MAX);
 	if (failed()) {
 		return primitiveFailForOSError(primitiveFailureCode());
 	}
@@ -978,7 +976,7 @@ primitiveSymlinkChangeOwner(void)
 EXPORT(sqInt)
 primitiveVersionString(void)
 {
-	popthenPush(1, stringFromCString("1.4.0d10"));
+	popthenPush(1, stringFromCString("2.0.0"));
 	return 0;
 }
 
@@ -1026,63 +1024,6 @@ processDirectory(fapath *faPath)
 }
 
 
-/*	Call stat() on cPathName, storing the results in
-	the buffer at statBufPointer. */
-
-	/* FileAttributesPlugin>>#putLStatFor:intoBuffer:targetName: */
-static sqInt
-putLStatForintoBuffertargetName(char *cPathName, faStatStruct *statBufPointer, sqInt *fileNameOop)
-{
-    char cLinkName[PATH_MAX];
-    sqInt len;
-    sqInt status;
-
-	
-#  if HAVE_LSTAT == 1
-	status = lstat(cPathName, statBufPointer);
-	if (status != 0) {
-		/* begin cantStatPath */
-		return -3;
-	}
-	if ((S_ISLNK(statBufPointer->st_mode)) == 0) {
-		fileNameOop[0] = (nilObject());
-	}
-	else {
-		len = readLinkintomaxLength(cPathName, cLinkName, PATH_MAX);
-		if (len < 0) {
-			return len;
-		}
-		status = faCharToByteArray(cLinkName, fileNameOop);
-	}
-#  else /* HAVE_LSTAT == 1 */
-
-	/* #HAVE_LSTAT = 1 */
-	/* begin invalidRequest */
-	status = -11;
-#  endif /* HAVE_LSTAT == 1 */
-	return status;
-}
-
-
-/*	Call stat() on cPathName, storing the results in
-	the buffer at statBufPointer. */
-
-	/* FileAttributesPlugin>>#putStatFor:intoBuffer:targetName: */
-static sqInt
-putStatForintoBuffertargetName(char *cPathName, faStatStruct *statBufPointer, sqInt *fileNameOop)
-{
-    sqInt status;
-
-	status = stat(cPathName, statBufPointer);
-	if (status != 0) {
-		/* begin cantStatPath */
-		return -3;
-	}
-	fileNameOop[0] = (nilObject());
-	return 0;
-}
-
-
 /*	Get the target filename of the supplied symbolic link. */
 
 	/* FileAttributesPlugin>>#readLink:into:maxLength: */