[Mono-dev] mcs patch to say goodbye to SeekableStreamReader

[Atsushi Eno]

Hi,

> I tried your patch with ComIStreamMarshaler.cs and used -codepage:65001. It
> worked as expected but it fails with SVN head. The expected behaviour is to
> compile without errors so you patch solves the bug.

Yep - now the fixed patch is going.

> I've done some tests with files readed as UTF-8 but containing invalid byte
> sequences even with BOM and it worked as expected.

Cool. Thanks for checking it :-)

> I think the patch should be ported to gmcs as well.

I'll ask Martin later.

> I think we should return using Encoding.Default as the default encoding in
> mcs but should comfigure mcs using -codepage:28591 -codepage:1252 in
> mcs/build to use that encoding.

Maybe after 1.1.9 as Miguel suggested? I don't think the default
encoding problem would happen only to mcs.

Atsushi Eno