[gtkmm/gtkmm-2-22] PrintOperation: get_default_page_setup(): refcount fix. + + * gtk/src/printoperation.hg: get_d



commit e69133e3aa81ccc5f1c0de6e9432b44eab51b1b8
Author: Murray Cumming <murrayc murrayc com>
Date:   Mon Jun 28 08:57:06 2010 +0200

    PrintOperation: get_default_page_setup(): refcount fix.
    +
    +       * gtk/src/printoperation.hg: get_default_page_setup(),
    +    get_print_settings(): Use refreturn, fixing bug #622957 (Kjell Ahlstedt).

 ChangeLog                 |    7 +++++++
 gtk/src/printoperation.hg |   38 +++++++++++++++++++-------------------
 2 files changed, 26 insertions(+), 19 deletions(-)
---
diff --git a/ChangeLog b/ChangeLog
index fb2f5f6..f66f800 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-06-28  Murray Cumming  <murrayc murrayc com>
+
+	PrintOperation: get_default_page_setup(): refcount fix.
+
+	* gtk/src/printoperation.hg: get_default_page_setup(),
+    get_print_settings(): Use refreturn, fixing bug #622957 (Kjell Ahlstedt).
+
 2.21.1:
 
 2010-06-23  Daniel Elstner  <danielk openismus com>
diff --git a/gtk/src/printoperation.hg b/gtk/src/printoperation.hg
index 931e6ae..8ca689f 100644
--- a/gtk/src/printoperation.hg
+++ b/gtk/src/printoperation.hg
@@ -38,18 +38,18 @@ _WRAP_GERROR(PrintError, GtkPrintError, GTK_PRINT_ERROR)
 /** @defgroup Printing Printing
  */
 
-/** PrintOperation is the high-level, portable printing API. It looks a bit different than other 
- * GTK+ dialogs such as the FileChooser, since some platforms don't expose enough infrastructure 
- * to implement a good print dialog. On such platforms, PrintOperation uses the native print 
+/** PrintOperation is the high-level, portable printing API. It looks a bit different than other
+ * GTK+ dialogs such as the FileChooser, since some platforms don't expose enough infrastructure
+ * to implement a good print dialog. On such platforms, PrintOperation uses the native print
  * dialog. On platforms which do not provide a native print dialog, GTK+ uses its own, see PrintUnixDialog.
  *
- * The typical way to use the high-level printing API is to create a PrintOperation object 
- * when the user chooses to print. Then you set some properties on it,such as the page size, any PrintSettings 
+ * The typical way to use the high-level printing API is to create a PrintOperation object
+ * when the user chooses to print. Then you set some properties on it,such as the page size, any PrintSettings
  * from previous print operations, the number of  pages, the current page, etc.
  *
- * Then you start the print operation by calling run(). It will then show a dialog to 
- * let the user select a printer and options. When the user finishes the dialog various signals will be 
- * emitted by the PrintOperation for you to handle, the main one being draw_page. You should then 
+ * Then you start the print operation by calling run(). It will then show a dialog to
+ * let the user select a printer and options. When the user finishes the dialog various signals will be
+ * emitted by the PrintOperation for you to handle, the main one being draw_page. You should then
  * render the page on the provided PrintContext using Cairo.
  *
  * @newin{2,10}
@@ -71,7 +71,7 @@ public:
 
   _WRAP_METHOD(void set_default_page_setup(const Glib::RefPtr<PageSetup>& default_page_setup),
                gtk_print_operation_set_default_page_setup)
-  _WRAP_METHOD(Glib::RefPtr<PageSetup> get_default_page_setup() const, gtk_print_operation_get_default_page_setup)
+  _WRAP_METHOD(Glib::RefPtr<PageSetup> get_default_page_setup() const, gtk_print_operation_get_default_page_setup, refreturn)
 
   _WRAP_METHOD(void set_print_settings(const Glib::RefPtr<PrintSettings>& print_settings),
                gtk_print_operation_set_print_settings)
@@ -100,8 +100,8 @@ public:
 
   _WRAP_METHOD(void draw_page_finish(), gtk_print_operation_draw_page_finish)
   _WRAP_METHOD(void set_defer_drawing(), gtk_print_operation_set_defer_drawing)
-  
-  
+
+
   _WRAP_METHOD(void set_support_selection(bool support_selection = true), gtk_print_operation_set_support_selection)
   _WRAP_METHOD(bool get_support_selection() const, gtk_print_operation_get_support_selection)
   _WRAP_METHOD(void set_has_selection(bool has_selection = true), gtk_print_operation_set_has_selection)
@@ -168,15 +168,15 @@ public:
 
   //TODO: Make these static members of a class instead of non-class functions?
 
-  //Note: gtk_run_page_setup_dialog() can take a null page_setup object, but the application should always 
+  //Note: gtk_run_page_setup_dialog() can take a null page_setup object, but the application should always
   //store and reuse a page_setup object, so I see no need to provide an overload for that. murrayc.
   //TODO: The parent parameter may also be null, so maybe we should add an overload for that.
 
-  /** Runs a page setup dialog, letting the user modify the values from page_setup. 
-   * If the user cancels the dialog, the returned PageSetup is identical to that passed in @a page_setup, 
+  /** Runs a page setup dialog, letting the user modify the values from page_setup.
+   * If the user cancels the dialog, the returned PageSetup is identical to that passed in @a page_setup,
    * otherwise it contains the modifications done in the dialog.
-   * 
-   * Note that this function may use a recursive mainloop to show the page setup dialog. 
+   *
+   * Note that this function may use a recursive mainloop to show the page setup dialog.
    * See run_page_setup_dialog_async() if this is a problem.
    *
    * @param parent Transient parent.
@@ -191,15 +191,15 @@ public:
                                                 const Glib::RefPtr<const PrintSettings>& print_settings);
   //TODO: Add overloads that don't take page_setup.
 
-  /** For example, 
+  /** For example,
    * void on_setup_done(const Glib::RefPtr<PageSetup>& page_setup);
    */
   typedef sigc::slot< void, const Glib::RefPtr<PageSetup>& > SlotPrintSetupDone;
 
   /** Runs a page setup dialog, letting the user modify the values from page_setup.
    *
-   * In contrast to run_page_setup_dialog(), this function returns after showing the 
-   * page setup dialog on platforms that support this, and calls the @a slot from a 
+   * In contrast to run_page_setup_dialog(), this function returns after showing the
+   * page setup dialog on platforms that support this, and calls the @a slot from a
    * signal handler for the ::response signal of the dialog.
    *
    * @param parent Transient parent.



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]