commit: fix compile errors, fix revert-buffer bug on binary/Latin 1 files, Mule-ize some files

Ben Wing ben at 666.com
Mon Feb 8 07:09:38 EST 2010


On Sun, Feb 7, 2010 at 9:46 PM, Vin Shelton <ethersoft at gmail.com> wrote:
> Ben -
>
> Thanks for fixing up the earlier build problems.
>
> Part of this patch isn't correct, tho -
>
> On Sun, Feb 7, 2010 at 8:12 AM, Ben Wing <ben at xemacs.org> wrote:
>> changeset:   5000:44d7bde26046
>> parent:      4994:76af7fc13e81
>> user:        Ben Wing <ben at xemacs.org>
>> date:        Sun Feb 07 05:21:40 2010 -0600
>> files:       src/ChangeLog src/emacs.c src/fileio.c src/fns.c src/lisp.h src/nt.c
>> description:
>> fix compile errors, fix revert-buffer bug on binary/Latin 1 files, Mule-ize some files
> ...
>>   nr.dwScope = RESOURCE_GLOBALNET;
>>   nr.dwType = RESOURCETYPE_DISK;
>>   nr.dwDisplayType = RESOURCEDISPLAYTYPE_SERVER;
>>   nr.dwUsage = RESOURCEUSAGE_CONTAINER;
>>   nr.lpLocalName = NULL;
>> -  PATHNAME_CONVERT_OUT (path, nr.lpRemoteName);
>> +  PATHNAME_CONVERT_OUT (path, extpath);
>> +  nr.lpRemoteName = (LPTSTR) extpath;
>
>
> Yields the following:
>
> nt.c
> C:\cygwin\usr\local\src\xemacs-21.5-test-2010-02-07\src\nt.c(966) :
> error C2440: '=' : cannot convert from 'char *' to 'unsigned short *'
>        Types pointed to are unrelated; conversion requires
> reinterpret_cast, C-style cast or function-style cast
>
> Changing the cast from LPTSTR to unsigned short * shuts up the
> compiler, but I don't know if you want a different type there.  Please
> fix this.
>
> Thank you,
>  Vin

It should be XELPTSTR and I've committed it this way.  Hopefully it'll work now.

Sorry about these problems.  I don't have Visual C++ on my current
machine so I'm relegated to using Cygwin, which doesn't catch quite
the same things as VC++.

ben



More information about the XEmacs-Patches mailing list