On Monday 28 January 2008 02:25:21 Bryan Henderson wrote:
> In the absence of any more expert information, I looked into this
> more myself, and am convinced that someone just got lost in this code
> and the initialization that is under if (autoload) {} isn't supposed
> to be there. Many things besides VBI fail to get initialized unless
> you autoload.
>
> The patch below fixes it. It also moves the ifdefs so the code is
> easier to read.
>
> The changes are much smaller than the patch makes it look, because of
> indentation changes. All the patch does is put the autoload test on
> the actual autoloads instead of a whole block of initialization, and
> fold the ifdef into that test.
Hi Bryan,
Yes, you are right that this is a bug. However, the whole autoload
module option is basically bogus and has in fact been removed in later
ivtv versions (the ivtv-0.10 branch no longer has it). So I won't be
applying this patch (sorry) since I'm not updating 0.4 anymore except
for really important bug fixes, which this isn't.
Regards,
Hans
_______________________________________________
ivtv-devel mailing list
[email protected]
http://ivtvdriver.org/mailman/listinfo/ivtv-devel
