[Kde-bindings] Kde-bindings Digest, Vol 116, Issue 12

Thu Dec 13 21:51:08 UTC 2012

unsubscribe


On 14 December 2012 08:30, <kde-bindings-request at kde.org> wrote:

> Send Kde-bindings mailing list submissions to
>         kde-bindings at kde.org
>
> To subscribe or unsubscribe via the World Wide Web, visit
>         https://mail.kde.org/mailman/listinfo/kde-bindings
> or, via email, send a message with subject or body 'help' to
>         kde-bindings-request at kde.org
>
> You can reach the person managing the list at
>         kde-bindings-owner at kde.org
>
> When replying, please edit your Subject line so it is more specific
> than "Re: Contents of Kde-bindings digest..."
>
> Today's Topics:
>
>    1. Re: Qyoto: Patch v2: Translate obvious getters    into
>       properties or "Get" methods (Crowell, Brian)
>
>
> ---------- Forwarded message ----------
> From: "Crowell, Brian" <BCrowell at barbnet.com>
> To: Dimitar Dobrev <dpldobrev at yahoo.com>, "KDE bindings for other
> programming languages" <kde-bindings at kde.org>
> Cc:
> Date: Thu, 13 Dec 2012 21:30:15 +0000
> Subject: Re: [Kde-bindings] Qyoto: Patch v2: Translate obvious getters
> into properties or "Get" methods
>
> On address: I’m easy, actually, but I haven’t gotten out of the mode of
> “addressing-people-by-their-first-name-is-presumptive-on-my-part.” First
> names it is.****
>
> ** **
>
> On uics: HasHeightForWidth() is part of the “special” code in
> cswriteinitialization.cpp. I’ll try to correct a few things in there. I’m
> going to fix an issue with properties on interfaces before I issue another
> version of the patch. I’m mainly offering it for review and feedback (which
> so far has worked); I’m not sure at what point you’d consider it right to
> commit it.****
>
> ** **
>
> --Brian****
>
> ** **
>
> ** **
>
> ** **
>
> *From:* kde-bindings-bounces at kde.org [mailto:kde-bindings-bounces at kde.org]
> *On Behalf Of *Dimitar Dobrev
> *Sent:* Thursday, December 13, 2012 3:17 PM
> *To:* KDE bindings
> *Subject:* Re: [Kde-bindings] Qyoto: Patch v2: Translate obvious getters
> into properties or "Get" methods****
>
> ** **
>
>
>     This is better, however, there is another problem that had been
> "hidden" (that is, the compiler stopped there) by the Model issue: some of
> the code generated by uics is no longer valid, the "HasHeightForWidth"
> method in particular. Sorry about me not noticing earlier but I'd like to
> ask you to update uics as well. This is the only method I've found so if
> there are others, I'll fix them myself. On the other hand, the properties
> are OK, so I'll push them. Whenever you're ready with uics, I'll push the
> Get_ part too.****
>
>     By the way, you didn't say what you think about the form of address. :)
> ****
>
> ** **
>
>     Dimitar****
>
> ** **
>    ------------------------------
>
> *From:* "Crowell, Brian" <BCrowell at barbnet.com>
> *To:* Dimitar Dobrev <dpldobrev at yahoo.com>; KDE bindings for other
> programming languages <kde-bindings at kde.org>
> *Sent:* Thursday, December 13, 2012 10:47 PM
> *Subject:* RE: [Kde-bindings] Qyoto: Patch v2: Translate obvious getters
> into properties or "Get" methods****
>
> ** **
>
> Version 2 then, with more properties, and fix for the Model issue.****
>
>  ****
>
> I’d be uncomfortable blacklisting methods. If List<QWidget>
> QWidget.Children() remains a method, nobody gets hurt, but if bool
> QMovie.JumpToNextFrame() becomes a property, chaos happens. There is one
> pattern that could be exploited: methods starting with nouns or adjectives
> like “Widget,” “Width,” “Window,” “File,” “Root,” “Physical,” and “Temp”
> are almost certainly properties, but so far only the “File” prefix has more
> than two entries in my list.****
>
>  ****
>
> --Brian****
>
>  ****
>
>  ****
>
> *From:* kde-bindings-bounces at kde.org [mailto:kde-bindings-bounces at kde.org<kde-bindings-bounces at kde.org>]
> *On Behalf Of *Dimitar Dobrev
> *Sent:* Thursday, December 13, 2012 1:15 PM
> *To:* KDE bindings for other programming languages
> *Subject:* Re: [Kde-bindings] Qyoto: Patch v1: Translate obvious getters
> into properties or "Get" methods****
>
>  ****
>
>     ****
>
>     Mr. Crowell,****
>
>     ****
>
>     Thanks for the patch, it looks nice. I'll discuss it in a while.
> First, I'd like to point out that there is no need for us here in open
> source communities to be that formal. :) If you agree with me, you may call
> me "Dimitar" and in turn I would call you "Brian". However, if you do
> prefer an official form of address, that's fine by me.****
>
>     About your patch: it looks very good, and I like how you reused the
> method for generating a property instead of writing your own. :) However, I
> have two remarks, a regression and a suggestion:****
>
>     1. QComboBox.Model used to be a read-write property, now it's read
> only. Could you please change your code so that it remains read-write?
> Also, if possible, check for other such cases because this is simply one I
> found, it doesn't mean there aren't more;****
>
>     2. I'm wondering if it would be better to have something like a list
> of commonly used in API-s verbs, and then if the method starts with has or
> is or is a single word (this can be checked by casing) and this single word
> is not a verb, the method should be generated as a property. The thing is
> that your way you have to look for properties all over the huge Qt API, and
> that with each Qt version (for example, Qt 5 is coming very soon). Of
> course, the verb approach may actually turn out to be worse because there
> may be way too many verbs to enumerate. So this is simply a suggestion for
> you to think about if you like.
>     ****
>
>     Dimitar****
>
>  ****
>     ------------------------------
>
> *From:* "Crowell, Brian" <BCrowell at barbnet.com>
> *To:* KDE bindings for other programming languages <kde-bindings at kde.org>
> *Sent:* Thursday, December 13, 2012 8:23 PM
> *Subject:* [Kde-bindings] Qyoto: Patch v1: Translate obvious getters into
> properties or "Get" methods****
>
>
> As discussed with Mr. Dobrev, this patch finds methods with certain names
> and signatures and translates them into properties. This covers "Is" and
> "Has" properties along with a specified list of noun phrases. Getter
> methods that take arguments are renamed "Get" (Sibling(int row, int column)
> becomes GetSibling(int row, int column)).
>
> The list of property names is incomplete. I'll add more as I find them.
>
> --Brian
> _______________________________________________
> Kde-bindings mailing list
> Kde-bindings at kde.org
> https://mail.kde.org/mailman/listinfo/kde-bindings****
>
> ** **
>
> _______________________________________________
> Kde-bindings mailing list
> Kde-bindings at kde.org
> https://mail.kde.org/mailman/listinfo/kde-bindings
>
>


-- 
Marc Jackson
Agile Developer/DevOps - Unix/Linux/Windoze
Mobile: +61 468 321467
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-bindings/attachments/20121214/09ac9527/attachment.html>