[Pharo 7.0-dev] Build #593: 21390-Do-not-store-any-non-literals-in-the-literal-array

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

[Pharo 7.0-dev] Build #593: 21390-Do-not-store-any-non-literals-in-the-literal-array

ci-pharo-ci-jenkins2
There is a new Pharo build available!
       
The status of the build #593 was: SUCCESS.

The Pull Request #932 was integrated: "21390-Do-not-store-any-non-literals-in-the-literal-array"
Pull request url: https://github.com/pharo-project/pharo/pull/932

Issue Url: https://pharo.fogbugz.com/f/cases/21390
Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/development/593/
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo 7.0-dev] Build #593: 21390-Do-not-store-any-non-literals-in-the-literal-array

Marcus Denker-4
This fixes the Reflectivity compiler to not store normal objects in the literalArray… so instead of “pushLiteral:” is now uses
“pushLiteralVariable:”.

This is needed as we do now have a compiler option to compile methods with read-only literals.

        Marcus

> On 22 Feb 2018, at 11:45, [hidden email] wrote:
>
> There is a new Pharo build available!
>
> The status of the build #593 was: SUCCESS.
>
> The Pull Request #932 was integrated: "21390-Do-not-store-any-non-literals-in-the-literal-array"
> Pull request url: https://github.com/pharo-project/pharo/pull/932
>
> Issue Url: https://pharo.fogbugz.com/f/cases/21390
> Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/development/593/


Reply | Threaded
Open this post in threaded view
|

Re: [Pharo 7.0-dev] Build #593: 21390-Do-not-store-any-non-literals-in-the-literal-array

Stephane Ducasse-3
I really like this idea of read only literals :)
Tx for pushing it.

On Thu, Feb 22, 2018 at 11:52 AM, Marcus Denker <[hidden email]> wrote:

> This fixes the Reflectivity compiler to not store normal objects in the literalArray… so instead of “pushLiteral:” is now uses
> “pushLiteralVariable:”.
>
> This is needed as we do now have a compiler option to compile methods with read-only literals.
>
>         Marcus
>
>> On 22 Feb 2018, at 11:45, [hidden email] wrote:
>>
>> There is a new Pharo build available!
>>
>> The status of the build #593 was: SUCCESS.
>>
>> The Pull Request #932 was integrated: "21390-Do-not-store-any-non-literals-in-the-literal-array"
>> Pull request url: https://github.com/pharo-project/pharo/pull/932
>>
>> Issue Url: https://pharo.fogbugz.com/f/cases/21390
>> Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/development/593/
>
>

Reply | Threaded
Open this post in threaded view
|

Re: [Pharo 7.0-dev] Build #593: 21390-Do-not-store-any-non-literals-in-the-literal-array

NorbertHartl


> Am 25.02.2018 um 22:02 schrieb Stephane Ducasse <[hidden email]>:
>
> I really like this idea of read only literals :)
> Tx for pushing it.

+1

>
>> On Thu, Feb 22, 2018 at 11:52 AM, Marcus Denker <[hidden email]> wrote:
>> This fixes the Reflectivity compiler to not store normal objects in the literalArray… so instead of “pushLiteral:” is now uses
>> “pushLiteralVariable:”.
>>
>> This is needed as we do now have a compiler option to compile methods with read-only literals.
>>
>>        Marcus
>>
>>> On 22 Feb 2018, at 11:45, [hidden email] wrote:
>>>
>>> There is a new Pharo build available!
>>>
>>> The status of the build #593 was: SUCCESS.
>>>
>>> The Pull Request #932 was integrated: "21390-Do-not-store-any-non-literals-in-the-literal-array"
>>> Pull request url: https://github.com/pharo-project/pharo/pull/932
>>>
>>> Issue Url: https://pharo.fogbugz.com/f/cases/21390
>>> Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/development/593/
>>
>>