Fixed remaining int vs. long warnings.

[wxWidgets.git] / src / html / helpdata.cpp

diff --git a/src/html/helpdata.cpp b/src/html/helpdata.cpp
index 2781e510a719ee631184b0a7d02a45848baa5603..cb5eb0e862144830bebb3a1007ec82566daba95a 100644 (file)
--- a/src/html/helpdata.cpp
+++ b/src/html/helpdata.cpp
@@ -32,6 +32,9 @@
 #include "wx/tokenzr.h"
 #include "wx/wfstream.h"
 #include "wx/busyinfo.h"
 #include "wx/tokenzr.h"
 #include "wx/wfstream.h"
 #include "wx/busyinfo.h"

+#include "wx/encconv.h"
+#include "wx/fontmap.h"
+#include "wx/log.h"

 #include "wx/html/htmlpars.h"
 #include "wx/html/htmldefs.h"
 
 #include "wx/html/htmlpars.h"
 #include "wx/html/htmldefs.h"
 


@@ -55,18 +58,6 @@ static char* ReadLine(char *line, char *buf)
 }
 
 
 }
 
 

-static wxString SafeFileName(const wxString& s)
-{
-    wxString res = s;
-    res.Replace(wxT(":"), wxT("_"), TRUE);
-    res.Replace(wxT(" "), wxT("_"), TRUE);
-    res.Replace(wxT("/"), wxT("_"), TRUE);
-    res.Replace(wxT("\\"), wxT("_"), TRUE);
-    res.Replace(wxT("#"), wxT("_"), TRUE);
-    res.Replace(wxT("."), wxT("_"), TRUE);
-    return res;
-}
-

 
 static int LINKAGEMODE IndexCompareFunc(const void *a, const void *b)
 {
 
 static int LINKAGEMODE IndexCompareFunc(const void *a, const void *b)
 {


@@ -102,7 +93,7 @@ class HP_TagHandler : public wxHtmlTagHandler
         wxHtmlBookRecord *m_Book;
 
     public:
         wxHtmlBookRecord *m_Book;
 
     public:

-        HP_TagHandler(wxHtmlBookRecord *b) : wxHtmlTagHandler() {m_Book = b; m_Items = NULL; m_ItemsCnt = 0; m_Name = m_Page = wxEmptyString; m_Level = 0; }
+        HP_TagHandler(wxHtmlBookRecord *b) : wxHtmlTagHandler() {m_Book = b; m_Items = NULL; m_ItemsCnt = 0; m_Name = m_Page = wxEmptyString; m_Level = 0; m_ID = -1; }

         wxString GetSupportedTags() { return wxT("UL,OBJECT,PARAM"); }
         bool HandleTag(const wxHtmlTag& tag);
         void WriteOut(wxHtmlContentsItem*& array, int& size);
         wxString GetSupportedTags() { return wxT("UL,OBJECT,PARAM"); }
         bool HandleTag(const wxHtmlTag& tag);
         void WriteOut(wxHtmlContentsItem*& array, int& size);


@@ -231,7 +222,7 @@ bool wxHtmlHelpData::LoadMSProject(wxHtmlBookRecord *book, wxFileSystem& fsys, c
     HP_TagHandler *handler = new HP_TagHandler(book);
     parser.AddTagHandler(handler);
 
     HP_TagHandler *handler = new HP_TagHandler(book);
     parser.AddTagHandler(handler);
 

-    f = ( contentsfile.IsEmpty() ? 0 : fsys.OpenFile(contentsfile) );
+    f = ( contentsfile.IsEmpty() ? (wxFSFile*) NULL : fsys.OpenFile(contentsfile) );

     if (f) {
         sz = f -> GetStream() -> GetSize();
         buf = new char[sz + 1];
     if (f) {
         sz = f -> GetStream() -> GetSize();
         buf = new char[sz + 1];


@@ -243,8 +234,10 @@ bool wxHtmlHelpData::LoadMSProject(wxHtmlBookRecord *book, wxFileSystem& fsys, c
         handler -> WriteOut(m_Contents, m_ContentsCnt);
         delete[] buf;
     }
         handler -> WriteOut(m_Contents, m_ContentsCnt);
         delete[] buf;
     }

+    else
+        wxLogError(_("Cannot open contents file: %s"), contentsfile.mb_str());

 
 

-    f = ( indexfile.IsEmpty() ? 0 : fsys.OpenFile(indexfile) );
+    f = ( indexfile.IsEmpty() ? (wxFSFile*) NULL : fsys.OpenFile(indexfile) );

     if (f) {
         sz = f -> GetStream() -> GetSize();
         buf = new char[sz + 1];
     if (f) {
         sz = f -> GetStream() -> GetSize();
         buf = new char[sz + 1];


@@ -256,32 +249,66 @@ bool wxHtmlHelpData::LoadMSProject(wxHtmlBookRecord *book, wxFileSystem& fsys, c
         handler -> WriteOut(m_Index, m_IndexCnt);
         delete[] buf;
     }
         handler -> WriteOut(m_Index, m_IndexCnt);
         delete[] buf;
     }

+    else if (!indexfile.IsEmpty())
+        wxLogError(_("Cannot open index file: %s"), indexfile.mb_str());

     return TRUE;
 }
 
 
     return TRUE;
 }
 
 

+
+
+#if wxUSE_UNICODE
+
+#define READ_STRING(f, s, lng) { char tmpc; for (int i = 0; i < lng; i++) { f -> Read(&tmpc, 1); s[i] = (wxChar)tmpc;} }
+#define WRITE_STRING(f, s, lng) { char tmpc; for (int i = 0; i < lng; i++) { tmpc = (char)s[i]; f -> Write(&tmpc, 1);} }
+
+#else
+
+#define READ_STRING(f, s, lng) f -> Read(s, lng * sizeof(char));
+#define WRITE_STRING(f, s, lng) f -> Write(s, lng * sizeof(char));
+
+#endif
+
+
+#define CURRENT_CACHED_BOOK_VERSION     1
+

 bool wxHtmlHelpData::LoadCachedBook(wxHtmlBookRecord *book, wxInputStream *f)
 {
     int i, st;
 bool wxHtmlHelpData::LoadCachedBook(wxHtmlBookRecord *book, wxInputStream *f)
 {
     int i, st;

-    int x;
+    wxInt32 x;
+    wxInt32 version;

 
 

+    /* load header - version info : */
+
+    f -> Read(&x, sizeof(x));
+    version = wxINT32_SWAP_ON_BE(x);
+    
+    if (version != CURRENT_CACHED_BOOK_VERSION) 
+    {
+        wxLogError(_("Incorrect version of HTML help book"));
+        return FALSE;
+        // NOTE: when adding new version, please ensure backward compatibility!
+    }
+    

     /* load contents : */
 
     f -> Read(&x, sizeof(x));
     st = m_ContentsCnt;
     /* load contents : */
 
     f -> Read(&x, sizeof(x));
     st = m_ContentsCnt;

-    m_ContentsCnt += x;
-    m_Contents = (wxHtmlContentsItem*) realloc(m_Contents, (m_ContentsCnt / wxHTML_REALLOC_STEP + 1) * wxHTML_REALLOC_STEP * sizeof(wxHtmlContentsItem));
+    m_ContentsCnt += wxINT32_SWAP_ON_BE(x);
+    m_Contents = (wxHtmlContentsItem*) realloc(m_Contents, 
+                                               (m_ContentsCnt / wxHTML_REALLOC_STEP + 1) * 
+                                               wxHTML_REALLOC_STEP * sizeof(wxHtmlContentsItem));

     for (i = st; i < m_ContentsCnt; i++) {
         f -> Read(&x, sizeof(x));
     for (i = st; i < m_ContentsCnt; i++) {
         f -> Read(&x, sizeof(x));

-        m_Contents[i].m_Level = x;
-        f -> Read(&x, sizeof(x));
-        m_Contents[i].m_ID = x;
+        m_Contents[i].m_Level = wxINT32_SWAP_ON_BE(x);

         f -> Read(&x, sizeof(x));
         f -> Read(&x, sizeof(x));

+        m_Contents[i].m_ID = wxINT32_SWAP_ON_BE(x);
+        f -> Read(&x, sizeof(x)); x = wxINT32_SWAP_ON_BE(x);

         m_Contents[i].m_Name = new wxChar[x];
         m_Contents[i].m_Name = new wxChar[x];

-        f -> Read(m_Contents[i].m_Name, x*sizeof(wxChar));
-        f -> Read(&x, sizeof(x));
+        READ_STRING(f, m_Contents[i].m_Name, x);
+        f -> Read(&x, sizeof(x)); x = wxINT32_SWAP_ON_BE(x);

         m_Contents[i].m_Page = new wxChar[x];
         m_Contents[i].m_Page = new wxChar[x];

-        f -> Read(m_Contents[i].m_Page, x*sizeof(wxChar));
+        READ_STRING(f, m_Contents[i].m_Page, x);

         m_Contents[i].m_Book = book;
     }
 
         m_Contents[i].m_Book = book;
     }
 


@@ -289,15 +316,16 @@ bool wxHtmlHelpData::LoadCachedBook(wxHtmlBookRecord *book, wxInputStream *f)
 
     f -> Read(&x, sizeof(x));
     st = m_IndexCnt;
 
     f -> Read(&x, sizeof(x));
     st = m_IndexCnt;

-    m_IndexCnt += x;
-    m_Index = (wxHtmlContentsItem*) realloc(m_Index, (m_IndexCnt / wxHTML_REALLOC_STEP + 1) * wxHTML_REALLOC_STEP * sizeof(wxHtmlContentsItem));
+    m_IndexCnt += wxINT32_SWAP_ON_BE(x);
+    m_Index = (wxHtmlContentsItem*) realloc(m_Index, (m_IndexCnt / wxHTML_REALLOC_STEP + 1) * 
+                                                     wxHTML_REALLOC_STEP * sizeof(wxHtmlContentsItem));

     for (i = st; i < m_IndexCnt; i++) {
     for (i = st; i < m_IndexCnt; i++) {

-        f -> Read(&x, sizeof(x));
+        f -> Read(&x, sizeof(x)); x = wxINT32_SWAP_ON_BE(x);

         m_Index[i].m_Name = new wxChar[x];
         m_Index[i].m_Name = new wxChar[x];

-        f -> Read(m_Index[i].m_Name, x*sizeof(wxChar));
-        f -> Read(&x, sizeof(x));
+        READ_STRING(f, m_Index[i].m_Name, x);
+        f -> Read(&x, sizeof(x)); x = wxINT32_SWAP_ON_BE(x);

         m_Index[i].m_Page = new wxChar[x];
         m_Index[i].m_Page = new wxChar[x];

-        f -> Read(m_Index[i].m_Page, x*sizeof(wxChar));
+        READ_STRING(f, m_Index[i].m_Page, x);

         m_Index[i].m_Book = book;
     }
     return TRUE;
         m_Index[i].m_Book = book;
     }
     return TRUE;


@@ -307,40 +335,47 @@ bool wxHtmlHelpData::LoadCachedBook(wxHtmlBookRecord *book, wxInputStream *f)
 bool wxHtmlHelpData::SaveCachedBook(wxHtmlBookRecord *book, wxOutputStream *f)
 {
     int i;
 bool wxHtmlHelpData::SaveCachedBook(wxHtmlBookRecord *book, wxOutputStream *f)
 {
     int i;

-    int x;
+    wxInt32 x;
+
+    /* save header - version info : */
+
+    x = wxINT32_SWAP_ON_BE(CURRENT_CACHED_BOOK_VERSION);
+    f -> Write(&x, sizeof(x));

 
     /* save contents : */
 
     x = 0;
     for (i = 0; i < m_ContentsCnt; i++) if (m_Contents[i].m_Book == book && m_Contents[i].m_Level > 0) x++;
 
     /* save contents : */
 
     x = 0;
     for (i = 0; i < m_ContentsCnt; i++) if (m_Contents[i].m_Book == book && m_Contents[i].m_Level > 0) x++;

+    x = wxINT32_SWAP_ON_BE(x);

     f -> Write(&x, sizeof(x));
     for (i = 0; i < m_ContentsCnt; i++) {
         if (m_Contents[i].m_Book != book || m_Contents[i].m_Level == 0) continue;
     f -> Write(&x, sizeof(x));
     for (i = 0; i < m_ContentsCnt; i++) {
         if (m_Contents[i].m_Book != book || m_Contents[i].m_Level == 0) continue;

-        x = m_Contents[i].m_Level;
+        x = wxINT32_SWAP_ON_BE(m_Contents[i].m_Level);

         f -> Write(&x, sizeof(x));
         f -> Write(&x, sizeof(x));

-        x = m_Contents[i].m_ID;
+        x = wxINT32_SWAP_ON_BE(m_Contents[i].m_ID);

         f -> Write(&x, sizeof(x));
         f -> Write(&x, sizeof(x));

-        x = wxStrlen(m_Contents[i].m_Name) + 1;
+        x = wxINT32_SWAP_ON_BE(wxStrlen(m_Contents[i].m_Name) + 1);

         f -> Write(&x, sizeof(x));
         f -> Write(&x, sizeof(x));

-        f -> Write(m_Contents[i].m_Name, x*sizeof(wxChar));
-        x = wxStrlen(m_Contents[i].m_Page) + 1;
+        WRITE_STRING(f, m_Contents[i].m_Name, x);
+        x = wxINT32_SWAP_ON_BE(wxStrlen(m_Contents[i].m_Page) + 1);

         f -> Write(&x, sizeof(x));
         f -> Write(&x, sizeof(x));

-        f -> Write(m_Contents[i].m_Page, x*sizeof(wxChar));
+        WRITE_STRING(f, m_Contents[i].m_Page, x);

     }
 
     /* save index : */
 
     x = 0;
     for (i = 0; i < m_IndexCnt; i++) if (m_Index[i].m_Book == book && m_Index[i].m_Level > 0) x++;
     }
 
     /* save index : */
 
     x = 0;
     for (i = 0; i < m_IndexCnt; i++) if (m_Index[i].m_Book == book && m_Index[i].m_Level > 0) x++;

+    x = wxINT32_SWAP_ON_BE(x);

     f -> Write(&x, sizeof(x));
     for (i = 0; i < m_IndexCnt; i++) {
         if (m_Index[i].m_Book != book || m_Index[i].m_Level == 0) continue;
     f -> Write(&x, sizeof(x));
     for (i = 0; i < m_IndexCnt; i++) {
         if (m_Index[i].m_Book != book || m_Index[i].m_Level == 0) continue;

-        x = wxStrlen(m_Index[i].m_Name) + 1;
+        x = wxINT32_SWAP_ON_BE(wxStrlen(m_Index[i].m_Name) + 1);

         f -> Write(&x, sizeof(x));
         f -> Write(&x, sizeof(x));

-        f -> Write(m_Index[i].m_Name, x*sizeof(wxChar));
-        x = wxStrlen(m_Index[i].m_Page) + 1;
+        WRITE_STRING(f, m_Index[i].m_Name, x);
+        x = wxINT32_SWAP_ON_BE(wxStrlen(m_Index[i].m_Page) + 1);

         f -> Write(&x, sizeof(x));
         f -> Write(&x, sizeof(x));

-        f -> Write(m_Index[i].m_Page, x*sizeof(wxChar));
+        WRITE_STRING(f, m_Index[i].m_Page, x);

     }
     return TRUE;
 }
     }
     return TRUE;
 }


@@ -351,22 +386,37 @@ void wxHtmlHelpData::SetTempDir(const wxString& path)
     if (path == wxEmptyString) m_TempPath = path;
     else {
         if (wxIsAbsolutePath(path)) m_TempPath = path;
     if (path == wxEmptyString) m_TempPath = path;
     else {
         if (wxIsAbsolutePath(path)) m_TempPath = path;

-        else m_TempPath = wxGetCwd() + "/" + path;
+        else m_TempPath = wxGetCwd() + _T("/") + path;

 
 

-        if (m_TempPath[m_TempPath.Length() - 1] != '/')
-            m_TempPath << "/";
+        if (m_TempPath[m_TempPath.Length() - 1] != _T('/'))
+            m_TempPath << _T('/');

     }
 }
 
 
     }
 }
 
 

-bool wxHtmlHelpData::AddBookParam(const wxString& title, const wxString& contfile,
+
+static wxString SafeFileName(const wxString& s)
+{
+    wxString res(s);
+    res.Replace(wxT("#"), wxT("_"));
+    res.Replace(wxT(":"), wxT("_"));
+    res.Replace(wxT("\\"), wxT("_"));
+    res.Replace(wxT("/"), wxT("_"));
+    return res;
+}
+
+bool wxHtmlHelpData::AddBookParam(const wxFSFile& bookfile,
+                                  wxFontEncoding encoding,
+                                  const wxString& title, const wxString& contfile,

                                   const wxString& indexfile, const wxString& deftopic,
                                   const wxString& path)
 {
     wxFileSystem fsys;
     wxFSFile *fi;
     wxHtmlBookRecord *bookr;
                                   const wxString& indexfile, const wxString& deftopic,
                                   const wxString& path)
 {
     wxFileSystem fsys;
     wxFSFile *fi;
     wxHtmlBookRecord *bookr;

-    wxString safetitle;
+    
+    int IndexOld = m_IndexCnt,
+        ContentsOld = m_ContentsCnt;

 
     if (! path.IsEmpty())
         fsys.ChangePathTo(path, TRUE);
 
     if (! path.IsEmpty())
         fsys.ChangePathTo(path, TRUE);


@@ -387,23 +437,54 @@ bool wxHtmlHelpData::AddBookParam(const wxString& title, const wxString& contfil
     int cont_start = m_ContentsCnt++;
 
     // Try to find cached binary versions:
     int cont_start = m_ContentsCnt++;
 
     // Try to find cached binary versions:

-    safetitle = SafeFileName(title);
-    fi = fsys.OpenFile(safetitle + wxT(".cached"));
-    if (fi == NULL) fi = fsys.OpenFile(m_TempPath + safetitle + wxT(".cached"));
-    if ((fi == NULL) || (m_TempPath == wxEmptyString)) {
-        LoadMSProject(bookr, fsys, indexfile, contfile);
-        if (m_TempPath != wxEmptyString) {
-            wxFileOutputStream *outs = new wxFileOutputStream(m_TempPath + safetitle + wxT(".cached"));
-            SaveCachedBook(bookr, outs);
-            delete outs;
+    // 1. save file as book, but with .hhp.cached extension
+    // 2. same as 1. but in temp path
+    // 3. otherwise or if cache load failed, load it from MS.
+    
+    fi = fsys.OpenFile(bookfile.GetLocation() + wxT(".cached"));
+    
+    if (fi == NULL || 
+          fi -> GetModificationTime() < bookfile.GetModificationTime() || 
+          !LoadCachedBook(bookr, fi -> GetStream()))
+    {
+        if (fi != NULL) delete fi;
+        fi = fsys.OpenFile(m_TempPath + wxFileNameFromPath(bookfile.GetLocation()) + wxT(".cached"));
+        if (m_TempPath == wxEmptyString || fi == NULL || 
+            fi -> GetModificationTime() < bookfile.GetModificationTime() || 
+            !LoadCachedBook(bookr, fi -> GetStream()))
+        {
+            LoadMSProject(bookr, fsys, indexfile, contfile);
+            if (m_TempPath != wxEmptyString) 
+            {
+                wxFileOutputStream *outs = new wxFileOutputStream(m_TempPath + 
+                                                  SafeFileName(wxFileNameFromPath(bookfile.GetLocation())) + wxT(".cached"));
+                SaveCachedBook(bookr, outs);
+                delete outs;
+            }

         }
         }

-    } else {
-        LoadCachedBook(bookr, fi -> GetStream());
-        delete fi;

     }
     }

+    
+    if (fi != NULL) delete fi;

 
     // Now store the contents range
     bookr->SetContentsRange(cont_start, m_ContentsCnt);
 
     // Now store the contents range
     bookr->SetContentsRange(cont_start, m_ContentsCnt);

+    
+    // Convert encoding, if neccessary:
+    if (encoding != wxFONTENCODING_SYSTEM)
+    {
+        wxFontEncodingArray a = wxEncodingConverter::GetPlatformEquivalents(encoding);
+        if (a.GetCount() != 0 && a[0] != encoding)
+        {
+            int i;
+            wxEncodingConverter conv;
+            conv.Init(encoding, a[0]);
+            
+            for (i = IndexOld; i < m_IndexCnt; i++)
+                conv.Convert(m_Index[i].m_Name);
+            for (i = ContentsOld; i < m_ContentsCnt; i++)
+                conv.Convert(m_Contents[i].m_Name);
+        }
+    }

 
     m_BookRecords.Add(bookr);
     if (m_IndexCnt > 0)
 
     m_BookRecords.Add(bookr);
     if (m_IndexCnt > 0)


@@ -446,15 +527,21 @@ bool wxHtmlHelpData::AddBook(const wxString& book)
         char linebuf[300];
 
         wxString title = _("noname"),
         char linebuf[300];
 
         wxString title = _("noname"),

-                         safetitle,
-                         start = wxEmptyString,
-                                 contents = wxEmptyString, index = wxEmptyString;
+                 safetitle,
+                 start = wxEmptyString,
+                 contents = wxEmptyString, 
+                 index = wxEmptyString,
+                 charset = wxEmptyString;

 
         if (wxIsAbsolutePath(book)) bookFull = book;
         else bookFull = wxGetCwd() + "/" + book;
 
         fi = fsys.OpenFile(bookFull);
 
         if (wxIsAbsolutePath(book)) bookFull = book;
         else bookFull = wxGetCwd() + "/" + book;
 
         fi = fsys.OpenFile(bookFull);

-        if (fi == NULL) return FALSE;
+        if (fi == NULL) 
+       {
+           wxLogError(_("Cannot open HTML help book: %s"), bookFull.mb_str());
+           return FALSE;
+       }

         fsys.ChangePathTo(bookFull);
         s = fi -> GetStream();
         sz = s -> GetSize();
         fsys.ChangePathTo(bookFull);
         s = fi -> GetStream();
         sz = s -> GetSize();


@@ -462,7 +549,6 @@ bool wxHtmlHelpData::AddBook(const wxString& book)
         buff[sz] = 0;
         s -> Read(buff, sz);
         lineptr = buff;
         buff[sz] = 0;
         s -> Read(buff, sz);
         lineptr = buff;

-        delete fi;

 
         do {
             lineptr = ReadLine(lineptr, linebuf);
 
         do {
             lineptr = ReadLine(lineptr, linebuf);


@@ -475,10 +561,18 @@ bool wxHtmlHelpData::AddBook(const wxString& book)
                 index = linebuf + strlen("Index file=");
             if (strstr(linebuf, "Contents file=") == linebuf)
                 contents = linebuf + strlen("Contents file=");
                 index = linebuf + strlen("Index file=");
             if (strstr(linebuf, "Contents file=") == linebuf)
                 contents = linebuf + strlen("Contents file=");

+            if (strstr(linebuf, "Charset=") == linebuf)
+                charset = linebuf + strlen("Charset=");

         } while (lineptr != NULL);
         delete[] buff;
         } while (lineptr != NULL);
         delete[] buff;

-
-        return AddBookParam(title, contents, index, start, fsys.GetPath());
+    
+        wxFontEncoding enc;
+        if (charset == wxEmptyString) enc = wxFONTENCODING_SYSTEM;
+        else enc = wxTheFontMapper -> CharsetToEncoding(charset);
+        bool rtval = AddBookParam(*fi, enc, 
+                                  title, contents, index, start, fsys.GetPath());
+        delete fi;
+        return rtval;

     }
 }
 
     }
 }
 


@@ -587,19 +681,14 @@ wxHtmlSearchStatus::wxHtmlSearchStatus(wxHtmlHelpData* data, const wxString& key
 
 bool wxHtmlSearchStatus::Search()
 {
 
 bool wxHtmlSearchStatus::Search()
 {

-    wxFileSystem fsys;

     wxFSFile *file;
     int i = m_CurIndex;  // shortcut
     bool found = FALSE;
     wxChar *thepage;
 
     if (!m_Active) {
     wxFSFile *file;
     int i = m_CurIndex;  // shortcut
     bool found = FALSE;
     wxChar *thepage;
 
     if (!m_Active) {

-    // sanity check. Illegal use, but we'll try to prevent a crash anyway
-#if !defined(__VISAGECPP__)
-        wxASSERT(0);
-#else
+        // sanity check. Illegal use, but we'll try to prevent a crash anyway

         wxASSERT(m_Active);
         wxASSERT(m_Active);

-#endif

         return FALSE;
     }
 
         return FALSE;
     }
 


@@ -622,6 +711,7 @@ bool wxHtmlSearchStatus::Search()
     }
     else m_LastPage = thepage;
     
     }
     else m_LastPage = thepage;
     

+    wxFileSystem fsys;

     file = fsys.OpenFile(m_Data->m_Contents[i].m_Book -> GetBasePath() + thepage);
     if (file) 
     {
     file = fsys.OpenFile(m_Data->m_Contents[i].m_Book -> GetBasePath() + thepage);
     if (file) 
     {