ocompletion bug fix

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

ocompletion bug fix

SeanTAllen
I just fixed a bug that has been driving me crazy for a while in the
Ocompletion package.

When you select an area of text and delete, if you are deleting all of
it, it throws a subscript out of bounds error.

What is the proper process for getting this change integrated?
I'm not really sure who this small fix should be sent to.

_______________________________________________
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: [Pharo-project] ocompletion bug fix

Adrian Lienhard
It is best to create an issue in our tracker and put Romain Robbes, the maintainer of Ocompletion, in cc. Also post the issue URL to the mailing list.

This problem, though, sounds familiar to me. Which versions are you using? You may find already closed reports of this problem in the tracker.

Thanks,
Adrian

On Jan 20, 2010, at 18:59 , Sean Allen wrote:

> I just fixed a bug that has been driving me crazy for a while in the
> Ocompletion package.
>
> When you select an area of text and delete, if you are deleting all of
> it, it throws a subscript out of bounds error.
>
> What is the proper process for getting this change integrated?
> I'm not really sure who this small fix should be sent to.
>
> _______________________________________________
> 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: [Pharo-project] ocompletion bug fix

Mariano Martinez Peck
Yes, please, when submit the issue tracker, said exactly which version and browser are you using and if you can say the OCompletion version, better

On Wed, Jan 20, 2010 at 3:06 PM, Adrian Lienhard <[hidden email]> wrote:
It is best to create an issue in our tracker and put Romain Robbes, the maintainer of Ocompletion, in cc. Also post the issue URL to the mailing list.

This problem, though, sounds familiar to me. Which versions are you using? You may find already closed reports of this problem in the tracker.

Thanks,
Adrian

On Jan 20, 2010, at 18:59 , Sean Allen wrote:

> I just fixed a bug that has been driving me crazy for a while in the
> Ocompletion package.
>
> When you select an area of text and delete, if you are deleting all of
> it, it throws a subscript out of bounds error.
>
> What is the proper process for getting this change integrated?
> I'm not really sure who this small fix should be sent to.
>
> _______________________________________________
> 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: [Pharo-project] ocompletion bug fix

SeanTAllen
In reply to this post by Adrian Lienhard
I could find anything in the tracker...

We are currently using:

Pharo image: Pharo
Pharo core version: PharoCore1.0rc1 Latest update: #10500
Virtual machine used: pharo-vm-0.15.2f-linux


On Wed, Jan 20, 2010 at 1:06 PM, Adrian Lienhard <[hidden email]> wrote:

> It is best to create an issue in our tracker and put Romain Robbes, the maintainer of Ocompletion, in cc. Also post the issue URL to the mailing list.
>
> This problem, though, sounds familiar to me. Which versions are you using? You may find already closed reports of this problem in the tracker.
>
> Thanks,
> Adrian
>
> On Jan 20, 2010, at 18:59 , Sean Allen wrote:
>
>> I just fixed a bug that has been driving me crazy for a while in the
>> Ocompletion package.
>>
>> When you select an area of text and delete, if you are deleting all of
>> it, it throws a subscript out of bounds error.
>>
>> What is the proper process for getting this change integrated?
>> I'm not really sure who this small fix should be sent to.
>>
>> _______________________________________________
>> 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: [Pharo-project] ocompletion bug fix

SeanTAllen
In reply to this post by Adrian Lienhard
I see it was fixed once I went to check the Ocompletion version, in
the next version after what we had installed:

Ocompletion-DannyChan.44 fixes the bug.

I didn't find a ticket in the issue tracker though.


On Wed, Jan 20, 2010 at 1:06 PM, Adrian Lienhard <[hidden email]> wrote:

