[webkit-dev] Web Audio API

[Darin Fisher]

My objection (and it's only a slight one) was about using "Web" as a prefix
for class names defined in WebCore.

WebSockets is the main example of the Web prefix used in WebCore, and that's
probably because "sockets" by itself would be too confusing.  However, I
have found the use of the Web prefix in WebCore to lead to some confusion by
itself since WebKit layers tend to use the Web prefix for their
classes/interfaces.

I realize that the WebCore:: namespace makes this issue technically moot.
 I'm just concerned about it being confusing to have WebCore::WebFoo and
WebKit API level WebFoo.

Regards,
-Darin


On Tue, Aug 24, 2010 at 5:29 PM, Chris Rogers <crogers at google.com> wrote:

> Good, it looks like we're getting close.  So we've agreed to how the files
> should be split up, but Darin Fisher still was concerned about the 'web'
> prefix.
>
> Darin, was it the directory name WebCore/webaudio that you didn't like or:
> #if ENABLE(WEB_AUDIO)
>
> Alternatives might be:
>
> WebCore/audio   or    WebCore/audiocontext
> #if ENABLE(AUDIO_CONTEXT)     or      #if ENABLE(AUDIO_API)
>
> I'm assuming that WebCore/platform/audio we can all agree on...
>
> Chris
>
>
> On Tue, Aug 24, 2010 at 5:20 PM, Simon Fraser <simon.fraser at apple.com>wrote:
>
>> On Aug 24, 2010, at 4:47 PM, Chris Rogers wrote:
>>
>> Hi Simon, thanks for helping here.
>>
>> By the way, anybody who is interested can look at the files in:
>> https://svn.webkit.org/repository/webkit/branches/audio/WebCore/audio/
>>
>> There are actually a number of audio files which could be considered
>> re-usable, although I know of nothing at this time other than the web audio
>> API which would use them:
>>
>> AudioBus.cpp
>> Biquad.cpp
>> Reverb.cpp
>> FFTConvolver.cpp (and other FFT-related files)
>>
>> possibly also in this category are:
>> Cone.cpp
>> Distance.cpp
>>  MidSide.cpp
>> SinWave.cpp
>> Panner.cpp (and subclasses)
>> (maybe a few others I've missed)
>>
>> Basically, these are the lowest-level building blocks which the
>> higher-level parts (such as AudioContext, and AudioNode) use.  These
>> lowest-level building blocks do not (or should not) have any dependencies to
>> the higher-level code which implement the actual API (and have IDL files).
>>  They also don't have any dependencies on other parts of WebCore, although
>> they do use stuff in wtf.
>>
>> So are you suggesting:
>>
>> WebCore/webaudio                <--- IDL files and API implementation
>> WebCore/platform/audio        <--- lower-level building blocks such as
>> AudioBus.cpp
>>
>>
>> Fine by me!
>>
>>
>> then with the conditional
>>
>> #if WEB_AUDIO            <----- I added an underscore here
>>
>>
>> That would be #if ENABLE(WEB_AUDIO) in the code.
>>
>> Simon
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20100824/56c8bcb1/attachment.html>