[1/3] d3dx9: Partial implementation of D3DXAssembleShader function. [try 3]

Stefan Dösinger stefandoesinger at gmx.at
Mon Feb 8 15:17:32 CST 2010

Am 08.02.2010 um 21:51 schrieb Matteo Bruni:
> In this case the '{' should go on its own line, sure. In general in
> shader.c I tried to keep the style already used there, while in the
> shader assembler files the style is different (although it should be
> consistent on itself).
> Let me know if you believe I should use everywhere the style used in shader.c.
I personally don't have a strong opinion on this. It's certainly preferable to use the same style as in shader.c everywhere, but for me this is not a commit-stopper. Henri is usually pretty uptight about style in wined3d(at least in the parts he has reformatted in the past), but I think if he had objections he'd have voiced them by now.

I'd recommend to talk to the other d3dx9 developers about those issues, and whether you want to set any styles for the library as a whole.

>>> + * Copyright 2009 Matteo Bruni
>> Maybe make this 2009-2010?
> Well, in 2010 I've only done little changes. The real development on
> this was in 2009. What's the policy in these cases? I believe I can
> put the 2010 safely.
I have seen people adjust the years for simple fixes, although I don't know if there is a formal policy. Since your patches are going in now I guess anything from "2009", "2009, 2010" to "2010" is warranted.

Since most countries don't require a copyright notice any longer(although still recommend it) I guess there is no real legal significance to the year. In most places copyright expiration doesn't depend on the publication date but on the year of the death of the last surviving creator.

IANAL, somebody who knows this better please correct me.

More information about the wine-devel mailing list