GNOME.org
 

[gtkmm] Gtkmm-forge digest, Vol 1 #690 - 8 msgs

Send Gtkmm-forge mailing list submissions to
	gtkmm-forge lists sourceforge net

To subscribe or unsubscribe via the World Wide Web, visit
	https://lists.sourceforge.net/lists/listinfo/gtkmm-forge
or, via email, send a message with subject or body 'help' to
	gtkmm-forge-request lists sourceforge net

You can reach the person managing the list at
	gtkmm-forge-admin lists sourceforge net

When replying, please edit your Subject line so it is more specific
than "Re: Contents of Gtkmm-forge digest..."


gtkmm-forge is the mailing list that receives gtkmm bug reports from bugzilla.  A daily digest is sent to gtkmm-main, to encourage people to help fixing the bugs. Do not try to unsubscribe gtkmm-forge from gtkmm-list.


Today's Topics:

   1. [Bug 142758] New:  - Add Gtk::ComboBoxEntry::get_entry() (bugzilla-daemon bugzilla gnome org)
   2. [Bug 142758]  - Add Gtk::ComboBoxEntry::get_entry() (bugzilla-daemon bugzilla gnome org)
   3. [Bug 142849] New:  - Replace AlignmentEnum by float constants (bugzilla-daemon bugzilla gnome org)
   4. [Bug 142849]  - Replace AlignmentEnum by float constants (bugzilla-daemon bugzilla gnome org)
   5. [Bug 109966]  - Dispatcher does not work on win32 (bugzilla-daemon bugzilla gnome org)
   6. [Bug 109966]  - Dispatcher does not work on win32 (bugzilla-daemon bugzilla gnome org)
   7. [Bug 142849]  - Replace AlignmentEnum by float constants (bugzilla-daemon bugzilla gnome org)
   8. [Bug 142849]  - Replace AlignmentEnum by float constants (bugzilla-daemon bugzilla gnome org)

--__--__--

Message: 1
From: bugzilla-daemon bugzilla gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Tue, 18 May 2004 22:09:48 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 142758] New:  - Add Gtk::ComboBoxEntry::get_entry()

http://bugzilla.gnome.org/show_bug.cgi?id=142758
gtkmm | general | Ver: unspecified

           Summary: Add Gtk::ComboBoxEntry::get_entry()
           Product: gtkmm
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: UNCONFIRMED
          Severity: enhancement
          Priority: Normal
         Component: general
        AssignedTo: gtkmm-forge lists sourceforge net
        ReportedBy: daniel elstner gmx net


I think it'd be nice to add a get_entry() method to Gtk::ComboBoxEntry in order
to avoid the need to dynamic_cast<> the result of Gtk::Bin::get_child().  This
would probably boil down to:

    _WRAP_METHOD(Entry* get_entry(), gtk_bin_get_child)

Not terribly important, though.

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


--__--__--

Message: 2
From: bugzilla-daemon bugzilla gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Tue, 18 May 2004 22:10:28 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 142758]  - Add Gtk::ComboBoxEntry::get_entry()

http://bugzilla.gnome.org/show_bug.cgi?id=142758
gtkmm | general | Ver: unspecified

daniel elstner gmx net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
      everconfirmed|0                           |1
           Keywords|                            |API



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


--__--__--

Message: 3
From: bugzilla-daemon bugzilla gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Thu, 20 May 2004 12:04:39 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 142849] New:  - Replace AlignmentEnum by float constants

http://bugzilla.gnome.org/show_bug.cgi?id=142849
gtkmm | general | Ver: unspecified

           Summary: Replace AlignmentEnum by float constants
           Product: gtkmm
           Version: unspecified
          Platform: Other
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: general
        AssignedTo: gtkmm-forge lists sourceforge net
        ReportedBy: daniel elstner gmx net


Currently, every single method or constructor which takes an alignment argument
has to be overloaded in order to support the symbolic constants defined by
Gtk::AlignmentEnum.  Just one example:

    void Gtk::Entry::set_alignment(float xalign);
    void Gtk::Entry::set_alignment(AlignmentEnum xalign);

This approach introduces a couple of problems:

