Use realpath on Linux when saving in an editor. This makes the save w…

…ork for relative symlinks and symlinks-to-symlinks too.

This should fully fix #1576.

Also, move a comment line back to the code it refers to.

LiteEditor/cl_editor.cpp

@@ -1487,10 +1487,10 @@ bool LEditor::SaveToFile(const wxFileName& fileName)
     file.Write(buf.data(), strlen(buf.data()));
     file.Close();
 
-    // keep the original file permissions
     wxFileName symlinkedFile = fileName;
     if(wxIsFileSymlink(fileName)) { symlinkedFile = wxReadLink(fileName); }
 
+    // keep the original file permissions
     mode_t origPermissions = 0;
     if(!FileUtils::GetFilePermissions(symlinkedFile, origPermissions)) {
         clWARNING() << "Failed to read file permissions." << fileName << clEndl;

Plugin/globals.cpp

@@ -1138,6 +1138,11 @@ wxFileName wxReadLink(const wxFileName& filename)
 {
 #ifndef __WXMSW__
     if(wxIsFileSymlink(filename)) {
+    #if defined(__WXGTK__)
+        // Use 'realpath' on Linux, otherwise this breaks on relative symlinks, and (untested) on symlinks-to-symlinks
+        return wxFileName(CLRealPath(filename.GetFullPath()));
+
+    #else // OSX
         wxFileName realFileName;
         char _tmp[512];
         memset(_tmp, 0, sizeof(_tmp));
@@ -1146,6 +1151,7 @@ wxFileName wxReadLink(const wxFileName& filename)
             realFileName = wxFileName(wxString(_tmp, wxConvUTF8, len));
             return realFileName;
         }
+    #endif // !OSX
     }
     return filename;