patch to fix bug 8072, but need some help with tests

ricardo filipe ricardo_barbano at hotmail.com
Mon Dec 15 18:51:48 CST 2008


hi guys!
after many days tinkering on this i managed to make the notification work!
(pump fist)

now media player classic correctly displays the files when browsing on "Open File" with no native libraries :)

i would like some feedback on the patch though because there is some "repeated" code and there is a LPITEMIDLIST cast that i've tried to take out using pointers, but i didn't manage to since the variable is a constant... but you guys might know some other way to not use the cast, no?

also i would like some help doing tests for this because i'm sure AJ won't accept this without tests.

thanks :)


_________________________________________________________________
Receba GRÁTIS as mensagens do Messenger no seu celular quando você estiver offline. Conheça  o MSN Mobile!
http://mobile.live.com/signup/signup2.aspx?lc=pt-br
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://www.winehq.org/pipermail/wine-devel/attachments/20081216/27888dd4/attachment-0001.htm 
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 0001-comdlg32-notify-with-CDN_INCLUDEITEM-on-IShellBrowser.txt
Url: http://www.winehq.org/pipermail/wine-devel/attachments/20081216/27888dd4/attachment-0001.txt 


More information about the wine-devel mailing list