Gtk::Adjustment ignores initial value in constructor

Bug #69806 reported by Vincent Levesque
8
Affects Status Importance Assigned to Milestone
gtkmm
Fix Released
Medium
gtkmm2.4 (Ubuntu)
Fix Released
Medium
Ubuntu Desktop Bugs
Edgy
Won't Fix
Low
Unassigned

Bug Description

Binary package hint: libgtkmm-2.4-dev

Gtk::Adjustment seems to ignore the initial value it is given in its constructor. In the attached example, an adjustment is initialized to 1 and used with a SpinButton. When you run it though, the SpinButton has a value of 0 and get_value() returns 0. Using set_value() in the Window's constructor, on the other hand, correctly sets the initial value.

I was able to reproduce this bug on two computers running Ubuntu 6.10 but not on another one running Ubuntu 6.06. Using older versions of GCC doesn't seem to help.

See attached sources files.

Revision history for this message
Vincent Levesque (vleves) wrote :
Revision history for this message
Vincent Levesque (vleves) wrote :
description: updated
Revision history for this message
Daniel Holbach (dholbach) wrote :

Thanks for your bug report. This looks valid. Murray, do you want me to forward it upstream?

On a brief glance:

Code in gtkmm 2.8.5:
Adjustment::Adjustment(double value,
                       double lower, double upper,
                       double step_increment, double page_increment,
                       double page_size)
:
  Glib::ObjectBase(0), //Mark this class as gtkmmproc-generated, rather than a custom class, to allow vfunc optimisations.
  Gtk::Object(Glib::ConstructParams(adjustment_class_.init(), (char*) 0))
{
  gobj()->lower = lower;
  gobj()->upper = upper;
  gobj()->step_increment = step_increment;
  gobj()->page_increment = page_increment;
  gobj()->page_size = page_size;
  changed();

  set_value(value);
}

void Adjustment::set_value(double value)
{
    gtk_adjustment_set_value(gobj(), value);

}

(For some reason set_value() doesn't use
  gobj()->value = value;
  changed();
as the other set_*() do. But maybe that's a red herring.)

----

Code in gtkmm 2.10.2:
Adjustment::Adjustment(double value, double lower, double upper, double step_increment, double page_increment, double page_si
ze)
:
  Glib::ObjectBase(0), //Mark this class as gtkmmproc-generated, rather than a custom class, to allow vfunc optimisations.
  Gtk::Object(Glib::ConstructParams(adjustment_class_.init(), "value", value, "lower", lower, "upper", upper, "step_increment
", step_increment, "page_increment", page_increment, "page_size", page_size, (char*) 0))
{
  }

Changed in gtkmm2.4:
assignee: nobody → desktop-bugs
importance: Undecided → Medium
Revision history for this message
Murray Cumming (murrayc) wrote :

There is discussion about this on the mailing list at the moment, with ongoing discussion. It's fine to submit this to the GNOME bugzilla. Thanks.

Revision history for this message
Daniel Holbach (dholbach) wrote :
Changed in gtkmm2.4:
status: Unconfirmed → Confirmed
Changed in gtkmm:
status: Unknown → Unconfirmed
Revision history for this message
Daniel Holbach (dholbach) wrote :
Changed in gtkmm2.4:
status: Confirmed → Fix Committed
Changed in gtkmm:
status: Unconfirmed → Fix Released
Revision history for this message
Burnce (burnce) wrote :

The bug is still present in the most recent binary package for edgy (gtkmm 2.10.2).

It would be nice to have a package for a more recent, fixed version of gtkmm.
Both libglibmm-2.12.3 and lib-gtkmm-2.10.6 run fine when installed from feisty sources on an edgy installation:

$ sudo apt-get -b source -t unstable libglibmm-2.4-dev
$ sudo dpkg -i libglibmm-2.4-1c2a_2.12.3-0ubuntu1_i386.deb
$ sudo dpkg -i libglibmm-2.4-dev_2.12.3-0ubuntu1_i386.deb

$ sudo apt-get -b source -t unstable libgtkmm-2.4-dev
$ sudo dpkg -i libgtkmm-2.4-1c2a_2.10.6-0ubuntu1_i386.deb
$ sudo dpkg -i ;ibgtkmm-2.4-dev_2.10.6-0ubuntu1_i386.deb
$ sudo dpkg -i libgtkmm-2.4-doc_2.10.6-0ubuntu1_all.deb

Cheers,

Burnce

Revision history for this message
Daniel Holbach (dholbach) wrote :

Fixed in Feisty.

Changed in gtkmm2.4:
status: Fix Committed → Fix Released
Revision history for this message
Daniel Holbach (dholbach) wrote :

Confirmed to be a problem. If somebody wants to work on this, https://wiki.ubuntu.com/StableReleaseUpdates is the process for that.

Changed in gtkmm2.4:
status: Unconfirmed → Confirmed
importance: Undecided → Low
Revision history for this message
Hew (hew) wrote :

Ubuntu Edgy Eft is no longer supported, so a SRU will not be issued for this release. Marking Edgy as Won't Fix.

Changed in gtkmm2.4:
status: Confirmed → Won't Fix
Changed in gtkmm:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.