[Patch] CLongLong>>#value: can fail on LargeNegativeInteger

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

[Patch] CLongLong>>#value: can fail on LargeNegativeInteger

Thomas Girard
Hello,

please find attached a patch for:
  http://smalltalk.gnu.org/project/issue/647

I will also make this available as a pull request on github, along with
another cosmetic change.

Thanks,
Regards,

Thomas

_______________________________________________
help-smalltalk mailing list
[hidden email]
https://lists.gnu.org/mailman/listinfo/help-smalltalk

issue647.patch (1K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [Patch] CLongLong>>#value: can fail on LargeNegativeInteger

Thomas Girard
On 14/06/2012 22:27, Thomas Girard wrote:
> I will also make this available as a pull request on github, along with
> another cosmetic change.

That's:
  https://github.com/bonzini/smalltalk/pull/5

Regards,

Thomas

_______________________________________________
help-smalltalk mailing list
[hidden email]
https://lists.gnu.org/mailman/listinfo/help-smalltalk
Reply | Threaded
Open this post in threaded view
|

Re: [Patch] CLongLong>>#value: can fail on LargeNegativeInteger

Thomas Girard
(Cancel and replace previous emails on issue #647)

On 14/06/2012 23:28, Thomas Girard wrote:
>> I will also make this available as a pull request on github, along with
>> another cosmetic change.
>
> That's:
>   https://github.com/bonzini/smalltalk/pull/5

The issue also existed on CULong. Reworked patch attached.

Regards,

Thomas

_______________________________________________
help-smalltalk mailing list
[hidden email]
https://lists.gnu.org/mailman/listinfo/help-smalltalk

issue647.patch (2K) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [Patch] CLongLong>>#value: can fail on LargeNegativeInteger

Paolo Bonzini-2
Il 15/06/2012 00:04, Thomas Girard ha scritto:

> (Cancel and replace previous emails on issue #647)
>
> On 14/06/2012 23:28, Thomas Girard wrote:
>>> I will also make this available as a pull request on github, along with
>>> another cosmetic change.
>>
>> That's:
>>   https://github.com/bonzini/smalltalk/pull/5
>
> The issue also existed on CULong. Reworked patch attached.

Thanks, all merged.

Paolo




_______________________________________________
help-smalltalk mailing list
[hidden email]
https://lists.gnu.org/mailman/listinfo/help-smalltalk