[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 2/2] move src/audio to src/modules/audio
From: |
William Hubbs |
Subject: |
[PATCH 2/2] move src/audio to src/modules/audio |
Date: |
Mon, 20 Sep 2010 14:18:46 -0500 |
On Mon, Sep 20, 2010 at 10:56:58AM -0500, William Hubbs wrote:
> On Mon, Sep 20, 2010 at 02:44:48PM +0200, Andrei Kholodnyi wrote:
> > > The split that makes sense, I think, in the current state,
> > > is between the libraries that are needed or helpful in
> > > creating a module, and the modules themselves.
> >
> > > So another structure that makes sense would be I think:
> >
> > > src/modules/common (module_utils.c etc.)
> > > src/modules/common/audio (spd_audio.c etc.)
> > > src/modules/festival
> > > src/modules/espeak
> > > etc.
> >
> > I'd rather put audio under
> > src/modules/audio - by that means it is also clear that audio is a
> > common module.
>
> I agree with src/modules/audio; that is what my patch does.
> Also I like the split that you are talking about, Hynek, breaking down
> the modules directory based on common code and code for each module. I
> can write a patch for that pretty easily.
Andrei,
I think we can go ahead with moving src/audio to src/common/audio, and I
am working right now on a patch which will split up src/modules further
based on Hynek's suggestion.
What do you think?
Thanks,
William
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL:
<http://lists.freebsoft.org/pipermail/speechd/attachments/20100920/68eb0364/attachment.pgp>
- [PATCH 2/2] move src/audio to src/modules/audio, (continued)
- [PATCH 2/2] move src/audio to src/modules/audio, William Hubbs, 2010/09/18
- [PATCH 2/2] move src/audio to src/modules/audio, Andrei Kholodnyi, 2010/09/18
- [PATCH 2/2] move src/audio to src/modules/audio, Christopher Brannon, 2010/09/18
- [PATCH 2/2] move src/audio to src/modules/audio, William Hubbs, 2010/09/18
- [PATCH 2/2] move src/audio to src/modules/audio, Andrei Kholodnyi, 2010/09/18
- [PATCH 2/2] move src/audio to src/modules/audio, William Hubbs, 2010/09/18
- [PATCH 2/2] move src/audio to src/modules/audio, Hynek Hanke, 2010/09/20
- [PATCH 2/2] move src/audio to src/modules/audio, Bohdan R . Rau, 2010/09/20
- [PATCH 2/2] move src/audio to src/modules/audio, Andrei Kholodnyi, 2010/09/20
- [PATCH 2/2] move src/audio to src/modules/audio, William Hubbs, 2010/09/20
- [PATCH 2/2] move src/audio to src/modules/audio,
William Hubbs <=
- [PATCH 2/2] move src/audio to src/modules/audio, Andrei Kholodnyi, 2010/09/20
- [PATCH 2/2] move src/audio to src/modules/audio, William Hubbs, 2010/09/20