[MPlayer-dev-eng] [PATCH] Final (hopefully) 3d denoise patch

[Arpi]

Hi,

> > ok applied, with lots of modifications.
> > your code assumed that tere are always 2 mpi buffers, and so
> > dmpi==previous_frame==destination_frame. it's bad.
> 
> Uhmmm... I really miss some sort of "conceptual overview" about all 
> this. At least it'd provide a method (other than trial and error) for 
> new coders to do things. But well... this "we'll not teach you until you 
> already know it" sounds familiar from when I was on mplayer-users... }:)

i said that read DOCS?tech/libmpcodecs.txt
imho it explains this quite well, at least you wouldn't make so big
mistakes after rading that.

anyway my change was broken, just fixed in cvs.
it caused temporal thing nonexisting (prev and current frame was the same)

> > ok it's still far from being readable by a hardcore C coder...
> > it reminds me to some pascal examples from an old programming book :)
> 
> This would explain my idea of C being an uglification of Pascal. I'm 

:)


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
    "However, many people beg for its inclusion in Debian. Why?" - Gabucino
  "Because having new software in Debian is good." - Josselin Mouette
"Because having good software in Debian is new." - Gabucino