On 07/05/27 17:31 +0200, A. Pagaltzis wrote:
> * Jerome Quelin <[EMAIL PROTECTED]> [2007-05-27 09:50]:
> > so here's my question: how should i name this new module?
> >  - audio::item    => isn't it too vague?
> >  - audio::mpditem => isn't it too specific? (would it be used by others?)
> >  - other?
> 
> The modules I see are just structs, and the only code in there is
> the formatter method in Audio::MPD::Item::Song. I does not seem
> to me that these things are very generically useful; you’d have
> to do a lot of work to make them interesting for other purposes.

they are indeed just structs, as i said.

> I’d suggest Audio::MPD::Item as the distro name, since you’ve
> already organised them as a hierarchy in there. As a bonus you
> won’t even need to change any of the Audio::MPD code, just add
> the dependency and you’re done.

well, i don't like the way to prereq sthg in pococm that i won't really
use - that is, the whole non-poe logic of audio::mpd. i'd really like to
have a small dist that could be used by the two. it would be way
cleaner imho.

after having sent this mail, i thought that i could as well add other
helper classes used by both audio::mpd and pococ::mpd...

==> maybe Audio::MPDHelper could do the trick?


jérôme 
-- 
[EMAIL PROTECTED]

Reply via email to