After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 87513 - PORTING not very clear about new slot() behaviour
PORTING not very clear about new slot() behaviour
Status: RESOLVED FIXED
Product: gtkmm
Classification: Bindings
Component: reference documentation
2.0
Other Linux
: Normal minor
: ---
Assigned To: gtkmm-forge
gtkmm-forge
Depends on:
Blocks:
 
 
Reported: 2002-07-06 18:33 UTC by Morten Brix Pedersen
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to PORTING file (880 bytes, patch)
2002-07-06 18:57 UTC, Morten Brix Pedersen
none Details | Review
Final patch. (2.25 KB, patch)
2002-07-07 01:34 UTC, Morten Brix Pedersen
none Details | Review

Description Morten Brix Pedersen 2002-07-06 18:33:20 UTC
I have begun porting my application to gtkmm2, and looked in the PORTING
file first, however, the description for the sigc++ changes is very vague:

 11 * Use new signal connection syntax:
 12    e.g. change
 13    button.clicked.connect(SigC::slot(this
 14    to
 15    button.signal_clicked().connect(SigC::slot(*this

When I first read it, I saw that clicked. was changed to signal_clicked()
and thought that was it. I didn't notice the end of the line where it
stated this instead of *this.

It would be nice if this was clarified further, maybe add an extra item to
the PORTING file with an example.

It's almost impossible to understand the compiler error message you get
when just using 'this'.
Comment 1 Murray Cumming 2002-07-06 18:44:58 UTC
So please provide a patch.
Comment 2 Morten Brix Pedersen 2002-07-06 18:57:52 UTC
Created attachment 9684 [details] [review]
Patch to PORTING file
Comment 3 Morten Brix Pedersen 2002-07-07 01:31:15 UTC
I sinned and forgot to patch the ChangeLog, sorry :-)

Meanwhile I also found that the removal of set_policy() wasnt
mentioned, so heres an updated PORTING and ChangeLog patch.
Comment 4 Morten Brix Pedersen 2002-07-07 01:34:24 UTC
Created attachment 9688 [details] [review]
Final patch.
Comment 5 Murray Cumming 2002-07-07 11:06:21 UTC
Perfect. Thanks.