[ENH] Faster RBParser>>addCommentsTo:

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

[ENH] Faster RBParser>>addCommentsTo:

Andres Valloud-2
Hello VW,

  I gained another 10% RB speedup by avoiding creating collections
  when it is not necessary.  Total speedup so far: ~65%.  Enjoy!

--
Best regards,
 Andres                          mailto:[hidden email]

RBParser-addCommentsTo.st (922 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: [ENH] Faster RBParser>>addCommentsTo:

Adrian Kuhn-3
Are your changes available on Public Store?

-- Adrian



On 6. Nov 2006, at 9:37 , Andres Valloud wrote:

> Hello VW,
>
>   I gained another 10% RB speedup by avoiding creating collections
>   when it is not necessary.  Total speedup so far: ~65%.  Enjoy!
>
> --
> Best regards,
>  Andres                          mailto:[hidden email]
> <RBParser-addCommentsTo.st>

Reply | Threaded
Open this post in threaded view
|

Re: [ENH] Faster RBParser>>addCommentsTo:

Joachim Geidel
In reply to this post by Andres Valloud-2
Hi Andres,

I have published your patches together with some older ones as a package
called 'Tools-Speed patches' in the public repository. I added one the
patches (Store.Package>>addClasses:andMetaclasses:) to the existing
package 'Store-Speed patches' instead, as it is specific to Store.

Best regards,
Joachim

Andres Valloud wrote on 06.11.2006 09:37:
> Hello VW,
>
>   I gained another 10% RB speedup by avoiding creating collections
>   when it is not necessary.  Total speedup so far: ~65%.  Enjoy!
>

Reply | Threaded
Open this post in threaded view
|

Re[2]: [ENH] Faster RBParser>>addCommentsTo:

Andres Valloud-2
Hello Joachim,

Sunday, November 19, 2006, 9:50:12 AM, you wrote:

JG> I have published your patches together with some older ones as a
JG> package called 'Tools-Speed patches' in the public repository. I
JG> added one the patches (Store.Package>>addClasses:andMetaclasses:)
JG> to the existing package 'Store-Speed patches' instead, as it is
JG> specific to Store.

I expressed my gratitude in my blog regarding this, but I'd like to
make it explicit here as well.  I really appreciate that you have put
time into making this happen.  As of late, I have been extraordinarily
busy and there is no chance I could have finished doing this myself in
a reasonable amount of time.  I hope what I did is of use :).

Thanks again!

--
Best regards,
 Andres                            mailto:[hidden email]