[Mono-dev] Proposed changes to mono/mcs/class/corlib/Mono/DataConverter.cs

2014-12-08 Thread Michael McGlothlin
Couldn't figure out how to get Xamarin Studio to create an actual patch
file after trying, and failing, to push a change with git. It appears this
is the correct place to post this?

I usually put most of my code in PCL and then call it from platform
specific code so I'm trying to make DataConverter happy as PCL.

To make DataConverter compile as PCL I had to change Encoding.UTF7 to
Encoding.GetEncoding ( "utf-7" ) and the numeric calls to 12000 and 12001
on GetEncoding() to the string-based "utf-32" and "uff-32BE". These three
changes had to be made twice as there are two variants using this same
code. I also changed ArrayList to List to make PCL happy.



case '7':
e = Encoding.GetEncoding ("utf-7");
n = 1;
break;
case '3':
e = Encoding.GetEncoding ("utf-32");
n = 4;
break;
case '4':
e = Encoding.GetEncoding ("utf-32BE");
n = 4;
break;



static public IList Unpack (string description, byte[] buffer, int
startIndex)
{
DataConverter conv = CopyConv;
var result = new List ();
int idx = startIndex;
bool align = false;
int repeat = 0, n;


DataConverter.cs
Description: Binary data
___
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list


Re: [Mono-dev] Proposed changes to mono/mcs/class/corlib/Mono/DataConverter.cs

2014-12-08 Thread Stephen Shaw
I can't comment on the code changes themselves, but it would be helpful to
fork mono on github and then create a Pull Request (PR) with your changes.

Cheers,
Stephen

On Mon, Dec 8, 2014 at 8:54 PM, Michael McGlothlin <
mike.mcgloth...@gmail.com> wrote:

> Couldn't figure out how to get Xamarin Studio to create an actual patch
> file after trying, and failing, to push a change with git. It appears this
> is the correct place to post this?
>
> I usually put most of my code in PCL and then call it from platform
> specific code so I'm trying to make DataConverter happy as PCL.
>
> To make DataConverter compile as PCL I had to change Encoding.UTF7 to
> Encoding.GetEncoding ( "utf-7" ) and the numeric calls to 12000 and 12001
> on GetEncoding() to the string-based "utf-32" and "uff-32BE". These three
> changes had to be made twice as there are two variants using this same
> code. I also changed ArrayList to List to make PCL happy.
>
>
>
> case '7':
> e = Encoding.GetEncoding ("utf-7");
> n = 1;
> break;
> case '3':
> e = Encoding.GetEncoding ("utf-32");
> n = 4;
> break;
> case '4':
> e = Encoding.GetEncoding ("utf-32BE");
> n = 4;
> break;
>
>
>
> static public IList Unpack (string description, byte[] buffer, int
> startIndex)
> {
> DataConverter conv = CopyConv;
> var result = new List ();
> int idx = startIndex;
> bool align = false;
> int repeat = 0, n;
>
>
> ___
> Mono-devel-list mailing list
> Mono-devel-list@lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>
>
___
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list


Re: [Mono-dev] Proposed changes to mono/mcs/class/corlib/Mono/DataConverter.cs

2014-12-09 Thread Miguel de Icaza
Hey,

I have a variation of this patch now.   I need to rebuild my Mono before i
can commit it.

It will use a conditional compiler directive to use this other system

On Mon, Dec 8, 2014 at 11:53 PM, Stephen Shaw  wrote:

> I can't comment on the code changes themselves, but it would be helpful to
> fork mono on github and then create a Pull Request (PR) with your changes.
>
> Cheers,
> Stephen
>
> On Mon, Dec 8, 2014 at 8:54 PM, Michael McGlothlin <
> mike.mcgloth...@gmail.com> wrote:
>
>> Couldn't figure out how to get Xamarin Studio to create an actual patch
>> file after trying, and failing, to push a change with git. It appears this
>> is the correct place to post this?
>>
>> I usually put most of my code in PCL and then call it from platform
>> specific code so I'm trying to make DataConverter happy as PCL.
>>
>> To make DataConverter compile as PCL I had to change Encoding.UTF7 to
>> Encoding.GetEncoding ( "utf-7" ) and the numeric calls to 12000 and 12001
>> on GetEncoding() to the string-based "utf-32" and "uff-32BE". These three
>> changes had to be made twice as there are two variants using this same
>> code. I also changed ArrayList to List to make PCL happy.
>>
>>
>>
>> case '7':
>> e = Encoding.GetEncoding ("utf-7");
>> n = 1;
>> break;
>> case '3':
>> e = Encoding.GetEncoding ("utf-32");
>> n = 4;
>> break;
>> case '4':
>> e = Encoding.GetEncoding ("utf-32BE");
>> n = 4;
>> break;
>>
>>
>>
>> static public IList Unpack (string description, byte[] buffer, int
>> startIndex)
>> {
>> DataConverter conv = CopyConv;
>> var result = new List ();
>> int idx = startIndex;
>> bool align = false;
>> int repeat = 0, n;
>>
>>
>> ___
>> Mono-devel-list mailing list
>> Mono-devel-list@lists.ximian.com
>> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>>
>>
>
> ___
> Mono-devel-list mailing list
> Mono-devel-list@lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>
>
___
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list


Re: [Mono-dev] Proposed changes to mono/mcs/class/corlib/Mono/DataConverter.cs

2014-12-17 Thread Michael McGlothlin
How would I access your new version (In a different branch or such?)? How long 
until the changes will filter through to the NuGet package!

Thanks,
Sent from my iPad

> On Dec 9, 2014, at 8:25 AM, Miguel de Icaza  wrote:
> 
> Hey,
> 
> I have a variation of this patch now.   I need to rebuild my Mono before i 
> can commit it.
> 
> It will use a conditional compiler directive to use this other system
> 
>> On Mon, Dec 8, 2014 at 11:53 PM, Stephen Shaw  wrote:
>> I can't comment on the code changes themselves, but it would be helpful to 
>> fork mono on github and then create a Pull Request (PR) with your changes.
>> 
>> Cheers,
>> Stephen
>> 
>>> On Mon, Dec 8, 2014 at 8:54 PM, Michael McGlothlin 
>>>  wrote:
>>> Couldn't figure out how to get Xamarin Studio to create an actual patch 
>>> file after trying, and failing, to push a change with git. It appears this 
>>> is the correct place to post this?
>>> 
>>> I usually put most of my code in PCL and then call it from platform 
>>> specific code so I'm trying to make DataConverter happy as PCL.
>>> 
>>> To make DataConverter compile as PCL I had to change Encoding.UTF7 to 
>>> Encoding.GetEncoding ( "utf-7" ) and the numeric calls to 12000 and 12001 
>>> on GetEncoding() to the string-based "utf-32" and "uff-32BE". These three 
>>> changes had to be made twice as there are two variants using this same 
>>> code. I also changed ArrayList to List to make PCL happy.
>>> 
>>> 
>>> 
>>> case '7':
>>> e = Encoding.GetEncoding ("utf-7");
>>> n = 1;
>>> break;
>>> case '3':
>>> e = Encoding.GetEncoding ("utf-32");
>>> n = 4;
>>> break;
>>> case '4':
>>> e = Encoding.GetEncoding ("utf-32BE");
>>> n = 4;
>>> break;
>>> 
>>> 
>>> 
>>> static public IList Unpack (string description, byte[] buffer, 
>>> int startIndex)
>>> {
>>> DataConverter conv = CopyConv;
>>> var result = new List ();
>>> int idx = startIndex;
>>> bool align = false;
>>> int repeat = 0, n;
>>> 
>>> 
>>> ___
>>> Mono-devel-list mailing list
>>> Mono-devel-list@lists.ximian.com
>>> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>>> 
>> 
>> 
>> ___
>> Mono-devel-list mailing list
>> Mono-devel-list@lists.ximian.com
>> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>> 
> 
___
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list