Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Sep 28 03:09
    Abastro commented #383
  • Sep 28 03:08
    Abastro commented #383
  • Sep 28 03:07
    Abastro commented #383
  • Sep 27 18:42
    garetxe commented #383
  • Sep 27 18:38
    garetxe commented #388
  • Sep 27 18:22
    garetxe commented #389
  • Sep 26 10:50
    felixonmars closed #389
  • Sep 26 10:50
    felixonmars commented #389
  • Sep 26 07:55
    felixonmars opened #389
  • Sep 14 06:46
    Abastro edited #388
  • Sep 13 07:36
    Abastro opened #388
  • Sep 13 05:09
    Abastro closed #383
  • Aug 08 01:07
    rpglover64 commented #369
  • Aug 07 09:27

    garetxe on master

    Update version of Ubuntu for gi… (compare)

  • Aug 07 09:26

    garetxe on master

    Turn cabal.project into a symli… (compare)

  • Aug 07 09:10
    garetxe commented #387
  • Aug 07 09:01

    garetxe on master

    Turn cabal.project into a symli… (compare)

  • Aug 07 09:00

    garetxe on master

    Add Freetype2 bindings, require… Add bindings for GStreamer Edit… (compare)

  • Aug 07 05:15

    garetxe on master

    Add support for GValue elements… (compare)

  • Aug 07 00:15
    TheMBL09898 commented #387
