[PATCH] wined3d: Implement per-stage constant in glsl fixed fonction pipeline.

Christian Costa 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...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20130625/5a265248/attachment.html>


More information about the wine-devel mailing list