I don't know if this may be related to the problems at hand, but there are some compiler warnings about DesignAdd() and RemoveDesign().

In the latter for instance, there is :

{
    GList *plBoardDesigns = (GList *) data;
...
    plBoardDesigns = g_list_remove(plBoardDesigns, pbdeSelected);
... <no use of plBoardDesigns>
    DesignSave(pw, data);
}

- at the start, data and plBoardDesigns are identical, but after the g_list_remove(), this may no longer be the case. What is saved may not be what is expected.

- in plBoardDesigns = ..., do we really mean the local plBoardDesigns that is not referenced after that (although maybe it should be, as argument of DesignSave), or the global plBoardDesigns variable that is shadowed in this function ?

The issues in DesignAdd() are similar.

_______________________________________________
Bug-gnubg mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/bug-gnubg

Reply via email to