Changeset 227 for cpp/common/nonstd_stdio.cpp

Timestamp:

04/20/14 01:48:23 (11 years ago)

Author:

Maciej Komosinski

Message:

Android compilation and access to RESOURCES and HOME files

File:

• cpp/common/nonstd_stdio.cpp (modified) (5 diffs)

cpp/common/nonstd_stdio.cpp

@@ -139 +139 @@
 
 #ifdef __ANDROID__
-//#include "framsg.h"
+#include "framsg.h"
 #include "nonstd.h"
+#include "nonstd_stl.h"
 MFILE *mfopen(const char *path, const char *mode)
 {
-        NvFile *rfile=NULL; //umie tylko czytac
+        string respath=GET_APP_RESOURCES; //the macro can be char* or std::string, we don't know (nonstd.h, INITIAL_DIR_IS_RES, cwd.cpp) so we convert it to std::string
+        //printFM("Opening '%s', mode='%s'",path,mode);
+        //printFM("GET_APP_RESOURCES='%s'",respath.c_str());
+        NvFile *rfile=NULL; //can only read
         FILE *rwfile=NULL;
-        if (strstr(path,GET_APP_RESOURCES)==path) //otwieramy resource! wiec uzywamy czytania z assets
-        {
-                if (path[0]=='.' && path[1]=='/') path+=2; //nie rozpoznaje sciezek "./costam", a w sailorze zrobiles tak ze nie moze byc pusty path bo to jest znacznik ze jest niezainicjowany, dlatego uzywasz "./" i musimy je tu obcinac
-                rfile=NvFOpen(path); //"mode" not supported! umie tylko czytac
+        if (strstr(path,respath.c_str())==path) //opening resource! so we use a dedicated way to read from assets
+        {
+                path+=respath.length(); //strip the prefix, we need a relative path in assets
+                if (strstr(mode,"w"))
+                        printFM("Warning: attempt to open a read-only resource '%s' in writable mode '%s'",path,mode);
+                rfile=NvFOpen(path); //"mode" not supported! can only read
+                //printFM("Opened RES file as %p",rfile);
                 if (rfile==NULL) return NULL;
-        } else //"normalny" dostep (do HOME)
+        } else //a "normal" access (HOME)
         {
                 rwfile=fopen(path,mode);
+                //printFM("Opened HOME file as %p",rwfile);
                 if (rwfile==NULL) return NULL;
         }
@@ -174 +182 @@
 {
         if (f->rfile)
-                return size==0?0:NvFRead(ptr, size, count, f->rfile)/size; //blad nvidii w interpretacji zwracanej wartosci - zwraca liczbe bajtow zamiast liczbe porcji
+                return NvFRead(ptr, size, count, f->rfile); //nvidia introduced my corrections in SDK v10.14, so a fix is no longer needed here
         else
                 return fread(ptr, size, count, f->rwfile);
@@ -182 +190 @@
 {
         if (f->rfile)
-                return 0; //write not supported funkcjami nvidii
+                return 0; //write not supported in assets using nvidia functions
         else
                 return fwrite(ptr, size, count, f->rwfile);
@@ -197 +205 @@
 {
         if (f->rfile)
-                return NvFGets(str, num, f->rfile);
+        {
+                char *ret=NvFGets(str, num, f->rfile);
+                //fixing nvidia inconsistency... their function never returns NULL (fix submitted)
+                if (ret!=NULL && *ret==0 && num>0) //nothing has been read, must have been eof
+                        return NULL;
+                return ret;
+        }
         else
                 return fgets(str,num,f->rwfile);
@@ -213 +227 @@
 {
         if (f->rfile)
-                return NvFSeek(f->rfile, position, type)==-1; // off_t AAsset_seek(AAsset* asset, off_t offset, int whence): Returns the new position on success, or (off_t) -1 on error.
+                return NvFSeek(f->rfile, position, type); //nvidia introduced my corrections in SDK v10.14, so a fix is no longer needed here
         else
                 return fseek(f->rwfile, position, type);