<div dir="ltr"><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Sun, Sep 16, 2018 at 11:51 PM Uli Schlachter <<a href="mailto:psychon@znc.in">psychon@znc.in</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">On 17.09.2018 00:36, Thomas Fischer wrote:<br>
> On Sun, Sep 16, 2018 at 9:33 AM Uli Schlachter <<a href="mailto:psychon@znc.in" target="_blank">psychon@znc.in</a>> wrote:<br>
> <br>
>> Hi Thomas,<br>
>><br>
>> sorry for being awfully slow.<br>
>><br>
>> On 26.08.2018 07:14, Thomas Fischer wrote:<br>
>> [...]> diff --git a/src/xproto.xml b/src/xproto.xml<br>
>>> index 9624700..f78f339 100644<br>
>>> --- a/src/xproto.xml<br>
>>> +++ b/src/xproto.xml<br>
>>> @@ -1828,6 +1828,10 @@ <xcb/xcb.h><br>
>>>      xcb_screen_iterator_t   iter   = xcb_setup_roots_iterator (setup);<br>
>>>      xcb_screen_t           *screen = iter.data;<br>
>>><br>
>>> +    uint32_t value_mask = XCB_CW_BACK_PIXEL | XCB_CW_EVENT_MASK;<br>
>>> +    uint32_t value_list[] = { screen->black_pixel,<br>
>>> +    XCB_EVENT_MASK_KEY_RELEASE | XCB_EVENT_MASK_EXPOSE<br>
>>> +    /* | XCB_EVENT_MASK_* ... */ };<br>
>> [...]<br>
>><br>
>> What did you base your patch on? The string "xcb_setup_roots_iterator"<br>
>> does not appear in the current(?) version of xproto.xml. Thus, your<br>
>> patch does not apply here.<br>
>><br>
>> Cheers,<br>
>> Uli<br>
>><br>
> <br>
> Hello, it's no problem!<br>
> <br>
> The example was based on the tutorial here:<br>
> <a href="https://xcb.freedesktop.org/tutorial/basicwindowsanddrawing/" rel="noreferrer" target="_blank">https://xcb.freedesktop.org/tutorial/basicwindowsanddrawing/</a><br>
> <br>
> The "xcb_setup_roots_iterator" function is generated in<br>
> libxcb/src/xproto.{h,c}<br>
<br>
Sorry if I was not clear.<br>
<br>
My problem is not the content of your patch, but that the patch does not<br>
apply. Your patch touches code next to an example with<br>
"xcb_setup_roots_iterator", but there is no such example in the current<br>
version of xproto.xml.<br></blockquote><div><br></div><div>Sorry, I misunderstood there! It was late and I should have read the words "based" and "apply" from a git perspective.</div><div><br></div><div><div>> I have to first apply your patch "Add documentation" from beginning of</div><div>> August and only afterwards can "Add more documentation" apply. I totally</div><div>> forgot about that first patch (or assumed that the new patch replaces</div><div>> the old one). Sorry about that.</div></div><div><br></div><div>I'm still slightly new to open source development -- in the future I'll make sure to git commit --amend to prevent confusion.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
P.S.: And the commit messages could be improved, but I can just reword<br>
that while applying the patches.<br></blockquote><div><br></div><div>I agree, should I have named it something like this?</div><div>"xproto: Add createWindow documentation and example"</div><div><br></div><div>Thanks for being patient. How does the documentation look? I'd say 70% is from the X11 standard, the rest paraphrasing the standard or the Xlib manuals. </div><div>If there's something I should change please let me know.</div><div><br></div><div>Thanks,</div><div>Thomas</div><div> </div></div></div></div>