'Swap interlaced video field order' option

There needs to be a checkbox to swap the order of the fields of an interlaced video. There also neeed to be a hotkey to toggle that option.

Reasons:

• TV channels often broadcast H.264 or MPEG2 streams with incorrect field order (believe it or not).

• Many high-profile video encoding software suites produce incorrect field order.

added Component::Core: Video output label

It should be trivial to implement as a filter, but I really don't think this is an common (and well understood) enough feature to justify its own hotkey.

Field Swap

Added attachment:

• fieldswap.patch

Here is a swap field filter for VLC.

Replying to [comment:3 jb]:

Here is a swap field filter for VLC.

Thank you! Is this going to be committed into the next release?

Replying to [comment:2 courmisch]:

It should be trivial to implement as a filter, but I really don't think this is an common (and well understood) enough feature to justify its own hotkey.

Here where I live about 20% of the videos I record on my PVR have incorrect field order. It depends on which TV channel you record.

You can imagine that I will have to do A LOT of field switching. You can also imagine that it is very cumbersome to have to open the preferences and search for the correct option every time.

Besides, a default key doesn't have to be assigned to the shortcut. It may be left up to the user to assign it. And adding one more shortcut (very useful for me) to the gazillion of shortcuts already there cannot be a real problem, can it?

Replying to [comment:4 vadiav]:

Replying to [comment:3 jb]:

Here is a swap field filter for VLC.

Thank you! Is this going to be committed into the next release?

No idea. It's not really tested.

Do you want/can test it?

Replying to [comment:7 jb]:

Do you want/can test it?

Sure! I'd be glad to. Do I have to build VLC or is there a test binary?

"bump"

Implementing this feature would be great for me too!

jb, HOW can we test this patch? I mean without having to compile stuff?

What is the status of this RFE? Is anybody going to work on it any further? Can I help testing or developing it?

You have the patch, you can compile and test it.

The malloc() and free() are useless in this case.

But is that really what the bug report wants? Wouldn´t it be simpler to invert the TFF/BFF boolean in the picture and let the deinterlacer deal with it?

added Component::Video: filters label and removed Component::Core: Video output label

It probably would be easier, yes. But then it should be in the core, as a new value for deinterlace ? (Off/On/Auto/Reverse)?

pic->b_top_field_first = !pic->b_top_field_first ??

Probably, yes.

Btw, I think that the fieldswap patch was different, and was not enough for this ticket.

And please add a hotkey shortcut for this function too. Thank you! :)