[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: |
Fri, 17 Sep 2010 15:30:52 -0500 |
Hi Andrei,
On Fri, Sep 17, 2010 at 09:34:34PM +0200, Andrei Kholodnyi wrote:
> Personally I dislike having source code and a subdir in one place.
> Module consists of 3 parts, communication part, synth part and an audio part.
> Probably it would be better to have modules/audio, modules/synth,
> modules/main.
Maybe so, and this is the first step to that. It moves the src/audio
directory to src/modules/audio since the audio code is only used by the
modules.
It sounds to me like we are suggesting the same thing.
> Apart of that there are some utils that could be e.g. separated in a lib.
I haven't looked into that yet. lib (or maybe even common) would be for
code that is shared between the server and modules. I don't know of any
code like that yet.
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/20100917/3107a719/attachment-0001.pgp>
- [PATCH 1/2] remove audio include path from tests Makefile, William Hubbs, 2010/09/17
- [PATCH 1/2] remove audio include path from tests Makefile, William Hubbs, 2010/09/17
- [PATCH 2/2] move src/audio to src/modules/audio, William Hubbs, 2010/09/17
- [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, 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