common/date-chooser: Housekeeping
Add some assertions and comments
This commit is contained in:
committed by
Georges Basile Stavracas Neto
parent
303cba1622
commit
931254c2a4
@@ -48,6 +48,11 @@ static GParamSpec *properties[NUM_PROPS] = { NULL, };
|
|||||||
|
|
||||||
G_DEFINE_TYPE (GcalDateChooserDay, gcal_date_chooser_day, GTK_TYPE_BUTTON)
|
G_DEFINE_TYPE (GcalDateChooserDay, gcal_date_chooser_day, GTK_TYPE_BUTTON)
|
||||||
|
|
||||||
|
|
||||||
|
/*
|
||||||
|
* GObject overrides
|
||||||
|
*/
|
||||||
|
|
||||||
static void
|
static void
|
||||||
gcal_date_chooser_day_dispose (GObject *object)
|
gcal_date_chooser_day_dispose (GObject *object)
|
||||||
{
|
{
|
||||||
@@ -158,6 +163,18 @@ gcal_date_chooser_day_init (GcalDateChooserDay *self)
|
|||||||
gtk_widget_init_template (GTK_WIDGET (self));
|
gtk_widget_init_template (GTK_WIDGET (self));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Public API
|
||||||
|
*/
|
||||||
|
|
||||||
|
/**
|
||||||
|
* gcal_date_chooser_day_new:
|
||||||
|
*
|
||||||
|
* Creates a new `DateChooserDay`.
|
||||||
|
*
|
||||||
|
* Returns: the newly created `DateChooserDay`
|
||||||
|
*/
|
||||||
GtkWidget*
|
GtkWidget*
|
||||||
gcal_date_chooser_day_new (void)
|
gcal_date_chooser_day_new (void)
|
||||||
{
|
{
|
||||||
@@ -174,7 +191,8 @@ gcal_date_chooser_day_set_date (GcalDateChooserDay *self,
|
|||||||
gboolean today;
|
gboolean today;
|
||||||
gint weekday;
|
gint weekday;
|
||||||
|
|
||||||
g_assert (date != NULL);
|
g_assert (GCAL_IS_DATE_CHOOSER_DAY (self));
|
||||||
|
g_assert (date);
|
||||||
|
|
||||||
g_clear_pointer (&self->date, g_date_time_unref);
|
g_clear_pointer (&self->date, g_date_time_unref);
|
||||||
self->date = g_date_time_ref (date);
|
self->date = g_date_time_ref (date);
|
||||||
@@ -200,6 +218,8 @@ gcal_date_chooser_day_set_date (GcalDateChooserDay *self,
|
|||||||
GDateTime*
|
GDateTime*
|
||||||
gcal_date_chooser_day_get_date (GcalDateChooserDay *self)
|
gcal_date_chooser_day_get_date (GcalDateChooserDay *self)
|
||||||
{
|
{
|
||||||
|
g_assert (GCAL_IS_DATE_CHOOSER_DAY (self));
|
||||||
|
|
||||||
return self->date;
|
return self->date;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -207,6 +227,8 @@ void
|
|||||||
gcal_date_chooser_day_set_other_month (GcalDateChooserDay *self,
|
gcal_date_chooser_day_set_other_month (GcalDateChooserDay *self,
|
||||||
gboolean other_month)
|
gboolean other_month)
|
||||||
{
|
{
|
||||||
|
g_assert (GCAL_IS_DATE_CHOOSER_DAY (self));
|
||||||
|
|
||||||
if (other_month)
|
if (other_month)
|
||||||
gtk_widget_add_css_class (GTK_WIDGET (self), "other-month");
|
gtk_widget_add_css_class (GTK_WIDGET (self), "other-month");
|
||||||
else
|
else
|
||||||
@@ -227,7 +249,7 @@ void
|
|||||||
gcal_date_chooser_day_set_has_dot (GcalDateChooserDay *self,
|
gcal_date_chooser_day_set_has_dot (GcalDateChooserDay *self,
|
||||||
gboolean has_dot)
|
gboolean has_dot)
|
||||||
{
|
{
|
||||||
g_return_if_fail (GCAL_IS_DATE_CHOOSER_DAY (self));
|
g_assert (GCAL_IS_DATE_CHOOSER_DAY (self));
|
||||||
|
|
||||||
has_dot = !!has_dot;
|
has_dot = !!has_dot;
|
||||||
|
|
||||||
@@ -248,7 +270,7 @@ gcal_date_chooser_day_set_has_dot (GcalDateChooserDay *self,
|
|||||||
gboolean
|
gboolean
|
||||||
gcal_date_chooser_day_get_has_dot (GcalDateChooserDay *self)
|
gcal_date_chooser_day_get_has_dot (GcalDateChooserDay *self)
|
||||||
{
|
{
|
||||||
g_return_val_if_fail (GCAL_IS_DATE_CHOOSER_DAY (self), FALSE);
|
g_assert (GCAL_IS_DATE_CHOOSER_DAY (self));
|
||||||
|
|
||||||
return gtk_widget_get_visible (self->dot_revealer);
|
return gtk_widget_get_visible (self->dot_revealer);
|
||||||
}
|
}
|
||||||
@@ -267,7 +289,7 @@ void
|
|||||||
gcal_date_chooser_day_set_dot_visible (GcalDateChooserDay *self,
|
gcal_date_chooser_day_set_dot_visible (GcalDateChooserDay *self,
|
||||||
gboolean dot_visible)
|
gboolean dot_visible)
|
||||||
{
|
{
|
||||||
g_return_if_fail (GCAL_IS_DATE_CHOOSER_DAY (self));
|
g_assert (GCAL_IS_DATE_CHOOSER_DAY (self));
|
||||||
|
|
||||||
dot_visible = !!dot_visible;
|
dot_visible = !!dot_visible;
|
||||||
|
|
||||||
@@ -288,7 +310,7 @@ gcal_date_chooser_day_set_dot_visible (GcalDateChooserDay *self,
|
|||||||
gboolean
|
gboolean
|
||||||
gcal_date_chooser_day_get_dot_visible (GcalDateChooserDay *self)
|
gcal_date_chooser_day_get_dot_visible (GcalDateChooserDay *self)
|
||||||
{
|
{
|
||||||
g_return_val_if_fail (GCAL_IS_DATE_CHOOSER_DAY (self), FALSE);
|
g_assert (GCAL_IS_DATE_CHOOSER_DAY (self));
|
||||||
|
|
||||||
return gtk_revealer_get_reveal_child (GTK_REVEALER (self->dot_revealer));
|
return gtk_revealer_get_reveal_child (GTK_REVEALER (self->dot_revealer));
|
||||||
}
|
}
|
||||||
@@ -297,7 +319,11 @@ void
|
|||||||
gcal_date_chooser_day_set_selected (GcalDateChooserDay *self,
|
gcal_date_chooser_day_set_selected (GcalDateChooserDay *self,
|
||||||
gboolean selected)
|
gboolean selected)
|
||||||
{
|
{
|
||||||
GtkWidget *widget = GTK_WIDGET (self);
|
GtkWidget *widget;
|
||||||
|
|
||||||
|
g_assert (GCAL_IS_DATE_CHOOSER_DAY (self));
|
||||||
|
|
||||||
|
widget = GTK_WIDGET (self);
|
||||||
|
|
||||||
if (G_UNLIKELY (selected))
|
if (G_UNLIKELY (selected))
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -1090,6 +1090,18 @@ gcal_date_chooser_init (GcalDateChooser *self)
|
|||||||
gtk_widget_add_controller (GTK_WIDGET (self), GTK_EVENT_CONTROLLER (drop_target));
|
gtk_widget_add_controller (GTK_WIDGET (self), GTK_EVENT_CONTROLLER (drop_target));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Public API
|
||||||
|
*/
|
||||||
|
|
||||||
|
/**
|
||||||
|
* gcal_date_chooser_new:
|
||||||
|
*
|
||||||
|
* Creates a new `DateChooser`.
|
||||||
|
*
|
||||||
|
* Returns: the newly created `DateChooser`
|
||||||
|
*/
|
||||||
GtkWidget*
|
GtkWidget*
|
||||||
gcal_date_chooser_new (void)
|
gcal_date_chooser_new (void)
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -15,8 +15,7 @@
|
|||||||
* License along with this library. If not, see <http://www.gnu.org/licenses/>.
|
* License along with this library. If not, see <http://www.gnu.org/licenses/>.
|
||||||
*/
|
*/
|
||||||
|
|
||||||
#ifndef __GCAL_DATE_CHOOSER_H__
|
#pragma once
|
||||||
#define __GCAL_DATE_CHOOSER_H__
|
|
||||||
|
|
||||||
#include <adwaita.h>
|
#include <adwaita.h>
|
||||||
|
|
||||||
@@ -59,5 +58,3 @@ void gcal_date_chooser_set_split_month_year (GcalDateChoose
|
|||||||
gboolean setting);
|
gboolean setting);
|
||||||
|
|
||||||
G_END_DECLS
|
G_END_DECLS
|
||||||
|
|
||||||
#endif /* __GCAL_DATE_CHOOSER_H__ */
|
|
||||||
|
|||||||
@@ -1,6 +1,5 @@
|
|||||||
<?xml version="1.0" encoding="UTF-8"?>
|
<?xml version="1.0" encoding="UTF-8"?>
|
||||||
<interface>
|
<interface>
|
||||||
<requires lib="gtk+" version="3.16"/>
|
|
||||||
<template class="GcalDateChooser" parent="AdwBin">
|
<template class="GcalDateChooser" parent="AdwBin">
|
||||||
<child>
|
<child>
|
||||||
<object class="GtkGrid">
|
<object class="GtkGrid">
|
||||||
|
|||||||
Reference in New Issue
Block a user