[LAD] "enhanced event port" LV2 extension proposal

[Lars Luthman]

On Sat, 2007-12-01 at 14:09 -0500, Dave Robillard wrote:
> On Fri, 2007-11-30 at 16:21 +0200, Nedko Arnaudov wrote:
> > Krzysztof Foltman <wdev at foltman.com> writes:
> > 
> > > Lars Luthman wrote:
> > >
> > >> non-standard hacks in a specification. But with the current event header
> > >> proposal we don't have a pointer _or_ a flexible array member in it, so
> > >> this discussion is sort of pointless.
> > >
> > > So, basically, we have a choice between:
> > >
> > > struct LV2_EVENT_HEADER_LLKF
> > > {
> > >   uint32_t timestamp; // 16:16
> > >   uint16_t payload_size;
> > >   uint16_t event_type;
> > > };
> 
> Might as well break the time stamp into two separate uint16_t's and make
> life easy.

Agreed. Most plugins won't care about the fractional part. The only
drawback I can think of is that on a platform that aligns struct members
to 32 bits you won't be able to load the complete timestamp as a
uint32_t without some shifting and |ing, but that will probably be a
special case anyway. Having the timestamp as two separate members makes
everything completely self-documented.


> Please, please let this silly 'type of the data member' angle of
> conversation die... :)
> 
> Hereby humbly requesting that:
> 
> // data follows here
> 
> be the last thing in the event struct for the purposes of this
> conversation, since it's irrelevant and not a point of debate

Agreed.


--ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.linuxaudio.org/pipermail/linux-audio-dev/attachments/20071201/c857cf79/attachment.pgp>