Lars Gullik Bjønnes wrote:

> Angus Leeming <[EMAIL PROTECTED]> writes:
> 
> | We do in LyX 1.3.x which is what bennett is trying to get working. We
> | don't support the (2.95 + 2.95 stdlib) combo in LyX 1.4.x.
> 
> I think I saw some patches for 1.4.x as well...

You haven't seen any formal patches. Not in this thread anyway. 
Nonetheless, you're right to point out that the fix is not needed for LyX 
1.4.x. Thanks for that.

> | So, I guess that defines the way forward. I'll commit my original work
> | around (#if defined (WIN32)) to the 1.3.x branch,
> 
> Without having "ios::in | ios::binary" checked first?

Shrug. Since ios::binary has no effect at all on a unix machine, why do we 
care? Changes to LyX 1.3.x are all about making it work. Hacks are 
perfectly acceptable, no?

Anyway, Bennett, if you'd be good enough to test out 
        ifstream ifs(from.c_str(), ios::in | ios::binary);
then I'll apply that instead if you find that it works.

-- 
Angus

Reply via email to