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

Juha Jeronen juha.jeronen at jyu.fi
Mon Mar 28 21:56:16 CEST 2011

Hi all,

Updated Phosphor patch attached. The related minor patch to RenderX is
complete and hasn't been touched; reposting that one only to keep the
set together.

Please check if this version is ok or if it still needs something :)

Without further ado, the...


- Minor shortening to PHOSPHOR_CHROMA_LONGTEXT. Now I think it's optimal.

- Renamed PlaneToFieldPlane() to FieldFromPlane(), more consistent with
the arguments.

- NULL in ppsz_filter_options is now on a separate line to accommodate
future changes more easily.

- DarkenField() updated:
  - initialize remove_high_u8 and ..._u64 in the suggested, more
sensible way
  - MMX implementation of 4:2:2 chroma handler now 2x shorter and faster
  - temporary variable "I" eliminated
  - i_strength made const
  - use (1 << i_strength) instead of variable "div"
  - variable "div" eliminated

- Open():
  - don't initialize p_sys->phosphor.i_chroma_for_420 twice
  - catch zero for Phosphor settings as part of the out-of-range check

That's it for today,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-RenderX-cache-vlc_CPU.patch
Type: text/x-patch
Size: 1443 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110328/f20a5ab6/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Phosphor-deinterlacer.patch
Type: text/x-patch
Size: 41056 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20110328/f20a5ab6/attachment-0001.bin>

More information about the vlc-devel mailing list