a) overloading introduces ambiguities (this particularly problematic with
constructors)
b) overloads make function pointers hard to use
c) there's the risk of forgetting to add an overload
d) methods that take n float arguments would need 2^n overloads in order to
support all possible variants
e) unnecessary bloat (consider link time, symbol resolving on startup, etc.)

I propose to get rid of all these overloads at the next API/ABI break, and to
replace Gtk::AlignmentEnum by float constants:

    namespace Gtk
    {
      const float ALIGN_LEFT   = 0.0;
      const float ALIGN_CENTER = 0.5;
      const float ALIGN_RIGHT  = 1.0;
      const float ALIGN_TOP    = 0.0;
      const float ALIGN_BOTTOM = 1.0;
    }

The only disadvantage would be that the existence of these constants is not
obvious from the method's prototype.  I think this could be solved by adding
some magic to gmmproc to generate appropriate hints in the documentation.

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


--__--__--

Message: 4
From: bugzilla-daemon bugzilla gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Thu, 20 May 2004 12:05:22 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 142849]  - Replace AlignmentEnum by float constants

http://bugzilla.gnome.org/show_bug.cgi?id=142849
gtkmm | general | Ver: unspecified

daniel elstner gmx net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
      everconfirmed|0                           |1
           Keywords|                            |API



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


--__--__--

Message: 5
From: bugzilla-daemon bugzilla gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Thu, 20 May 2004 15:33:58 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 109966]  - Dispatcher does not work on win32

http://bugzilla.gnome.org/show_bug.cgi?id=109966
glibmm | threads | Ver: 2.4.x

daniel elstner gmx net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
          Component|general                     |threads
            Product|gtkmm                       |glibmm
         Resolution|FIXED                       |
            Version|2.2                         |2.4.x



------- Additional Comments From daniel elstner gmx net  2004-05-20 15:33 -------
Reopened because parts of the new win32 code look fishy to me.  For instance,
the DispatchNotifyData objects in the queue are both read and written without
any locking at all.  Only locking the queue is not sufficient, you have to lock
the data too.

I think it'd be best to get rid of of dynamic memory allocation altogether --
there's simply no need for this.  Further, there is absolutely no need for
Locked_Queue<> to be a template, and neither should all methods be inlined.

Other issues: DispatchNotifier::create_pipe() calls g_win32_error_message()
whereas warn_failed_pipe_io() uses g_strerror(), so I guess the latter is
incorrect.  Also, the win32 code should pass the name of the function which
returned the error to warn_failed_pipe_io().

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


--__--__--

Message: 6
From: bugzilla-daemon bugzilla gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Thu, 20 May 2004 16:53:59 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 109966]  - Dispatcher does not work on win32

http://bugzilla.gnome.org/show_bug.cgi?id=109966
glibmm | threads | Ver: 2.4.x





------- Additional Comments From daniel elstner gmx net  2004-05-20 16:53 -------
Created an attachment (id=27890)
 --> (http://bugzilla.gnome.org/attachment.cgi?id=27890&action=view)
fixed-up patch

It would be nice if someone could try the new patch on Win32.  I hope it
compiles cleanly; right now I have now way to test glibmm on Win32.

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


--__--__--

Message: 7
From: bugzilla-daemon bugzilla gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Fri, 21 May 2004 14:06:52 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 142849]  - Replace AlignmentEnum by float constants

http://bugzilla.gnome.org/show_bug.cgi?id=142849
gtkmm | general | Ver: unspecified





------- Additional Comments From murrayc murrayc com  2004-05-21 14:06 -------
For the documentation, we could just use a typedef instead of a float, and
document that typedef.

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


--__--__--

Message: 8
From: bugzilla-daemon bugzilla gnome org
To: gtkmm-forge lists sourceforge net
Cc: 
Date: Fri, 21 May 2004 14:46:27 -0400 (EDT)
Subject: [gtkmm bugzilla] [Bug 142849]  - Replace AlignmentEnum by float constants

http://bugzilla.gnome.org/show_bug.cgi?id=142849
gtkmm | general | Ver: unspecified





------- Additional Comments From daniel elstner gmx net  2004-05-21 14:46 -------
Sounds good.

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.



--__--__--

_______________________________________________
Gtkmm-forge mailing list
Gtkmm-forge lists sourceforge net
https://lists.sourceforge.net/lists/listinfo/gtkmm-forge


End of Gtkmm-forge Digest
[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]