[PySide] Submitting PySide patches (Matthew Woehlke)

Johnny Casey jjcasey at pobox.com
Sat Jun 22 17:14:10 CEST 2013


Looking at 
Change I99f2d3c6: generator/shiboken/cppgenerator.cpp
Lines 1717, 1751 don't look like white space changes.  They change the actual strings from
since SBK_STR_NAME isn't a symbol, but rather the string contents.



>Message: 7
>Date: Fri, 21 Jun 2013 13:30:12 -0400
>From: Matthew Woehlke <matthew.woehlke at kitware.com>
>Subject: Re: [PySide] Submitting PySide patches
>To: pyside at qt-project.org
>Message-ID: <kq22iu$5pd$1 at ger.gmane.org>
>Content-Type: text/plain; charset=UTF-8; format=flowed
>On 2013-06-21 13:13, John Cummings wrote:
>>>>> Are there not default reviewers (other than the 'bot)?
>>>> Can Gerrit be set up to add default reviewers?
>>> Good question. Maybe not in the way you can with RB, but we generally
>>> have interested parties set to receive mail related to specific
>>> projects, which is about the same difference. (So I guess what I'm
>>> really asking is if you and/or other maintainers get mail for new
>>> requests without having to be specifically added?)
>Apparently *users* can do this with git receive-pack?.
>> I get a code review email when my name gets added as a reviewer for a specific
>> patch. I do not believe receive an email otherwise. In any case, you should be
>> able to ask for either of us John's to review the patch.
>Have you (and/or John E.) considered subscribing to pyside and/or 
>shiboken? Having the maintainers subscribed to get mail for all requests 
>for those projects is probably not a bad idea, especially as us 
>neophytes are likely to not assign reviewers otherwise (due to any 
>combination of ignorance, inexperience, and/or forgetfulness).
>> Right now I'm working my way through a series of patches that John Ehresman
>> submitted for my review. So, feel free to add another one. Yours can't possibly
>> get into as many low-level aspects of PyObject as some of John's patches have. ;-)
>Definitely not :-). https://codereview.qt-project.org/58889 and 
>https://codereview.qt-project.org/58890 are both whitespace-only. All 
>they do is add spaces between string literals and identifier-like 
>tokens, which are required in C++11 to prevent the compiler thinking you 
>are using special literals.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.qt-project.org/pipermail/pyside/attachments/20130622/bd0e0051/attachment.html>

More information about the PySide mailing list