[vlc-devel] [PATCH] Direct3D shader support (HLSL)

xxcv xxcv07 at gmail.com
Sat Aug 14 13:13:27 CEST 2010

Hi JB,
On 14/08/2010 9:06 PM, Jean-Baptiste Kempf wrote:
> Hello Sasha,
> On Fri, Aug 13, 2010 at 10:34:44PM -0700, Sasha Koruga wrote :
>> I hope I am doing this right! I git cloned the latest vlc version
>> today, and added the changes I've been working on. The patches are
>> attached to this email.
I'm afraid to tell you that you're 3-4 patches late :)
> Thanks a lot for your work. This is very needed for VLC.
> About your patches:
>   - please do NOT send .zip on this mailing list. This makes it very hard
>     to review
>   - please learn to squash your patches, so you only submit complete
>     patches. For example, all your untabify patches shouldn't even exist.
>     To do that correctly, use git rebase -i origin/master
>     This will help to reorder, squash, etc... your patches.
>     1 and 2 should be merged, for example
>   - please make your patches with a better title, for example 3 and 4
>     have the same title, but are not the same.
This has already been done see here:
>   - please separate, when possible cosmetics from actual code changes.

> Shouldn't the shaders be in special files?
> If if (!sys->hd3d9x_dll), shouldn't you delete the shader-effect
> callback too?
> Best Regards,

My VLC, My Way, My Style~

More information about the vlc-devel mailing list