[Mono-dev] [PATCH] Use Encoding.Default as the default code page formcs

Kornél Pál kornelpal at hotmail.com
Mon Oct 24 05:40:34 EDT 2005


Hi,

I posted this patch two weeks ago but I got no comments. Could you review
please?

Kornél

----- Original Message -----
From: "Kornél Pál" <kornelpal at hotmail.com>
To: <mono-devel-list at lists.ximian.com>
Sent: Monday, October 10, 2005 11:47 AM
Subject: [Mono-dev] [PATCH] Use Encoding.Default as the default code page
formcs


> Hi,
>
> csc.exe uses Encoding.Default as the default code page that is system
> dependent unlike our current hard coded default code pages.
> Encoding.Default
> takes care (or at least should take care:) about everything so we don't
> have
> to use try-catch or store it in a variable like default_encoding. This
> patch
> simply changes default code page from 28591 to Encoding.Default.
>
> In addition to that this is the expected behaviour as csc.exe does the
> same
> this is what developers require as they edit source files on their system
> using their system default code page so they want to compile it using the
> same code page to get the expected result when compiling code.
>
> Note that previously we followed a wrong practicle by defining the code
> page
> we use in mcs tree as the default code page of our mcs copiler rather than
> passig this code page to mcs as command line argument. This is now fixed
> so
> we should use Encoding.Default as the default code page of mcs.
>
> Please review and approve the patch.
>
> Kornél
>


--------------------------------------------------------------------------------


> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: MCSDefaultCodePage.diff
Type: application/octet-stream
Size: 1083 bytes
Desc: not available
Url : http://lists.ximian.com/pipermail/mono-devel-list/attachments/20051024/30a83278/attachment.obj 


More information about the Mono-devel-list mailing list