[vlc-devel] [PATCH] Phosphor deinterlacer, reworked += 3

Juha Jeronen juha.jeronen at jyu.fi
Sun Mar 6 00:26:57 CET 2011


Hi all,

Patch split into two as suggested. Aside from that, I updated some of
the new comments and removed an unused parameter that was accidentally
left in in the previous version.

Please review the attached version. If needed, I can make one more round
of changes for this set. After that, I think it's time to declare this
set complete :)


Changelog (on top of "reworked += 2"):

- minor comment fixes for the new stuff in vlc_picture.h
- added back inadvertently deleted blank line after set_subcategory(
SUBCAT_VIDEO_VFILTER ) (no cosmetics!)
- minor update to PHOSPHOR_CHROMA_LONGTEXT: mention that Upconvert
produces the most accurate simulation
- minor update to the heading comment of the Phosphor section: light
decay, not just luma decay
- minor function comment update to ComposeFrame(): mention the specific
chroma conversions that are done
- DarkenField(): removed the parameter p_filter (not needed)
- function comment update to RenderPhosphor():
  - explain the different 4:2:0 chroma modes better
  - fixed incorrect comment about Bob (obviously, it doesn't render the
old field!)
  - added note on main differences to an actual CRT TV
  - updated explanation for i_order/i_field (still not good but will do
for now)


 -J

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Added-plane_ToFieldPlane-and-plane_CopyPixelsLineByL.patch
Type: text/x-patch
Size: 5622 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110306/b5d0c24c/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Phosphor-deinterlacer.patch
Type: text/x-patch
Size: 41082 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110306/b5d0c24c/attachment-0001.bin>


More information about the vlc-devel mailing list