r15835 - gnucash/trunk/src - Rename GncSxInstances->list to instance_list, which at least emacs doesn't hilight as a (potential)keyword.
Josh Sled
jsled at cvs.gnucash.org
Thu Apr 5 21:20:27 EDT 2007
Author: jsled
Date: 2007-04-05 21:20:27 -0400 (Thu, 05 Apr 2007)
New Revision: 15835
Trac: http://svn.gnucash.org/trac/changeset/15835
Modified:
gnucash/trunk/src/app-utils/gnc-sx-instance-model.c
gnucash/trunk/src/app-utils/gnc-sx-instance-model.h
gnucash/trunk/src/app-utils/test/test-sx.c
gnucash/trunk/src/gnome-utils/gnc-sx-instance-dense-cal-adapter.c
gnucash/trunk/src/gnome/dialog-sx-since-last-run.c
Log:
Rename GncSxInstances->list to instance_list, which at least emacs doesn't hilight as a (potential)keyword.
Modified: gnucash/trunk/src/app-utils/gnc-sx-instance-model.c
===================================================================
--- gnucash/trunk/src/app-utils/gnc-sx-instance-model.c 2007-04-06 01:15:15 UTC (rev 15834)
+++ gnucash/trunk/src/app-utils/gnc-sx-instance-model.c 2007-04-06 01:20:27 UTC (rev 15835)
@@ -337,8 +337,6 @@
{
GList *vars = NULL;
g_hash_table_foreach(inst->variable_bindings, _build_list_from_hash_elts, &vars);
- // @@fixme sort by name
- // @@fixme make sure the returned list is freed by callers.
return vars;
}
@@ -373,7 +371,7 @@
seq_num = gnc_sx_get_instance_count(sx, postponed->data);
inst = gnc_sx_instance_new(instances, SX_INSTANCE_STATE_POSTPONED, &inst_date, postponed->data, seq_num);
//inst->temporal_state = postponed->data;
- instances->list = g_list_append(instances->list, inst);
+ instances->instance_list = g_list_append(instances->instance_list, inst);
}
}
@@ -388,7 +386,7 @@
int seq_num;
seq_num = gnc_sx_get_instance_count(sx, sequence_ctx);
inst = gnc_sx_instance_new(instances, SX_INSTANCE_STATE_TO_CREATE, &cur_date, sequence_ctx, seq_num);
- instances->list = g_list_append(instances->list, inst);
+ instances->instance_list = g_list_append(instances->instance_list, inst);
gnc_sx_incr_temporal_state(sx, sequence_ctx);
cur_date = xaccSchedXactionGetInstanceAfter(sx, &cur_date, sequence_ctx);
}
@@ -400,7 +398,7 @@
int seq_num;
seq_num = gnc_sx_get_instance_count(sx, sequence_ctx);
inst = gnc_sx_instance_new(instances, SX_INSTANCE_STATE_REMINDER, &cur_date, sequence_ctx, seq_num);
- instances->list = g_list_append(instances->list, inst);
+ instances->instance_list = g_list_append(instances->instance_list, inst);
gnc_sx_incr_temporal_state(sx, sequence_ctx);
cur_date = xaccSchedXactionGetInstanceAfter(sx, &cur_date, sequence_ctx);
}
@@ -515,13 +513,13 @@
// variable_names
// sx = null
- for (instance_iter = instances->list; instance_iter != NULL; instance_iter = instance_iter->next)
+ for (instance_iter = instances->instance_list; instance_iter != NULL; instance_iter = instance_iter->next)
{
GncSxInstance *inst = (GncSxInstance*)instance_iter->data;
gnc_sx_instance_free(inst);
}
- g_list_free(instances->list);
- instances->list = NULL;
+ g_list_free(instances->instance_list);
+ instances->instance_list = NULL;
g_free(instances);
}
@@ -739,8 +737,8 @@
// step through the lists pairwise, and retain the existing
// instance if the dates align, as soon as they don't stop and
// cleanup.
- existing_iter = existing->list;
- new_iter = new_instances->list;
+ existing_iter = existing->instance_list;
+ new_iter = new_instances->instance_list;
for (; existing_iter != NULL && new_iter != NULL; existing_iter = existing_iter->next, new_iter = new_iter->next)
{
GncSxInstance *existing_inst, *new_inst;
@@ -759,7 +757,7 @@
if (existing_remain)
{
// delete excess
- gnc_g_list_cut(&existing->list, existing_iter);
+ gnc_g_list_cut(&existing->instance_list, existing_iter);
g_list_foreach(existing_iter, (GFunc)gnc_sx_instance_free, NULL);
}
@@ -767,13 +765,13 @@
{
// append new
GList *new_iter_iter;
- gnc_g_list_cut(&new_instances->list, new_iter);
+ gnc_g_list_cut(&new_instances->instance_list, new_iter);
for (new_iter_iter = new_iter; new_iter_iter != NULL; new_iter_iter = new_iter_iter->next)
{
GncSxInstance *inst = (GncSxInstance*)new_iter_iter->data;
inst->parent = existing;
- existing->list = g_list_append(existing->list, new_iter_iter->data);
+ existing->instance_list = g_list_append(existing->instance_list, new_iter_iter->data);
}
g_list_free(new_iter);
}
@@ -801,7 +799,7 @@
existing->variable_names = new_instances->variable_names;
new_instances->variable_names = NULL;
- for (inst_iter = existing->list; inst_iter != NULL; inst_iter = inst_iter->next)
+ for (inst_iter = existing->instance_list; inst_iter != NULL; inst_iter = inst_iter->next)
{
GList *var_iter;
GncSxInstance *inst = (GncSxInstance*)inst_iter->data;
@@ -888,10 +886,10 @@
NULL);
if (kvp_val == NULL)
{
- // @@fixme: this should be more of an assert...
GString *err = g_string_new("");
g_string_printf(err, "Null account kvp value for SX [%s], cancelling creation.",
xaccSchedXactionGetName(instance->parent->sx));
+ g_critical("%s", err->str);
*creation_errors = g_list_append(*creation_errors, err);
return FALSE;
}
@@ -1176,14 +1174,14 @@
// If there are no instances, then skip; specifically, skip
// re-setting SchedXaction fields, which will dirty the book
// spuriously.
- if (g_list_length(instances->list) == 0)
+ if (g_list_length(instances->instance_list) == 0)
continue;
last_occur_date = xaccSchedXactionGetLastOccurDate(instances->sx);
instance_count = gnc_sx_get_instance_count(instances->sx, NULL);
remain_occur_count = xaccSchedXactionGetRemOccur(instances->sx);
- for (instance_iter = instances->list; instance_iter != NULL; instance_iter = instance_iter->next)
+ for (instance_iter = instances->instance_list; instance_iter != NULL; instance_iter = instance_iter->next)
{
GncSxInstance *inst = (GncSxInstance*)instance_iter->data;
gboolean sx_is_auto_create;
@@ -1255,7 +1253,7 @@
// ensure 'remind' constraints are met:
{
GList *inst_iter;
- inst_iter = g_list_find(instance->parent->list, instance);
+ inst_iter = g_list_find(instance->parent->instance_list, instance);
g_assert(inst_iter != NULL);
if (instance->state != SX_INSTANCE_STATE_REMINDER)
{
@@ -1312,7 +1310,7 @@
for (sx_iter = model->sx_instance_list; sx_iter != NULL; sx_iter = sx_iter->next)
{
GncSxInstances *instances = (GncSxInstances*)sx_iter->data;
- for (inst_iter = instances->list; inst_iter != NULL; inst_iter = inst_iter->next)
+ for (inst_iter = instances->instance_list; inst_iter != NULL; inst_iter = inst_iter->next)
{
GncSxInstance *inst = (GncSxInstance*)inst_iter->data;
@@ -1357,7 +1355,7 @@
GncSxInstances *instances = (GncSxInstances*)sx_iter->data;
gboolean sx_is_auto_create = FALSE, sx_notify = FALSE;
xaccSchedXactionGetAutoCreate(instances->sx, &sx_is_auto_create, &sx_notify);
- for (inst_iter = instances->list; inst_iter != NULL; inst_iter = inst_iter->next)
+ for (inst_iter = instances->instance_list; inst_iter != NULL; inst_iter = inst_iter->next)
{
GncSxInstance *inst = (GncSxInstance*)inst_iter->data;
summary->num_instances++;
Modified: gnucash/trunk/src/app-utils/gnc-sx-instance-model.h
===================================================================
--- gnucash/trunk/src/app-utils/gnc-sx-instance-model.h 2007-04-06 01:15:15 UTC (rev 15834)
+++ gnucash/trunk/src/app-utils/gnc-sx-instance-model.h 2007-04-06 01:20:27 UTC (rev 15835)
@@ -75,7 +75,7 @@
GDate next_instance_date;
/** GList<GncSxInstance*> **/
- GList *list; /* @fixme: s/list/?/ */
+ GList *instance_list;
} GncSxInstances;
typedef enum
@@ -126,7 +126,7 @@
void gnc_sx_instance_model_update_sx_instances(GncSxInstanceModel *model, SchedXaction *sx);
void gnc_sx_instance_model_remove_sx_instances(GncSxInstanceModel *model, SchedXaction *sx);
-/** @return GList<GncSxVariable*> **/
+/** @return GList<GncSxVariable*>. Caller owns the list, but not the items. **/
GList *gnc_sx_instance_get_variables(GncSxInstance *inst);
Account* gnc_sx_get_template_transaction_account(SchedXaction *sx);
Modified: gnucash/trunk/src/app-utils/test/test-sx.c
===================================================================
--- gnucash/trunk/src/app-utils/test/test-sx.c 2007-04-06 01:15:15 UTC (rev 15834)
+++ gnucash/trunk/src/app-utils/test/test-sx.c 2007-04-06 01:20:27 UTC (rev 15835)
@@ -35,8 +35,8 @@
do_test(g_list_length(model->sx_instance_list) == 1, "1 GncSxInstances");
insts = (GncSxInstances*)model->sx_instance_list->data;
- do_test(g_list_length(insts->list) == 3, "yesterday, today and tomorrow");
- for (iter = insts->list; iter != NULL; iter = iter->next)
+ do_test(g_list_length(insts->instance_list) == 3, "yesterday, today and tomorrow");
+ for (iter = insts->instance_list; iter != NULL; iter = iter->next)
{
GncSxInstance *inst = (GncSxInstance*)iter->data;
do_test(inst->state == SX_INSTANCE_STATE_TO_CREATE, "to-create");
@@ -90,8 +90,8 @@
do_test(g_list_length(model->sx_instance_list) == 1, "1 instances");
instances = (GncSxInstances*)model->sx_instance_list->data;
- do_test(g_list_length(instances->list) == 1, "1 instance");
- instance = (GncSxInstance*)instances->list->data;
+ do_test(g_list_length(instances->instance_list) == 1, "1 instance");
+ instance = (GncSxInstance*)instances->instance_list->data;
do_test(g_date_compare(when, &instances->next_instance_date) == 0, "next instance is expected");
do_test(g_date_compare(when, &instance->date) == 0, "instance date is expected");
@@ -103,7 +103,7 @@
static GncSxInstance*
_nth_instance(GncSxInstances *instances, int i)
{
- return (GncSxInstance*)g_list_nth_data(instances->list, i);
+ return (GncSxInstance*)g_list_nth_data(instances->instance_list, i);
}
static void
@@ -127,7 +127,7 @@
do_test(g_list_length(model->sx_instance_list) == 1, "one sx");
insts = (GncSxInstances*)g_list_nth_data(model->sx_instance_list, 0);
- do_test(g_list_length(insts->list) == 4, "4 instances");
+ do_test(g_list_length(insts->instance_list) == 4, "4 instances");
inst = _nth_instance(insts, 2);
gnc_sx_instance_model_change_instance_state(model, inst, SX_INSTANCE_STATE_TO_CREATE);
Modified: gnucash/trunk/src/gnome/dialog-sx-since-last-run.c
===================================================================
--- gnucash/trunk/src/gnome/dialog-sx-since-last-run.c 2007-04-06 01:15:15 UTC (rev 15834)
+++ gnucash/trunk/src/gnome/dialog-sx-since-last-run.c 2007-04-06 01:20:27 UTC (rev 15835)
@@ -469,7 +469,7 @@
char instance_date_buf[MAX_DATE_LENGTH+1];
int instance_index = -1;
- for (inst_iter = instances->list; inst_iter != NULL; inst_iter = inst_iter->next)
+ for (inst_iter = instances->instance_list; inst_iter != NULL; inst_iter = inst_iter->next)
{
GncSxInstance *inst = (GncSxInstance*)inst_iter->data;
qof_print_gdate(instance_date_buf, MAX_DATE_LENGTH, &inst->date);
@@ -594,12 +594,12 @@
gtk_tree_path_free(path);
instances = (GncSxInstances*)g_list_nth_data(model->instances->sx_instance_list, instances_index);
- if (instance_index < 0 || instance_index >= g_list_length(instances->list))
+ if (instance_index < 0 || instance_index >= g_list_length(instances->instance_list))
{
return NULL;
}
- return (GncSxInstance*)g_list_nth_data(instances->list, instance_index);
+ return (GncSxInstance*)g_list_nth_data(instances->instance_list, instance_index);
}
gboolean
@@ -688,7 +688,7 @@
indices[0] = g_list_index(model->instances->sx_instance_list, instance->parent);
if (indices[0] == -1)
return NULL;
- indices[1] = g_list_index(instance->parent->list, instance);
+ indices[1] = g_list_index(instance->parent->instance_list, instance);
if (indices[1] == -1)
return NULL;
variables = gnc_sx_instance_get_variables(instance);
Modified: gnucash/trunk/src/gnome-utils/gnc-sx-instance-dense-cal-adapter.c
===================================================================
--- gnucash/trunk/src/gnome-utils/gnc-sx-instance-dense-cal-adapter.c 2007-04-06 01:15:15 UTC (rev 15834)
+++ gnucash/trunk/src/gnome-utils/gnc-sx-instance-dense-cal-adapter.c 2007-04-06 01:20:27 UTC (rev 15835)
@@ -225,7 +225,7 @@
= (GncSxInstances*)g_list_find_custom(adapter->instances->sx_instance_list, GUINT_TO_POINTER(tag), gsidca_find_sx_with_tag)->data;
if (insts == NULL)
return 0;
- return g_list_length(insts->list);
+ return g_list_length(insts->instance_list);
}
static void
@@ -237,7 +237,7 @@
= (GncSxInstances*)g_list_find_custom(adapter->instances->sx_instance_list, GUINT_TO_POINTER(tag), gsidca_find_sx_with_tag)->data;
if (insts == NULL)
return;
- inst = (GncSxInstance*)g_list_nth_data(insts->list, instance_index);
+ inst = (GncSxInstance*)g_list_nth_data(insts->instance_list, instance_index);
g_date_valid(&inst->date);
*date = inst->date;
g_date_valid(date);
More information about the gnucash-changes
mailing list