[mythtv] first contribution
hendrixski
hendrixski at storsint.com
Thu Sep 20 15:05:15 UTC 2007
> Thanks for your patch. Feel free to do some more!
> (i.e. understanding & documenting all
> of the other nasty classes :-)
>
While I'm probably not going to submit any more comment-only patches,
I'll definitely include comments into the patches I submit. Hope that
will help out in keeping the source readable long-term. :-)
>> I guess the last step is feedback?
>>
> Daniel has grabbed the ticket, so he may give you some
> before he commits it, but here are my trivial comments:
>
Cool, thanks. Now generally does the dev who grabs the ticket make some
small changes to the patch themselves before committing, or do they send
it back to the patch submitter with comments and wait for a fix before
committing? If I were to fix those things now what is "best
practices": just replace the file on the bugtracker, or post a new one
with a note like "XY and Z fixed"?
It's good I'm going through this kind of stuff with a trivial patch
rather than something with lots of lines of code. I guess I missed the
purpose @brief, and didn't know to catch those whitespace slips.
Thanks again.
More information about the mythtv-dev
mailing list