Index: trunk/doc/TODO =================================================================== --- trunk/doc/TODO (revision 31862) +++ trunk/doc/TODO (revision 31863) @@ -16,6 +16,7 @@ 2. For later releases =============================================================================== - CLEANUP: move layer_parse_group_str() from core to io_pcb, probably together with the default stackup setup code [report: Igor2] +- CLEANUP: TODO27: undo's removed list shouldn't depend on pobject IDs but probably remove list index because IDs are not unique if we want our operations to retrain IDs. Repro: shift-click subc replace twice, then undo; or drag&drop move selected object and undo [report: Igor2] - CLEANUP/BUG: undo operation while drawing a multiple segment line doesn't change segment attached to the crosshair [report:wojciechk8] - tool_line.c depends on pcb_undo()'s return value; can be fixed only when the old undo system is removed - CLEANUP/BUG: bug_files/pstk_offs.lht - most of the code gets confused if a padstack's shape is far away from the padstack origin; try {c r} or {c f} on R8-1; need to cache hole vs. shape overlap (1 bit per shape in the proto) [report: igor2] Index: trunk/src/obj_subc.c =================================================================== --- trunk/src/obj_subc.c (revision 31862) +++ trunk/src/obj_subc.c (revision 31863) @@ -2375,7 +2375,7 @@ pcb_obj_id_del(data, placed); /* NOTE: Can not keep ID because that would fool the undo system: double-replace a subc and there will be two instances with the same ID on the undo list - and searching by ID will get fooled! */ + and searching by ID will get fooled! TODO27 */ /* placed->ID = target_id;*/ (void)target_id; pcb_obj_id_reg(data, placed); placed->Flags.f &= ~fmask; Index: trunk/src_plugins/tool_std/tool_buffer.c =================================================================== --- trunk/src_plugins/tool_std/tool_buffer.c (revision 31862) +++ trunk/src_plugins/tool_std/tool_buffer.c (revision 31863) @@ -79,7 +79,7 @@ if (pcb_crosshair_note.Moving) { /* release of a drag&drop move of selected objects */ pcb_undo_restore_serial(); - /* NOTE: can not keep ids: + /* NOTE: can not keep ids: TODO27: 1. select an object, id #9 2. drag&drop move 3. drag&drop move it again (still selected)