Commit 397fc353 authored by Per Cederqvist's avatar Per Cederqvist

Removed bogus FIXME comments. (Bug 141).

* src/server/aux-items.c (aux_item_link_items): Removed FIXME
comment about bug 141.  Bug 141 is invalid.  The call to
mark_text_as_changed is not redundant, since we must mark both the
source and destination item as changed.
parent 3cc0297e
2006-08-01 Per Cederqvist <ceder@lysator.liu.se>
Removed bogus FIXME comments. (Bug 141).
* src/server/aux-items.c (aux_item_link_items): Removed FIXME
comment about bug 141. Bug 141 is invalid. The call to
mark_text_as_changed is not redundant, since we must mark both the
source and destination item as changed.
Document local-to-global. (Bug 144).
* doc/lyskomd.texi (local-to-global): Translated old Swedish text
that describes the reasoning behind the local-to-global structure,
......
......@@ -636,7 +636,7 @@ mark_linked_object_as_changed(Aux_item *item)
*
* Note that this may cause aux-item lists to be realloced, so pointers
* into such lists may become invalid. Take particular care with the
* pointers in aux-item trigger data
* pointers in aux-item trigger data.
*
* Call aux_item_fix_trigger_data if you need to fix the contents of
* trigger data after calling this function.
......@@ -728,7 +728,7 @@ aux_item_link_items(enum object_type src_type, /* Source object type */
/*
* Add the item and mark the destination as changed
* Add the item and mark the destination as changed.
*
* Note that these calls may cause the aux item list of the
* destination to move in memory due to calls to realloc.
......@@ -740,18 +740,12 @@ aux_item_link_items(enum object_type src_type, /* Source object type */
text_stat_add_aux_item_list((Text_stat *)dst_ptr,
(Text_no)dst_no, &item_list,
src_item->creator);
/* FIXME (bug 141): is this necessary? The call to
mark_linked_object_as_changed below seems to fix this,
right? --ceder 1999-07-10. */
mark_text_as_changed((Text_no)dst_no);
break;
case CONF_OBJECT_TYPE:
conf_stat_add_aux_item_list((Conference *)dst_ptr,
(Conf_no)dst_no, &item_list,
src_item->creator);
/* FIXME (bug 141): is this necessary? The call to
mark_linked_object_as_changed below seems to fix this,
right? --ceder 1999-07-10. */
mark_conference_as_changed((Conf_no)dst_no);
break;
......@@ -776,7 +770,7 @@ aux_item_link_items(enum object_type src_type, /* Source object type */
src_item = NULL;
/*
* Mark the source object as changed
* Mark the source object as changed.
*/
mark_linked_object_as_changed(dst_item_data);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment