On Thu, 25.09.08 12:15, Takashi Iwai (tiwai@suse.de) wrote:
I'd be very thankful for comments, especially from you Takashi and Jaroslav since it contains quite some details about the "safe" subset of the ALSA API!
Thanks for pushing this up. I enjoyed this reading a lot, especially about many DONTS that reminds me of your past screams.
Thanks!
Anyway, I'd like to mark async stuff as obsolete in a future version. This is a broken design, and should rest in piece. Meanwhile, I'm not in favor of adding deprecated link warning. A compile warning is fine, but link warning is way too annoying. And there are programs right now using async, we are responsible to keep them running as they are.
You are aware that the linking warnings are only shown during build-time -- not during runtime when dynamic linking happens. So basically the difference between compiler and linker warnings are not that big. Link time warnings just appear a little bit later during build time than compile time warnings.
The big advantage of linker warnings is that you can add arbitrary warning strings when a symbol is used. Doing that with just the compiler is impossible to my knowledge.
I.e. just define this:
#ifdef __GNUC__ #define WARN_REFERENCE(sym, msg) \ __asm__(".section .gnu.warning." #sym); \ __asm__(".asciz "" msg """); \ __asm__(".previous") #else #define WARN_REFERENCE(sym, msg) #endif
And then put a line like this next to the implementations of the symbols you want to warn the developer about:
WARN_REFERENCE(snd_async_add_pcm_handler, "Please do not use async handlers anymore. For more details, read http://mailman.alsa-project.org/pipermail/alsa-devel/2008-May/008030.html");
or
WARN_REFERENCE(snd_pcm_sw_params_set_xfer_align, "You are using snd_pcm_sw_params_set_xfer_align() which is obsolete and a NOP. You may safely remove it from your sources.");
and so on.
And everytime someone builds a piece of software that uses a symbol like this he gets are very clear explanation what is going and what he should be fixing. The end user won't see anything about this.
Does that sound acceptable to you?
Lennart