[update 1.1] #11241

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

[update 1.1] #11241

Stéphane Ducasse
11241
-----

- Issue 2102: Fix the fact that local temp can shadow silently instance var
- cleaning findChgSet
- fixing packageInfo typos overriden -> overridden
- Fix file = and added one test

Stef

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [update 1.1] #11241

Lukas Renggli
Great!

Adrian, is it possible to merge Issue 2102 into Pharo 1.0 too? I think
it is important to have that fixed there too.

Lukas


On 3 March 2010 14:20, Stéphane Ducasse <[hidden email]> wrote:

> 11241
> -----
>
> - Issue 2102:   Fix the fact that local temp can shadow silently instance var
> - cleaning findChgSet
> - fixing packageInfo typos overriden -> overridden
> - Fix file = and added one test
>
> Stef
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>



--
Lukas Renggli
http://www.lukas-renggli.ch

_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [update 1.1] #11241

Adrian Lienhard
Yes, I agree. I'll do it this evening.

Adrian

On Mar 3, 2010, at 14:32 , Lukas Renggli wrote:

> Great!
>
> Adrian, is it possible to merge Issue 2102 into Pharo 1.0 too? I think
> it is important to have that fixed there too.
>
> Lukas
>
>
> On 3 March 2010 14:20, Stéphane Ducasse <[hidden email]> wrote:
>> 11241
>> -----
>>
>> - Issue 2102:   Fix the fact that local temp can shadow silently instance var
>> - cleaning findChgSet
>> - fixing packageInfo typos overriden -> overridden
>> - Fix file = and added one test
>>
>> Stef
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>
>
>
> --
> Lukas Renggli
> http://www.lukas-renggli.ch
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [update 1.1] #11241

Stéphane Ducasse
there is a problem do not integrate it.

We cannot have twice the same block arg in a method

        [:each | each ...]
        [:each | each ...]

I will revert the change in 11242

Stef
       
On Mar 3, 2010, at 3:01 PM, Adrian Lienhard wrote:

> Yes, I agree. I'll do it this evening.
>
> Adrian
>
> On Mar 3, 2010, at 14:32 , Lukas Renggli wrote:
>
>> Great!
>>
>> Adrian, is it possible to merge Issue 2102 into Pharo 1.0 too? I think
>> it is important to have that fixed there too.
>>
>> Lukas
>>
>>
>> On 3 March 2010 14:20, Stéphane Ducasse <[hidden email]> wrote:
>>> 11241
>>> -----
>>>
>>> - Issue 2102:   Fix the fact that local temp can shadow silently instance var
>>> - cleaning findChgSet
>>> - fixing packageInfo typos overriden -> overridden
>>> - Fix file = and added one test
>>>
>>> Stef
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>>
>>
>>
>> --
>> Lukas Renggli
>> http://www.lukas-renggli.ch
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [update 1.1] #11241

Adrian Lienhard
ok, I'll wait.

Adrian

On Mar 3, 2010, at 18:39 , Stéphane Ducasse wrote:

> there is a problem do not integrate it.
>
> We cannot have twice the same block arg in a method
>
> [:each | each ...]
> [:each | each ...]
>
> I will revert the change in 11242
>
> Stef
>
> On Mar 3, 2010, at 3:01 PM, Adrian Lienhard wrote:
>
>> Yes, I agree. I'll do it this evening.
>>
>> Adrian
>>
>> On Mar 3, 2010, at 14:32 , Lukas Renggli wrote:
>>
>>> Great!
>>>
>>> Adrian, is it possible to merge Issue 2102 into Pharo 1.0 too? I think
>>> it is important to have that fixed there too.
>>>
>>> Lukas
>>>
>>>
>>> On 3 March 2010 14:20, Stéphane Ducasse <[hidden email]> wrote:
>>>> 11241
>>>> -----
>>>>
>>>> - Issue 2102:   Fix the fact that local temp can shadow silently instance var
>>>> - cleaning findChgSet
>>>> - fixing packageInfo typos overriden -> overridden
>>>> - Fix file = and added one test
>>>>
>>>> Stef
>>>>
>>>> _______________________________________________
>>>> Pharo-project mailing list
>>>> [hidden email]
>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>>
>>>
>>>
>>>
>>> --
>>> Lukas Renggli
>>> http://www.lukas-renggli.ch
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
Reply | Threaded
Open this post in threaded view
|

Re: [update 1.1] #11241

Stéphane Ducasse
I reverted the changes because you could get like me an error loading code :)

Stef

On Mar 3, 2010, at 6:48 PM, Adrian Lienhard wrote:

> ok, I'll wait.
>
> Adrian
>
> On Mar 3, 2010, at 18:39 , Stéphane Ducasse wrote:
>
>> there is a problem do not integrate it.
>>
>> We cannot have twice the same block arg in a method
>>
>> [:each | each ...]
>> [:each | each ...]
>>
>> I will revert the change in 11242
>>
>> Stef
>>
>> On Mar 3, 2010, at 3:01 PM, Adrian Lienhard wrote:
>>
>>> Yes, I agree. I'll do it this evening.
>>>
>>> Adrian
>>>
>>> On Mar 3, 2010, at 14:32 , Lukas Renggli wrote:
>>>
>>>> Great!
>>>>
>>>> Adrian, is it possible to merge Issue 2102 into Pharo 1.0 too? I think
>>>> it is important to have that fixed there too.
>>>>
>>>> Lukas
>>>>
>>>>
>>>> On 3 March 2010 14:20, Stéphane Ducasse <[hidden email]> wrote:
>>>>> 11241
>>>>> -----
>>>>>
>>>>> - Issue 2102:   Fix the fact that local temp can shadow silently instance var
>>>>> - cleaning findChgSet
>>>>> - fixing packageInfo typos overriden -> overridden
>>>>> - Fix file = and added one test
>>>>>
>>>>> Stef
>>>>>
>>>>> _______________________________________________
>>>>> Pharo-project mailing list
>>>>> [hidden email]
>>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> Lukas Renggli
>>>> http://www.lukas-renggli.ch
>>>>
>>>> _______________________________________________
>>>> Pharo-project mailing list
>>>> [hidden email]
>>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>>
>>>
>>> _______________________________________________
>>> Pharo-project mailing list
>>> [hidden email]
>>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>>
>>
>> _______________________________________________
>> Pharo-project mailing list
>> [hidden email]
>> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
>
>
> _______________________________________________
> Pharo-project mailing list
> [hidden email]
> http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project


_______________________________________________
Pharo-project mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project