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 547889 - Implementing C++ version of GNode test case for Glib:NodeTree
Implementing C++ version of GNode test case for Glib:NodeTree
Status: RESOLVED FIXED
Product: glibmm
Classification: Bindings
Component: tests
2.17.x
Other Linux
: Normal normal
: ---
Assigned To: gtkmm-forge
gtkmm-forge
Depends on: 546485 547901
Blocks:
 
 
Reported: 2008-08-15 09:45 UTC by Szilárd Pfeiffer
Modified: 2008-08-26 13:43 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
C++ version of GNode's test case (8.57 KB, patch)
2008-08-15 09:49 UTC, Szilárd Pfeiffer 		none Details | Review

Description Szilárd Pfeiffer 2008-08-15 09:45:41 UTC 
The test case of GNode is a little bit sofisticated and useful.
Comment 1 Szilárd Pfeiffer 2008-08-15 09:49:41 UTC 
Created attachment 116646 [details] [review]
C++ version of GNode's test case

ChangeLog:

2008-08-15  Szilárd Pfeiffer  <szilard.pfeiffer@gmail.com>

    * tests/glibmm_nodetree/main.cc: Implemet the C++ version of GNode test
    case.
    Bug #547889
Comment 2 Murray Cumming 2008-08-18 08:47:57 UTC 
I committed that, commenting out the use of NodeTree::copy, which is not yet in the API.

I would also like to see this use more C++-like variable scope, avoiding reuse of the same variables, such as "node".
Comment 3 Szilárd Pfeiffer 2008-08-18 10:47:40 UTC 
(In reply to comment #2)
> I committed that, commenting out the use of NodeTree::copy, which is not yet in
> the API.
> 

I set this bug to the dependent of bug 547901 to avoid this problem. Was it a wrong way?
Comment 4 Murray Cumming 2008-08-19 10:00:02 UTC 
Yeah. I just wanted to get some of it in now because it's useful to test the other things.
Comment 5 Murray Cumming 2008-08-26 13:43:15 UTC 
The whole test case is now uncommented.