Bug #11883 for Apache-VimColor: Implement caching of colourized pages

Tue Mar 15 12:16:52 2005 ARISTOTLE [...] cpan.org - Ticket created

Subject:

Implement caching of colourized pages

Text::VimColor is great, but it's also a hefty load to be incurring on each hit to a page. Since the contents involved are mostly static, it would be good if it wouldn't have to be invoked anew every time for the same file.

Tue Mar 15 12:35:26 2005 octopus [...] nospam.verplant.org - Taken

Tue Mar 15 12:39:54 2005 octopus [...] nospam.verplant.org - Correspondence added

RT-Send-CC:

octo [...] cpan.org

Good point. I did not yet implement a caching feature since I thougt of this module being used rarely to show some code to a mailing list or irc channel or something. However, if this module is being used more often in reality than I though a chaching method is definitely useful. I'll look at it later tonight.. Thanks for your feedback :) -octo

Tue Mar 15 12:54:39 2005 octopus [...] nospam.verplant.org - Cc octo@verplant.org added

Tue Mar 15 15:12:53 2005 octopus [...] nospam.verplant.org - Status changed from 'new' to 'open'

Tue Mar 15 15:12:53 2005 octopus [...] nospam.verplant.org - Correspondence added 90 min

[ARISTOTLE - Tue Mar 15 12:16:52 2005]: Show quoted text

> Text::VimColor is great, but it's also a hefty load to be incurring on > each hit to a page. Since the contents involved are mostly static, > it would be good if it wouldn't have to be invoked anew every time > for the same file.

I've implemented a simple caching algorithm and packed a new release. You can download the new distribution at the following URL, if you want to give it a try. I'll upload it to CPAN tomorrow, unless you find some bugs that need fixing.. http://verplant.org/files/Apache-VimColor-2.00.tar.gz Regards, -octo

Tue Mar 15 16:05:32 2005 Guest - Correspondence added

From:

pagaltzis [...] gmx.de

Show quoted text

> I've implemented a simple caching algorithm and packed a new release. > You can download the new distribution at the following URL, if you want > to give it a try.

The code looks good, but I would suggest you use Cache::Cache instead of rolling your own. That would make it easy for users to specify what kind of cache backend they want (such as they blazing fast Cache::FastMmap), along with extra options to fine-tune it to their situation.

Wed Mar 16 10:47:59 2005 octopus [...] nospam.verplant.org - Correspondence added 120 min

[pagaltzis@gmx.de - Tue Mar 15 15:40:29 2005]: Show quoted text

> The code looks good, but I would suggest you use Cache::Cache > instead of rolling your own. That would make it easy for users to > specify what kind of cache backend they want (such as they > blazing fast Cache::FastMmap), along with extra options to > fine-tune it to their situation.

I've implemented basic support for the default Cache::* Modules, namely Cache::(SizeAware)?((Shared)?Memory|File)Cache. Cache::FastMmap is not (yet) supported, but I wanted to get the original Cache::* modules working before doing fancy new stuff.. Same procedure, I'll wait a bit before uploading to CPAN so you have a chance to catch bugs that hid while I was testing.. http://verplant.org/files/Apache-VimColor-2.10.tar.gz Regards, -octo

Thu Mar 17 07:09:00 2005 ARISTOTLE [...] cpan.org - Correspondence added

From:

pagaltzis [...] gmx.de

[OCTO - Wed Mar 16 10:47:59 2005]: Show quoted text

> I've implemented basic support for the default Cache::* Modules, namely > Cache::(SizeAware)?((Shared)?Memory|File)Cache.

Looks fine to me, thanks.

Thu Mar 17 10:02:35 2005 octopus [...] nospam.verplant.org - Status changed from 'open' to 'resolved'

Thu Mar 17 10:02:35 2005 octopus [...] nospam.verplant.org - Fixed in 2.10 added