OpusGang / awsmfunc

awesome VapourSynth functions
MIT License
17 stars 4 forks source link

Fix plane count check in bandmask (had always passed). #6

Closed JustinTArthur closed 2 years ago

JustinTArthur commented 2 years ago

This test was always passing because it was creating a single-element list every time. I'm assuming the intention was to finalize an iterable if one was used instead of a sequence, so I maintained the conversion.

quietvoid commented 2 years ago

bandmask only supports a single plane index, so the check itself doesn't make sense.

JustinTArthur commented 2 years ago

I think that's what the check is trying to enforce.

quietvoid commented 2 years ago

The function expects an int already, users shouldn't be passing a list.

quietvoid commented 2 years ago

I just removed the check in 0951c33.