Interface suggestion about cl-gobject-introspection

[Kalimehtar]

I would save current API. such as

(defmacro call (&rest args)
   (if (listp (car args))
       `(gcall (nget ,@(car args)) ,@cdr args)
       `(nget ,@args)))

[andy128k]

IMHO gcall sounds ugly. What about "invoke"?

[hying-caritas]

I am not good at naming, because I am not good at English. I will accept your naming suggestion. What about "nget"? Do you think it is a good name?

[andy128k]

nget is supposed to return wrapper for an introspection object, which is derived from GIBaseInfo. What about "info"?

[andy128k]

Since we discuss interface changes, I would propose to add "property", "(setf property)", "field", "(setf field)" similar to "gcall".

Exclamation mark in setter is nice but too schemish :)

What do you think?

[andy128k]

@Kalimehtar Should we really keep current API? We could move such code to extra package (gir-compat).

[hying-caritas]

On Tue, Apr 15, 2014 at 7:18 PM, Andrey Kutejko [email protected]:

nget is supposed to return wrapper for an introspection object, which is
derived from GIBaseInfo. What about "info"?

Sometimes not wrapper for an introspection object. For example

(nget gtk "WindowType" :toplevel)

Will return a fixnum.

[hying-caritas]

Since we discuss interface changes, I would propose to add "property", "(setf property)", "field", "(setf field)" similar to "gcall".

Exclamation mark in setter is nice but too schemish :)

What do you think?

Sounds good for me.

[Kalimehtar]

Should we really keep current API

I'm not so about compat, but about omit extra parens.
There should be handy API for user. (nget ...)-style is good for higher order functions. But for me

window.add(gtk.label.new("ok"))

should be

(call window 'add (call gtk 'label 'new "ok"))

or even (we may make a macro, that quote name for method or class)

(@ window add (@ gtk label new "ok"))

and should not be

(call (nget window add) (call (nget gtk label new) "ok"))

[hying-caritas]

and should not be

(call (nget window add) (call (nget gtk label new) "ok"))

We can make this a little better:

(gcall (window add) (gcall (gtk label new) "ok")

But it is still 2 more pairs parens than your version.

[andy128k]

Just for inspiration: http://clojuredocs.org/clojure_core/clojure.core/-%3E http://clojuredocs.org/clojure_core/clojure.core/-%3E%3E

[hying-caritas]

For me, -> is a good replacement for "nget", especially I have seen it used in similar way in C/C++/perl.

[andy128k]

-> is (can be) more general than nget.

(-> *gtk*
    "Button"
    (invoke 'new 0))

[Kalimehtar]

-> is (can be) more general than nget

And cannot be used in mapcar, for example. -> is a good macro, but bad base API.

[hying-caritas]

I think we can start with the stuff we have consensus firstly. That is, Add "nget", "gcall", and keep "call" at least for now. We can add "->" in the future based on "nget" and "gcall".

As for naming, the formal name of "gcall" will be "invoke", but I don't think "info" is good replacement for "nget", any suggestion?

[Kalimehtar]

any suggestion

I'm agree with it. So add "nget" and "invoke" and replace "call" with a macro.
This way we will have an opportunity to change internal presentation for repository and objects from lambdas to structs.

[andy128k]

I personally don't like than nget has special case for enum values. Enum value is a GIValueInfo. This fits to "info".

One more thing. I am not sure somebody needs raw integer value for enum/bitfield.
But "invoke" can take both GIValueInfo and fixnum.

[hying-caritas]

One more thing. I am not sure somebody needs raw integer value for enum/bitfield.

I think raw integer is better for readability. And we finally need the raw integer value to fill in function argument.

And not only the enum, we need raw integer and string for const too.

[hying-caritas]

On Tue, Apr 15, 2014 at 5:01 PM, Kalimehtar [email protected]:

I would save current API. such as

(defmacro call (&rest args)
(if (listp (car args))
(gcall (nget ,@(car args)) ,@cdr args) (nget ,@Args)))

It appears that this will not keep original "call" syntax exactly.

Originally, it is:

(call gtk "Window" 'new 0)

With your macro, it will be:

(call (gtk "Window" 'new) 0)

I want to double check is this what you want? Because if appears to keep
originally "call", we need to convert everything except function to struct
firstly.

[Kalimehtar]

Oh. I failed.
Then drop "call". New API is better.

[hying-caritas]

OK. And I will keep using "nget" until we find a better name.