>.. this won't compile if S377m is disabled. Just go with the old #if >solution, sorry for causing confusion My mistake but I think it is a good idea to get rid of some bulky #if... Do you think a mixed usage of if and #if would be acceptable ? I think the following would be the best : if (!CONFIG_S337M_DEMUXER || st->codecpar->codec_id != AV_CODEC_ID_DOLBY_E) { (...) #if CONFIG_S337M_DEMUXER } else { (... using s337m_get_packet method ... ) #endif }
NB: I have to rework all the patchset with carl's feedback, but this point will still remain, so better to have it solved now. Thank you Nicolas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".