> It is best to create an issue in our tracker and put Romain Robbes, the maintainer of Ocompletion, in cc. Also post the issue URL to the mailing list.
>
> This problem, though, sounds familiar to me. Which versions are you using? You may find already closed reports of this problem in the tracker.
>
> Thanks,
> Adrian
>
> On Jan 20, 2010, at 18:59 , Sean Allen wrote:
>
>> I just fixed a bug that has been driving me crazy for a while in the
>> Ocompletion package.
>>
>> When you select an area of text and delete, if you are deleting all of
>> it, it throws a subscript out of bounds error.
>>
>> What is the proper process for getting this change integrated?
>> I'm not really sure who this small fix should be sent to.
>>
>> _______________________________________________
>> 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: [Pharo-project] ocompletion bug fix

Henrik Sperre Johansen
On 20.01.2010 19:35, Sean Allen wrote:
> I see it was fixed once I went to check the Ocompletion version, in
> the next version after what we had installed:
>
> Ocompletion-DannyChan.44 fixes the bug.
>
> I didn't find a ticket in the issue tracker though.
>    
Did you try searching the closed issues?
Just searching for OCompletion, I get:
http://code.google.com/p/pharo/issues/detail?id=1554
http://code.google.com/p/pharo/issues/detail?id=1567
and
http://code.google.com/p/pharo/issues/detail?id=1635

:)

Cheers,
Henry

_______________________________________________
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: [Pharo-project] ocompletion bug fix

SeanTAllen
On Wed, Jan 20, 2010 at 2:31 PM, Henrik Sperre Johansen
<[hidden email]> wrote:

> On 20.01.2010 19:35, Sean Allen wrote:
>> I see it was fixed once I went to check the Ocompletion version, in
>> the next version after what we had installed:
>>
>> Ocompletion-DannyChan.44 fixes the bug.
>>
>> I didn't find a ticket in the issue tracker though.
>>
> Did you try searching the closed issues?
>

obviously not. ;)

_______________________________________________
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: [Pharo-project] ocompletion bug fix

Adrian Lienhard
In reply to this post by SeanTAllen
The issue I remembered was named "Issue 1567: Highlight all -> hit backspace -> debugger appears".

As far as I can tell this is fixed in the latest Pharo image (10.01.1). It contains version Ocompletion-RomainRobbes.52. Please try with the most recent image and report back if you still see the problem.

Please also note that doing a software update does not update packages like Ocompletion (this is confusing and will be changed).

Cheers,
Adrian

On Jan 20, 2010, at 19:35 , Sean Allen wrote:

> I see it was fixed once I went to check the Ocompletion version, in
> the next version after what we had installed:
>
> Ocompletion-DannyChan.44 fixes the bug.
>
> I didn't find a ticket in the issue tracker though.
>
>
> On Wed, Jan 20, 2010 at 1:06 PM, Adrian Lienhard <[hidden email]> wrote:
>> It is best to create an issue in our tracker and put Romain Robbes, the maintainer of Ocompletion, in cc. Also post the issue URL to the mailing list.
>>
>> This problem, though, sounds familiar to me. Which versions are you using? You may find already closed reports of this problem in the tracker.
>>
>> Thanks,
>> Adrian
>>
>> On Jan 20, 2010, at 18:59 , Sean Allen wrote:
>>
>>> I just fixed a bug that has been driving me crazy for a while in the
>>> Ocompletion package.
>>>
>>> When you select an area of text and delete, if you are deleting all of
>>> it, it throws a subscript out of bounds error.
>>>
>>> What is the proper process for getting this change integrated?
>>> I'm not really sure who this small fix should be sent to.
>>>
>>> _______________________________________________
>>> 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: [Pharo-project] ocompletion bug fix

Stéphane Ducasse
In reply to this post by SeanTAllen
yeahhhhhhh
me too. Good to hear that it was killed!

Stef
On Jan 20, 2010, at 6:59 PM, Sean Allen wrote:

> I just fixed a bug that has been driving me crazy for a while in the
> Ocompletion package.
>
> When you select an area of text and delete, if you are deleting all of
> it, it throws a subscript out of bounds error.
>
> What is the proper process for getting this change integrated?
> I'm not really sure who this small fix should be sent to.
>
> _______________________________________________
> 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