2013/6/25 Henri Verbeet <hverb...@gmail.com>

> On 25 June 2013 11:05, Christian Costa <titan.co...@gmail.com> wrote:
> > 2013/6/25 Henri Verbeet <hverb...@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.


Reply via email to