[PATCH] wined3d: Implement per-stage constant in glsl fixed fonction pipeline.
titan.costa at gmail.com
Tue Jun 25 11:08:57 CDT 2013
2013/6/25 Henri Verbeet <hverbeet at gmail.com>
> On 25 June 2013 11:05, Christian Costa <titan.costa at gmail.com> wrote:
> > 2013/6/25 Henri Verbeet <hverbeet at gmail.com>
> >> At least try to pretend you put
> >> some effort into making sure the patch is as good as it can be before
> >> submitting it.
> > I could have send the patch to wine-devel first but would I get something
> > longer that "several issues" as feedback?
> No, you're missing the point. Why do you expect me to even look at a
> patch if you clearly didn't even check for obvious typos yourself?
> Bottom line, if you want me to spend time reviewing a patch and giving
> feedback, make sure it's worth that time.
Are we really discussing about a typo in the subject line? "Implement" just
comes naturally from the fixme when I create an empty patch. It turned out
after coding that "Fix" is more appropriate. I forgot to updated it. This
will be fix.
According to the feedback I got, altough the patch is not correct, it
doesn't seem it does not worth looking at. And now there is materials to
make it better.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the wine-devel