<div dir="ltr">Hi Eddy, <div><br></div><div>thanks a lot for looking into this for me. Searching for the contributors to the affected files is a smart idea indeed! </div><div><br></div><div>Cheers</div><div>Dmitriy</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 14, 2020 at 11:24 AM Edward Welbourne <<a href="mailto:edward.welbourne@qt.io">edward.welbourne@qt.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Dmitriy Purgin (14 January 2020 10:10) wrote:<br>
> I'm afraid I did something incorrectly when submitting my first<br>
> contribution to the Qt project and I would love to get a feedback if I<br>
> did it wrong.<br>
[snip]<br>
> Is there something I've missed? Should I have created a ticket in Jira<br>
> first? Or should I have targeted the 5.15 branch?<br>
<br>
Sounds like you followed the process perfectly.<br>
<br>
The one addition I would suggest is to do a git log on the affected<br>
files, to see who might know more about them.  Unfortunately the<br>
overwhelming majority of changes there are routine tidy-ups being done<br>
across many files, that just happen to include these.  I did find one<br>
contributor who'd done substantive work specific to this code, so I've<br>
added him as a reviewer.<br>
<br>
> I know that reviewers have a lot of other things to do and I don't<br>
> want to put pressure or something, I'm just wondering how long the<br>
> review process normally takes, and should I be already worried or not.<br>
<br>
Nothing to worry about.  We're all a bit busy, what with Qt 6 being this<br>
year, so review may take a while.  Patience (and sporadic pings on the<br>
review) should suffice.<br>
<br>
> [1] <a href="https://codereview.qt-project.org/c/qt/qtbase/+/285825" rel="noreferrer" target="_blank">https://codereview.qt-project.org/c/qt/qtbase/+/285825</a><br>
<br>
Good luck and thank you for your contribution,<br>
<br>
        Eddy.<br>
</blockquote></div>