+2008-02-07 Michael Natterer <mitch@imendio.com>
+
+ * gtk/gtkcellview.c
+ * gtk/gtklabel.c
+ * gtk/gtklayout.c
+ * gtk/gtklist.c
+ * gtk/gtkmenu.c
+ * gtk/gtkmenushell.c
+ * gtk/gtkmisc.c
+ * gtk/gtkoptionmenu.c
+ * gtk/gtkplug.c
+ * gtk/gtkpreview.c
+ * gtk/gtkprogress.c
+ * gtk/gtkradiomenuitem.c
+ * gtk/gtkscale.c
+ * gtk/gtkscrolledwindow.c
+ * gtk/gtksocket.c: remove g_return_if_fail() from private
+ functions and virtual function implementations.
+
2008-02-07 Michael Natterer <mitch@imendio.com>
* gtk/gtkaction.c
GtkCellViewCellInfo *info;
GtkCellView *cellview = GTK_CELL_VIEW (layout);
- g_return_if_fail (GTK_IS_CELL_VIEW (cellview));
- g_return_if_fail (GTK_IS_CELL_RENDERER (renderer));
g_return_if_fail (!gtk_cell_view_get_cell_info (cellview, renderer));
g_object_ref_sink (renderer);
GtkCellViewCellInfo *info;
GtkCellView *cellview = GTK_CELL_VIEW (layout);
- g_return_if_fail (GTK_IS_CELL_VIEW (cellview));
- g_return_if_fail (GTK_IS_CELL_RENDERER (renderer));
g_return_if_fail (!gtk_cell_view_get_cell_info (cellview, renderer));
g_object_ref_sink (renderer);
GtkCellViewCellInfo *info;
GtkCellView *cellview = GTK_CELL_VIEW (layout);
- g_return_if_fail (GTK_IS_CELL_VIEW (cellview));
info = gtk_cell_view_get_cell_info (cellview, renderer);
g_return_if_fail (info != NULL);
{
GtkCellView *cellview = GTK_CELL_VIEW (layout);
- g_return_if_fail (GTK_IS_CELL_VIEW (cellview));
-
while (cellview->priv->cell_list)
{
GtkCellViewCellInfo *info = (GtkCellViewCellInfo *)cellview->priv->cell_list->data;
GtkCellView *cellview = GTK_CELL_VIEW (layout);
GtkCellViewCellInfo *info;
- g_return_if_fail (GTK_IS_CELL_VIEW (cellview));
-
info = gtk_cell_view_get_cell_info (cellview, cell);
g_return_if_fail (info != NULL);
gtk_cell_view_cell_layout_clear_attributes (GtkCellLayout *layout,
GtkCellRenderer *renderer)
{
- GtkCellViewCellInfo *info;
GtkCellView *cellview = GTK_CELL_VIEW (layout);
+ GtkCellViewCellInfo *info;
GSList *list;
- g_return_if_fail (GTK_IS_CELL_VIEW (cellview));
- g_return_if_fail (GTK_IS_CELL_RENDERER (renderer));
-
info = gtk_cell_view_get_cell_info (cellview, renderer);
if (info != NULL)
{
GtkCellRenderer *cell,
gint position)
{
- GList *link;
- GtkCellViewCellInfo *info;
GtkCellView *cellview = GTK_CELL_VIEW (layout);
-
- g_return_if_fail (GTK_IS_CELL_VIEW (cellview));
- g_return_if_fail (GTK_IS_CELL_RENDERER (cell));
+ GtkCellViewCellInfo *info;
+ GList *link;
info = gtk_cell_view_get_cell_info (cellview, cell);
gchar *property,
GValue *value)
{
- g_return_if_fail (GTK_IS_CELL_VIEW (cell_view));
- g_return_if_fail (GTK_IS_CELL_RENDERER (renderer));
-
g_object_set_property (G_OBJECT (renderer), property, value);
/* force resize and redraw */
static void
gtk_label_finalize (GObject *object)
{
- GtkLabel *label;
-
- g_return_if_fail (GTK_IS_LABEL (object));
-
- label = GTK_LABEL (object);
-
+ GtkLabel *label = GTK_LABEL (object);
+
g_free (label->label);
g_free (label->text);
gtk_label_size_request (GtkWidget *widget,
GtkRequisition *requisition)
{
- GtkLabel *label;
+ GtkLabel *label = GTK_LABEL (widget);
GtkLabelPrivate *priv;
gint width, height;
PangoRectangle logical_rect;
GtkWidgetAuxInfo *aux_info;
-
- g_return_if_fail (GTK_IS_LABEL (widget));
- g_return_if_fail (requisition != NULL);
-
- label = GTK_LABEL (widget);
+
priv = GTK_LABEL_GET_PRIVATE (widget);
/*
gtk_label_style_set (GtkWidget *widget,
GtkStyle *previous_style)
{
- GtkLabel *label;
-
- g_return_if_fail (GTK_IS_LABEL (widget));
-
- label = GTK_LABEL (widget);
+ GtkLabel *label = GTK_LABEL (widget);
/* We have to clear the layout, fonts etc. may have changed */
gtk_label_clear_layout (label);
gint y)
{
GtkLayoutChild *child;
-
- g_return_if_fail (GTK_IS_LAYOUT (layout));
- g_return_if_fail (GTK_IS_WIDGET (widget));
- g_return_if_fail (widget->parent == GTK_WIDGET (layout));
-
+
child = get_child (layout, widget);
g_assert (child);
static void
gtk_layout_realize (GtkWidget *widget)
{
+ GtkLayout *layout = GTK_LAYOUT (widget);
GList *tmp_list;
- GtkLayout *layout;
GdkWindowAttr attributes;
gint attributes_mask;
- g_return_if_fail (GTK_IS_LAYOUT (widget));
-
- layout = GTK_LAYOUT (widget);
GTK_WIDGET_SET_FLAGS (layout, GTK_REALIZED);
attributes.window_type = GDK_WINDOW_CHILD;
}
}
-static void
+static void
gtk_layout_map (GtkWidget *widget)
{
+ GtkLayout *layout = GTK_LAYOUT (widget);
GList *tmp_list;
- GtkLayout *layout;
-
- g_return_if_fail (GTK_IS_LAYOUT (widget));
-
- layout = GTK_LAYOUT (widget);
GTK_WIDGET_SET_FLAGS (widget, GTK_MAPPED);
static void
gtk_layout_unrealize (GtkWidget *widget)
{
- GtkLayout *layout;
-
- g_return_if_fail (GTK_IS_LAYOUT (widget));
-
- layout = GTK_LAYOUT (widget);
+ GtkLayout *layout = GTK_LAYOUT (widget);
gdk_window_set_user_data (layout->bin_window, NULL);
gdk_window_destroy (layout->bin_window);
gtk_layout_size_request (GtkWidget *widget,
GtkRequisition *requisition)
{
+ GtkLayout *layout = GTK_LAYOUT (widget);
GList *tmp_list;
- GtkLayout *layout;
-
- g_return_if_fail (GTK_IS_LAYOUT (widget));
-
- layout = GTK_LAYOUT (widget);
requisition->width = 0;
requisition->height = 0;
gtk_layout_size_allocate (GtkWidget *widget,
GtkAllocation *allocation)
{
+ GtkLayout *layout = GTK_LAYOUT (widget);
GList *tmp_list;
- GtkLayout *layout;
-
- g_return_if_fail (GTK_IS_LAYOUT (widget));
widget->allocation = *allocation;
-
- layout = GTK_LAYOUT (widget);
tmp_list = layout->children;
}
static gint
-gtk_layout_expose (GtkWidget *widget, GdkEventExpose *event)
+gtk_layout_expose (GtkWidget *widget,
+ GdkEventExpose *event)
{
- GtkLayout *layout;
-
- g_return_val_if_fail (GTK_IS_LAYOUT (widget), FALSE);
-
- layout = GTK_LAYOUT (widget);
+ GtkLayout *layout = GTK_LAYOUT (widget);
if (event->window != layout->bin_window)
return FALSE;
gtk_layout_remove (GtkContainer *container,
GtkWidget *widget)
{
+ GtkLayout *layout = GTK_LAYOUT (container);
GList *tmp_list;
- GtkLayout *layout;
GtkLayoutChild *child = NULL;
-
- g_return_if_fail (GTK_IS_LAYOUT (container));
-
- layout = GTK_LAYOUT (container);
tmp_list = layout->children;
while (tmp_list)
GtkCallback callback,
gpointer callback_data)
{
- GtkLayout *layout;
+ GtkLayout *layout = GTK_LAYOUT (container);
GtkLayoutChild *child;
GList *tmp_list;
- g_return_if_fail (GTK_IS_LAYOUT (container));
- g_return_if_fail (callback != NULL);
-
- layout = GTK_LAYOUT (container);
-
tmp_list = layout->children;
while (tmp_list)
{
gtk_list_size_request (GtkWidget *widget,
GtkRequisition *requisition)
{
- GtkList *list;
+ GtkList *list = GTK_LIST (widget);
GtkWidget *child;
GList *children;
- g_return_if_fail (GTK_IS_LIST (widget));
- g_return_if_fail (requisition != NULL);
-
- list = GTK_LIST (widget);
requisition->width = 0;
requisition->height = 0;
gtk_list_size_allocate (GtkWidget *widget,
GtkAllocation *allocation)
{
- GtkList *list;
+ GtkList *list = GTK_LIST (widget);
GtkWidget *child;
GtkAllocation child_allocation;
GList *children;
- g_return_if_fail (GTK_IS_LIST (widget));
- g_return_if_fail (allocation != NULL);
-
- list = GTK_LIST (widget);
-
widget->allocation = *allocation;
if (GTK_WIDGET_REALIZED (widget))
gdk_window_move_resize (widget->window,
GdkWindowAttr attributes;
gint attributes_mask;
- g_return_if_fail (GTK_IS_LIST (widget));
-
GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
attributes.window_type = GDK_WINDOW_CHILD;
static void
gtk_list_unmap (GtkWidget *widget)
{
- GtkList *list;
-
- g_return_if_fail (GTK_IS_LIST (widget));
+ GtkList *list = GTK_LIST (widget);
if (!GTK_WIDGET_MAPPED (widget))
return;
- list = GTK_LIST (widget);
-
GTK_WIDGET_UNSET_FLAGS (widget, GTK_MAPPED);
if (list_has_grab (list))
gtk_list_motion_notify (GtkWidget *widget,
GdkEventMotion *event)
{
- GtkList *list;
+ GtkList *list = GTK_LIST (widget);
GtkWidget *item = NULL;
GtkAdjustment *adj;
GtkContainer *container;
gint focus_row = 0;
gint length = 0;
- g_return_val_if_fail (GTK_IS_LIST (widget), FALSE);
- g_return_val_if_fail (event != NULL, FALSE);
-
- list = GTK_LIST (widget);
-
if (!list->drag_selection || !list->children)
return FALSE;
gtk_list_button_press (GtkWidget *widget,
GdkEventButton *event)
{
- GtkList *list;
+ GtkList *list = GTK_LIST (widget);
GtkWidget *item;
- g_return_val_if_fail (GTK_IS_LIST (widget), FALSE);
- g_return_val_if_fail (event != NULL, FALSE);
-
if (event->button != 1)
return FALSE;
- list = GTK_LIST (widget);
item = gtk_get_event_widget ((GdkEvent*) event);
while (item && !GTK_IS_LIST_ITEM (item))
gtk_list_button_release (GtkWidget *widget,
GdkEventButton *event)
{
- GtkList *list;
+ GtkList *list = GTK_LIST (widget);
GtkWidget *item;
- g_return_val_if_fail (GTK_IS_LIST (widget), FALSE);
- g_return_val_if_fail (event != NULL, FALSE);
-
- list = GTK_LIST (widget);
-
/* we don't handle button 2 and 3 */
if (event->button != 1)
return FALSE;
gtk_list_style_set (GtkWidget *widget,
GtkStyle *previous_style)
{
- g_return_if_fail (widget != NULL);
-
if (previous_style && GTK_WIDGET_REALIZED (widget))
gdk_window_set_background (widget->window, &widget->style->base[GTK_WIDGET_STATE (widget)]);
}
{
GList *item_list;
- g_return_if_fail (GTK_IS_LIST (container));
g_return_if_fail (GTK_IS_LIST_ITEM (widget));
item_list = g_list_alloc ();
GtkWidget *widget)
{
GList *item_list;
-
- g_return_if_fail (GTK_IS_LIST (container));
- g_return_if_fail (widget != NULL);
+
g_return_if_fail (container == GTK_CONTAINER (widget->parent));
item_list = g_list_alloc ();
GtkCallback callback,
gpointer callback_data)
{
- GtkList *list;
+ GtkList *list = GTK_LIST (container);
GtkWidget *child;
GList *children;
- g_return_if_fail (GTK_IS_LIST (container));
- g_return_if_fail (callback != NULL);
-
- list = GTK_LIST (container);
children = list->children;
while (children)
static void
gtk_menu_destroy (GtkObject *object)
{
- GtkMenu *menu;
+ GtkMenu *menu = GTK_MENU (object);
GtkMenuAttachData *data;
GtkMenuPrivate *priv;
- g_return_if_fail (GTK_IS_MENU (object));
-
- menu = GTK_MENU (object);
-
gtk_menu_remove_scroll_timeout (menu);
data = g_object_get_data (G_OBJECT (object), attach_data_key);
gtk_menu_remove (GtkContainer *container,
GtkWidget *widget)
{
- GtkMenu *menu;
+ GtkMenu *menu = GTK_MENU (container);
- g_return_if_fail (GTK_IS_MENU (container));
g_return_if_fail (GTK_IS_MENU_ITEM (widget));
- menu = GTK_MENU (container);
-
/* Clear out old_active_menu_item if it matches the item we are removing
*/
if (menu->old_active_menu_item == widget)
static void
gtk_menu_unrealize (GtkWidget *widget)
{
- GtkMenu *menu;
-
- g_return_if_fail (GTK_IS_MENU (widget));
-
- menu = GTK_MENU (widget);
+ GtkMenu *menu = GTK_MENU (widget);
menu_grab_transfer_window_destroy (menu);
GdkWindowAttr attributes;
gint attributes_mask;
- g_return_if_fail (GTK_IS_MISC (widget));
-
GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
if (GTK_WIDGET_NO_WINDOW (widget))
static void
gtk_option_menu_destroy (GtkObject *object)
{
- GtkOptionMenu *option_menu;
-
- g_return_if_fail (GTK_IS_OPTION_MENU (object));
-
- option_menu = GTK_OPTION_MENU (object);
+ GtkOptionMenu *option_menu = GTK_OPTION_MENU (object);
if (option_menu->menu)
gtk_widget_destroy (option_menu->menu);
static void
gtk_plug_unrealize (GtkWidget *widget)
{
- GtkPlug *plug;
-
- g_return_if_fail (GTK_IS_PLUG (widget));
-
- plug = GTK_PLUG (widget);
+ GtkPlug *plug = GTK_PLUG (widget);
if (plug->socket_window != NULL)
{
static void
gtk_plug_realize (GtkWidget *widget)
{
- GtkWindow *window;
- GtkPlug *plug;
+ GtkWindow *window = GTK_WINDOW (widget);
+ GtkPlug *plug = GTK_PLUG (widget);
GdkWindowAttr attributes;
gint attributes_mask;
- g_return_if_fail (GTK_IS_PLUG (widget));
-
GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
- window = GTK_WINDOW (widget);
- plug = GTK_PLUG (widget);
attributes.window_type = GDK_WINDOW_CHILD; /* XXX GDK_WINDOW_PLUG ? */
attributes.title = window->title;
static void
gtk_preview_finalize (GObject *object)
{
- GtkPreview *preview;
-
- g_return_if_fail (GTK_IS_PREVIEW (object));
+ GtkPreview *preview = GTK_PREVIEW (object);
- preview = GTK_PREVIEW (object);
g_free (preview->buffer);
G_OBJECT_CLASS (gtk_preview_parent_class)->finalize (object);
static void
gtk_preview_realize (GtkWidget *widget)
{
- GtkPreview *preview;
+ GtkPreview *preview = GTK_PREVIEW (widget);
GdkWindowAttr attributes;
gint attributes_mask;
- g_return_if_fail (GTK_IS_PREVIEW (widget));
-
GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
- preview = GTK_PREVIEW (widget);
attributes.window_type = GDK_WINDOW_CHILD;
gtk_preview_size_allocate (GtkWidget *widget,
GtkAllocation *allocation)
{
- GtkPreview *preview;
+ GtkPreview *preview = GTK_PREVIEW (widget);
gint width, height;
- g_return_if_fail (GTK_IS_PREVIEW (widget));
-
- preview = GTK_PREVIEW (widget);
widget->allocation = *allocation;
if (GTK_WIDGET_REALIZED (widget))
GtkPreview *preview;
gint width, height;
- g_return_val_if_fail (GTK_IS_PREVIEW (widget), FALSE);
- g_return_val_if_fail (event != NULL, FALSE);
-
if (GTK_WIDGET_DRAWABLE (widget))
{
preview = GTK_PREVIEW (widget);
static void
gtk_progress_realize (GtkWidget *widget)
{
- GtkProgress *progress;
+ GtkProgress *progress = GTK_PROGRESS (widget);
GdkWindowAttr attributes;
gint attributes_mask;
- g_return_if_fail (GTK_IS_PROGRESS (widget));
-
- progress = GTK_PROGRESS (widget);
GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
attributes.window_type = GDK_WINDOW_CHILD;
static void
gtk_progress_destroy (GtkObject *object)
{
- GtkProgress *progress;
-
- g_return_if_fail (GTK_IS_PROGRESS (object));
-
- progress = GTK_PROGRESS (object);
+ GtkProgress *progress = GTK_PROGRESS (object);
if (progress->adjustment)
{
static void
gtk_progress_finalize (GObject *object)
{
- GtkProgress *progress;
-
- g_return_if_fail (GTK_IS_PROGRESS (object));
-
- progress = GTK_PROGRESS (object);
+ GtkProgress *progress = GTK_PROGRESS (object);
if (progress->offscreen_pixmap)
g_object_unref (progress->offscreen_pixmap);
gtk_progress_expose (GtkWidget *widget,
GdkEventExpose *event)
{
- g_return_val_if_fail (GTK_IS_PROGRESS (widget), FALSE);
- g_return_val_if_fail (event != NULL, FALSE);
-
if (GTK_WIDGET_DRAWABLE (widget))
gdk_draw_drawable (widget->window,
widget->style->black_gc,
gtk_progress_size_allocate (GtkWidget *widget,
GtkAllocation *allocation)
{
- g_return_if_fail (GTK_IS_PROGRESS (widget));
- g_return_if_fail (allocation != NULL);
-
widget->allocation = *allocation;
if (GTK_WIDGET_REALIZED (widget))
static void
gtk_radio_menu_item_destroy (GtkObject *object)
{
+ GtkRadioMenuItem *radio_menu_item = GTK_RADIO_MENU_ITEM (object);
GtkWidget *old_group_singleton = NULL;
- GtkRadioMenuItem *radio_menu_item;
GtkRadioMenuItem *tmp_menu_item;
GSList *tmp_list;
gboolean was_in_group;
- g_return_if_fail (GTK_IS_RADIO_MENU_ITEM (object));
-
- radio_menu_item = GTK_RADIO_MENU_ITEM (object);
-
was_in_group = radio_menu_item->group && radio_menu_item->group->next;
radio_menu_item->group = g_slist_remove (radio_menu_item->group,
static void
gtk_radio_menu_item_activate (GtkMenuItem *menu_item)
{
- GtkRadioMenuItem *radio_menu_item;
- GtkCheckMenuItem *check_menu_item;
+ GtkRadioMenuItem *radio_menu_item = GTK_RADIO_MENU_ITEM (menu_item);
+ GtkCheckMenuItem *check_menu_item = GTK_CHECK_MENU_ITEM (menu_item);
GtkCheckMenuItem *tmp_menu_item;
GSList *tmp_list;
gint toggled;
- g_return_if_fail (GTK_IS_RADIO_MENU_ITEM (menu_item));
-
- radio_menu_item = GTK_RADIO_MENU_ITEM (menu_item);
- check_menu_item = GTK_CHECK_MENU_ITEM (menu_item);
toggled = FALSE;
if (check_menu_item->active)
static void
gtk_scale_finalize (GObject *object)
{
- GtkScale *scale;
-
- g_return_if_fail (GTK_IS_SCALE (object));
-
- scale = GTK_SCALE (object);
+ GtkScale *scale = GTK_SCALE (object);
_gtk_scale_clear_layout (scale);
*y = local_y;
}
-void _gtk_scale_clear_layout (GtkScale *scale)
+void
+_gtk_scale_clear_layout (GtkScale *scale)
{
GtkScalePrivate *priv = GTK_SCALE_GET_PRIVATE (scale);
static void
gtk_scrolled_window_destroy (GtkObject *object)
{
- GtkScrolledWindow *scrolled_window;
-
- g_return_if_fail (GTK_IS_SCROLLED_WINDOW (object));
-
- scrolled_window = GTK_SCROLLED_WINDOW (object);
+ GtkScrolledWindow *scrolled_window = GTK_SCROLLED_WINDOW (object);
gtk_widget_unparent (scrolled_window->hscrollbar);
gtk_widget_unparent (scrolled_window->vscrollbar);
static void
gtk_socket_realize (GtkWidget *widget)
{
- GtkSocket *socket;
+ GtkSocket *socket = GTK_SOCKET (widget);
GdkWindowAttr attributes;
gint attributes_mask;
- g_return_if_fail (GTK_IS_SOCKET (widget));
-
- socket = GTK_SOCKET (widget);
GTK_WIDGET_SET_FLAGS (widget, GTK_REALIZED);
attributes.window_type = GDK_WINDOW_CHILD;
gtk_socket_size_allocate (GtkWidget *widget,
GtkAllocation *allocation)
{
- GtkSocket *socket;
-
- g_return_if_fail (GTK_IS_SOCKET (widget));
- g_return_if_fail (allocation != NULL);
-
- socket = GTK_SOCKET (widget);
+ GtkSocket *socket = GTK_SOCKET (widget);
widget->allocation = *allocation;
if (GTK_WIDGET_REALIZED (widget))
gtk_socket_focus (GtkWidget *widget,
GtkDirectionType direction)
{
- GtkSocket *socket;
-
- g_return_val_if_fail (GTK_IS_SOCKET (widget), FALSE);
-
- socket = GTK_SOCKET (widget);
+ GtkSocket *socket = GTK_SOCKET (widget);
if (socket->plug_widget)
return gtk_widget_child_focus (socket->plug_widget, direction);
static void
gtk_statusbar_destroy (GtkObject *object)
{
- GtkStatusbar *statusbar;
+ GtkStatusbar *statusbar = GTK_STATUSBAR (object);
GSList *list;
- g_return_if_fail (GTK_IS_STATUSBAR (object));
-
- statusbar = GTK_STATUSBAR (object);
-
for (list = statusbar->messages; list; list = list->next)
{
GtkStatusbarMsg *msg;