Tisoxin
@vherrmann:shmerver.de
[m]
    • Overlapping instances for GI.Constructible a0 tag0
        arising from a use of ‘GI.new’
      Matching instances:
        instance [overlappable] (GI.GObject a, tag ~ 'GI.AttrConstruct) =>
                                GI.Constructible a tag
          -- Defined in ‘Data.GI.Base.Constructible’
        instance (tag ~ 'GI.AttrSet) =>
                 GI.Constructible G.AccelGroupEntry tag
          -- Defined in ‘GI.Gtk.Structs.AccelGroupEntry’
        instance (tag ~ 'GI.AttrSet) => GI.Constructible G.AccelKey tag
          -- Defined in ‘GI.Gtk.Structs.AccelKey’
        ...plus 35 others
        ...plus 95 instances involving out-of-scope types
        (use -fprint-potential-instances to see them all)
      (The choice depends on the instantiation of ‘a0, tag0’
       To pick the first instance above, use IncoherentInstances
       when compiling the other instance declarations)
    • In the expression: GI.new
      In an equation for ‘myNew’: myNew = GI.new
   |
97 | myNew = G.new
   |         ^^^^^
in this case GI is Data.GI.Base.ShortPrelude and G is GI.GTK
Tisoxin
@vherrmann:shmerver.de
[m]
nvm, i haven't yet found a real solution, but i think i'm able to figure it out myself
Tisoxin
@vherrmann:shmerver.de
[m]
:]
Iñaki
@garetxe
@vherrmann:shmerver.de Great, glad to hear that you figured it out! (And sorry for the slow reply, I have been very busy the last few days)
Please don't hesitate to ask if you run into other issues, I'll try to keep a closer eye on this
Iñaki
@garetxe
(Implementing a reflex-type binding on top of gi-gtk sounds very interesting, by the way! Looking forward to the result.)
Iñaki
@garetxe
Hi! No, in general there is no need to worry about unref'ing memory manually for any type with a GObject or GBoxed instance, the bindings will take care of freeing the memory when the Haskell object gets garbage collected.
Miguel Negrão
@miguel-negrao
@garetxe Thanks ! That makes it quite a bit easier to use than the C api.
Miguel Negrão
@miguel-negrao
I'm getting a SIGSEGV with gi-gtk. The backtrace is at https://gist.github.com/miguel-negrao/ba7990d368bd7c886ecd2fdef2ef5842
To check that I'm not doing anything obviously wrong, I'm creating a widget with new and adding it to a container with boxPackStart box widget expand fill padding
I'm then destroying the widget with widgetDestroy which appears to trigger the crash.
Miguel Negrão
@miguel-negrao
The issue only happens with Switch
Iñaki
@garetxe

I'm then destroying the widget with widgetDestroy which appears to trigger the crash.

This sounds a bit like you are double freeing. Is it necessary to call widgetDestroy in your code? Every widget in Gtk is reference counted. In particular if you create a widget and add it to a box then it will have a refcount of 2: Haskell keeps one (which it will drop once the Haskell object is garbage collected) and the container keeps another.

So basically you never need to care about freeing things manually.

(By the way, when debugging crashes it is useful to run with the following environment vars set:

export G_SLICE="debug-blocks"
export MALLOC_CHECK_=2
export MALLOC_PERTURB_=$(($RANDOM % 255 + 1))
export HASKELL_GI_DEBUG_MEM=1

)

Iñaki
@garetxe

And another question: how would I replicate https://hackage.haskell.org/package/gi-gtk-4.0.5/docs/GI-Gtk-Objects-Scale.html#v:scaleNew with https://hackage.haskell.org/package/haskell-gi-base-0.26.0/docs/Data-GI-Base-Constructible.html#v:new , since it has some additional parameters ?

In general it should be perfectly fine to call scaleNew, but in this case it might be possible to make it work with new too. Have you tried setting the orientation and adjustment attributes? Something like

scale <- new Gtk.Scale [#orientation := ..., #adjustment := ...]
Miguel Negrão
@miguel-negrao
This is happening in the context of code, where I'm trying to have a more declarative approach to adding and removing widgets to a container. I use HList since widgets can have different types. The "map" type functions for hlist are these:
data HPackBox = HPackBox
instance (IsDescendantOf Box a, IsDescendantOf Widget b,
    Constructible b tag, MonadIO m, GObject a, GObject b,
    input ~ ((a, Bool, Bool, Word32), (ManagedPtr b -> b, [AttrOp b tag])), output ~ m b) =>
    ApplyAB HPackBox input output where
        applyAB _ ((box, expand, fill, padding), (ctr, attrs)) = do
            widget <- new ctr attrs
            boxPackStart box widget expand fill padding
            return widget

data HDestroyWidget = HDestroyWidget
instance (IsWidget o) => ApplyAB HDestroyWidget o (IO ()) where
    applyAB _ widget = widgetDestroy widget
Then I pass in the list of widgets to add:
(Button, [#label := ("Hello World 1"::Text)]) .*.
(Button, [#label := ("Hello World 1"::Text)]) .*.
(Button, [#label := ("Hello World 1"::Text)]) .*.
(Button, [#label := ("Hello World 1"::Text)]) .*.
(Switch, [])  .*. -- crashes ?
(Scale, [])  .*.
HNil)
So in order to have this interface be universal I would need to be able to properly construct any widget with new. So my question is if every newX method can be replicated with new and attributes.
Miguel Negrão
@miguel-negrao
Regarding using widgetDestroyI just need to remove the widget from the container, I can't find the source but I read somewhere that widgetDestroy is ok to remove from a container. I will try with containerRemove.
And also, thanks a lot for the help !!!
Miguel Negrão
@miguel-negrao
containerRemove doesn't cause the crash.
Iñaki
@garetxe

So my question is if every newX method can be replicated with new and attributes.

Ah, I see, thanks for the explanation. (Cool approach!) I hesitate to say that all widgets can be created in this way (I cannot think of an exception now, but I'm not sure), but this certainly seems to be the direction of travel.

containerRemove doesn't cause the crash.

Yes, this is definitely safer. By doing this you are telling the container to drop its reference to the widget, but you are not invalidating it, so the widget is not becoming invalid "behind Haskell's back", so to speak

Miguel Negrão
@miguel-negrao

So my question is if every newX method can be replicated with new and attributes.

Ah, I see, thanks for the explanation. (Cool approach!) I hesitate to say that all widgets can be created in this way (I cannot think of an exception now, but I'm not sure), but this certainly seems to be the direction of travel.

I tried the following and it worked !

    adjustment <- new Adjustment [#value := 0.0, #lower := 0.0, #upper := 51, #stepIncrement := 0.1, #pageIncrement := 1.0, #pageSize := 1.0]
    scale <- new Scale [ #orientation := OrientationVertical, #adjustment := adjustment]

So that means that every "xNew" method has corresponding attributes that are generated automatically with the same name as the name given to the function "xNew"'s arguments in the documentation ?

Iñaki
@garetxe

I tried the following and it worked !

Great :)

So that means that every "xNew" method has corresponding attributes that are generated automatically with the same name as the name given to the function "xNew"'s arguments in the documentation ?

It is not automatic, unfortunately, the writer of the C code has to make sure that the relevant arguments are exposes as GObject properties. But in practice, in Gtk at least, it seems to almost always be the case, and all the different constructors are just convenience wrappers over g_object_new, setting the properties. Something like what you see in gtk_button_new_with_label.

This is equivalent to calling new with attributes in haskell-gi.
tripa
@tripa:matrix.org
[m]
Ha ha I came with a similar question about new equivalence
I was missing gtk_message_dialog_new from the bindings, hacked around with a few properties in the generic new list and a separate #setTransientFor (that took me the longest to identify)
And I was wondering if it would give the same results
(…and that actually doesn't need to be separate, it just was because it took so long to pin down)
Iñaki
@garetxe
Yes, that should do it. If you want to check what gtk_message_dialog_new is doing internally, you can check here.
(The reason that gtk_message_dialog_new is not in the bindings is because it accepts a variable number of arguments, and we don't bind such functions.)
tripa
@tripa:matrix.org
[m]
Yes, source diving was how it ended for me (and how I identified the final setTransientFor)
Good to know for the missing binding, I was wondering
Perhaps a stub could be generated in Haddock to explain? I tried looking around for an explanation, but it's hard to know where to look
Iñaki
@garetxe
Good suggestion, thanks! Could you please file an issue so I don't forget about this? I was planning to do some work on the documentation generator (the gtk-doc format seems to have changed, so the haddocks don't look great right now), this might be a good chance.
If you have some ideas about the language please feel free to suggest them in the issue, I am not sure what the right approach is here.
tripa
@tripa:matrix.org
[m]
Will do!
Iñaki
@garetxe
Great, thanks!
tripa
@tripa:matrix.org
[m]
GObject.objectGetData documentation says it returns “the data if found, or Nothing if no such data exists”
But its signature is * -> MonadIO m => m (Ptr ())
After much head-scratching, I'm guessing it really returns nullPtr if no data exists?
Iñaki
@garetxe
@tripa:matrix.org Yes, that's right! The gtk-doc -> haddock translator uses a number of heuristics to improve the Haskell side docs, one of which is replacing NULL by Nothing. This is almost always the right thing to do, since nullable arguments on the C side are mapped to Maybe arguments. But this is indeed wrong for objectGetData.
eldritch cookie
@dark-ether
how do i get started? i tried to follow the gtk4 guide but there isn't a function called applicationRun
Iñaki
@garetxe
@dark-ether There is an example of how to set up a gtk4 application at https://github.com/haskell-gi/haskell-gi/tree/master/examples/Gtk4
Iñaki
@garetxe
You'll need ghc version 9.2.1 or newer to compile the adwaita example. If you don't want to or cannot use that version, remove the OverloadedRecordDot extension and change all method calls of the form obj.method to #method obj
eldritch cookie
@dark-ether
what does a expression like [#transitionType := Gtk.StackTransitionTypeUnderUp]
mean ? i tried searching but it still is unclear to me