[alsa-devel] [PATCH] salsa-lib: Add seq_midi_event.h

[Takashi Iwai]

At Sun, 17 Mar 2013 14:57:31 +0000,
Mihail Zenkov wrote:
> 
> --- salsa-lib-a55ce0c348cd16e7853dae11e8e0e1b5a101d884/configure.ac
> +++ salsa-lib-a55ce0c348cd16e7853dae11e8e0e1b5a101d884-1/configure.ac
> @@ -273,6 +273,7 @@
>  test "$sndseq" = "yes" && echo "#include \"seq_event.h\"" >> src/asoundlib.h
>  test "$sndseq" = "yes" && echo "#include \"seq.h\"" >> src/asoundlib.h
>  test "$sndseq" = "yes" && echo "#include \"seqmid.h\"" >> src/asoundlib.h
> +test "$sndseq" = "yes" && echo "#include \"seq_midi_event.h\"" >>
> src/asoundlib.h
>  cat "$srcdir"/src/asoundlib-tail.h >> src/asoundlib.h
> 
>  echo "SALSA-Lib build configuration:"
> --- salsa-lib-a55ce0c348cd16e7853dae11e8e0e1b5a101d884/src/Makefile.am
> +++ salsa-lib-a55ce0c348cd16e7853dae11e8e0e1b5a101d884-1/src/Makefile.am
> @@ -60,7 +60,7 @@
>  endif
>  if BUILD_SEQ
>  alsainclude_HEADERS += \
> -	seq.h seq_event.h seqmid.h
> +	seq.h seq_event.h seqmid.h seq_midi_event.h
>  endif
> 
>  noinst_HEADERS = local.h
> --- /dev/null
> +++ salsa-lib-a55ce0c348cd16e7853dae11e8e0e1b5a101d884-1/src/seq_midi_event.h
> @@ -0,0 +1,78 @@
> +#ifndef __ALSA_SEQ_MIDI_EVENT_H
> +#define __ALSA_SEQ_MIDI_EVENT_H
> +
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> +/**
> + *  \defgroup MIDI_Event Sequencer event <-> MIDI byte stream coder
> + *  \ingroup Sequencer
> + *  Sequencer event <-> MIDI byte stream coder
> + *  \{
> + */
> +
> +/** container for sequencer midi event parsers */
> +typedef struct snd_midi_event {
> +	size_t qlen;
> +	size_t read;
> +	int type;
> +	unsigned char lastcmd;
> +	unsigned char nostat;
> +	size_t bufsize;
> +	unsigned char *buf;
> +} snd_midi_event_t;

We don't have to define the actual struct here as all the functions
are just dummy without referring to the real data.

I'd like to avoid exposing it unless it's really needed.


> +__SALSA_EXPORT_FUNC
> +int snd_midi_event_new(size_t bufsize, snd_midi_event_t **rdev)
> +{
> +	return 0;

This should return an error.  We don't support the feature, so have to
notify it.

Ditto for the rest functions.


thanks,

Takashi


> +}
> +__SALSA_EXPORT_FUNC
> +int snd_midi_event_resize_buffer(snd_midi_event_t *dev, size_t bufsize)
> +{
> +	return 0;
> +}
> +__SALSA_EXPORT_FUNC
> +void snd_midi_event_free(snd_midi_event_t *dev)
> +{
> +}
> +__SALSA_EXPORT_FUNC
> +void snd_midi_event_init(snd_midi_event_t *dev)
> +{
> +}
> +__SALSA_EXPORT_FUNC
> +void snd_midi_event_reset_encode(snd_midi_event_t *dev)
> +{
> +}
> +__SALSA_EXPORT_FUNC
> +void snd_midi_event_reset_decode(snd_midi_event_t *dev)
> +{
> +}
> +__SALSA_EXPORT_FUNC
> +void snd_midi_event_no_status(snd_midi_event_t *dev, int on)
> +{
> +}
> +__SALSA_EXPORT_FUNC
> +long snd_midi_event_encode(snd_midi_event_t *dev, const unsigned char
> *buf, long count, snd_seq_event_t *ev)
> +{
> +	return 0;
> +}
> +__SALSA_EXPORT_FUNC
> +int snd_midi_event_encode_byte(snd_midi_event_t *dev, int c,
> snd_seq_event_t *ev)
> +{
> +	return 0;
> +}
> +__SALSA_EXPORT_FUNC
> +long snd_midi_event_decode(snd_midi_event_t *dev, unsigned char *buf,
> long count, const snd_seq_event_t *ev)
> +{
> +	return 0;
> +}
> +
> +
> +#ifdef __cplusplus
> +}
> +#endif
> +
> +#endif /* __ALSA_SEQ_MIDI_EVENT_H */
> +
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>