Great! I still want to add the support to optionally show icons of all
the files placed on Desktop folder.. I'm working on that.

On Mon, Feb 16, 2009 at 2:29 AM, Julien Danjou <[email protected]> wrote:
> At 1234753836 time_t, Antonio Terceiro wrote:
>> I agree with, you are reaching something very good.
>
> Well, I'm open to inclusion of such module FWIW.
> When you think it's ready for inclusion do not hesitate to ask for a
> review here.
>
> What's needed in your case is just a branch of awesome with your work as
> a patch on top of it so I can easily merge it.
>
> I don't really have time to do a first review right now, but at first
> sight the only thing I'd say is to add at least:
> * standard headers at the beginning of each module
> * code comments (where needed)
> * proper luadoc at the beginning of functions
> * make function local if needed (do not expert everything if it's not
>  needed)
>

-- 
To unsubscribe, send mail to [email protected].

Reply via email to