------- Comment #23 From joe 2013-04-08 08:50:53 [reply] -------
Thanks Hub,
I tested on latest version of Trunk. It is working now.
Mark it as fixed, please reopen if required. thanks
Joe
(In reply to comment #22)
> patch committed
>
> in trunk@32843
>
On Mon, Apr 8, 2013 at 10:44 AM, joe joe <joegsoc2013@gmail.com> wrote:
> Hi Hub,
>
> I saw your reply on the bug: http://bugzilla.abisource.com/show_bug.cgi?id=8487
>
>
> Are there more changes only to be done before commit? thanks
> ------- Comment #17 From Hubert Figuière 2013-04-08 02:37:25 [reply] -------
> (From update of attachment 5654 [details])
> That's better
>
> Joe
>
>
> On Sun, Apr 7, 2013 at 9:06 PM, joe joe <joegsoc2013@gmail.com> wrote:
>> http://bugzilla.abisource.com/show_bug.cgi?id=8487
>> ------- Comment #14 From joe 2013-04-07 14:48:48 [reply] -------
>> (In reply to comment #13)
>>> The idea looks good, but if you look closely to ut_go_file.cpp, ideally the
>>> case #elif ! defined (G_OS_WIN32) should just be a #else and file_stat should
>>> just be a GStatBuf as per the documentation. and it would be equivalent.
>>>
>>
>> Hi Hub,
>>
>> If we would like to merge the code together, should like the Patch in
>> the attachment.
>> Because the following code can't merge together since:
>> 1. the Macro definition of "S_IRUSR" && "S_IREAD" are not same in different OS.
>> 2. the file attributions are not the same. in windows, there are not Group
>> Permissions & Other Permissions
>>
>>
>> Joe
>>
>>
>>
>> On Sat, Apr 6, 2013 at 11:43 PM, joe joe <joegsoc2013@gmail.com> wrote:
>>> Hi,
>>>
>>> I have fixed bug #8487: read-only status not shown in title bar
>>> http://bugzilla.abisource.com/show_bug.cgi?id=8487
>>>
>>> Patch and fix screenshot are attached on that bug. Can someone help me
>>> to have a review? thanks
>>>
>>> Investigation:
>>> Only Windows don't show read-only status in title bar. the reason
>>> is that in Window, there is not function to read user-permission.
>>>
>>> Solution:
>>> I add some code to read user-permission and fix it.
>>>
>>> Index: af/util/xp/ut_go_file.cpp
>>> ===================================================================
>>> --- af/util/xp/ut_go_file.cpp (revision 32831)
>>> +++ af/util/xp/ut_go_file.cpp (working copy)
>>> @@ -1478,6 +1478,19 @@
>>> file_permissions->others_write = ((file_stat.st_mode & S_IWOTH) != 0);
>>> file_permissions->others_execute = ((file_stat.st_mode & S_IXOTH) != 0);
>>> }
>>> +#elif defined (G_OS_WIN32)
>>> + //fix bug #8487: read-only status not shown in title bar
>>> + GStatBuf file_stat;
>>> + char *filename = UT_go_filename_from_uri (uri);
>>> + int result = filename ? g_stat (filename, &file_stat) : -1;
>>> + g_free (filename);
>>> + if (result == 0) {
>>> + file_permissions = g_new0 (UT_GOFilePermissions, 1);
>>> + /* Owner Permissions */
>>> + file_permissions->owner_read = ((file_stat.st_mode & S_IREAD) != 0);
>>> + file_permissions->owner_write = ((file_stat.st_mode & S_IWRITE) != 0);
>>> + file_permissions->owner_execute = ((file_stat.st_mode & S_IEXEC) != 0);
>>> + }
>>> #endif
>>> return file_permissions;
>>> }
>>>
>>>
>>>
>>>
>>> Joe
Received on Mon Apr 8 08:56:43 2013
This archive was generated by hypermail 2.1.8 : Mon Apr 08 2013 - 08:56:43 CEST