Index: trunk/src/data.c =================================================================== --- trunk/src/data.c (revision 21406) +++ trunk/src/data.c (revision 21407) @@ -606,7 +606,7 @@ void pcb_data_list_terms(pcb_data_t *data, vtp0_t *dst, pcb_objtype_t type) { -#warning subc TODO: subc in subc +TODO("subc TODO: subc in subc") /* if (type & PCB_OBJ_SUBC) PCB_SUBC_LOOP(data); { if (subc->term != NULL) vtp0_append(dst, subc); } PCB_END_LOOP;*/ @@ -791,7 +791,7 @@ } } -#warning TODO: rewrite to be more generic +TODO("rewrite to be more generic") #include "obj_pstk_draw.h" #include "obj_text_draw.h" #include "obj_poly_draw.h" Index: trunk/src/find_misc.c =================================================================== --- trunk/src/find_misc.c (revision 21406) +++ trunk/src/find_misc.c (revision 21407) @@ -49,7 +49,7 @@ static void reassign_no_drc_flags(void) { int layer; -#warning layer TODO: decide whether it is from attribute or not +TODO("layer: decide whether it is from attribute or not") for (layer = 0; layer < pcb_max_layer; layer++) { pcb_layer_t *l = LAYER_PTR(layer); l->meta.real.no_drc = pcb_attribute_get(&l->Attributes, "PCB::skip-drc") != NULL; @@ -250,7 +250,7 @@ pcb_conn_lookup_uninit(); } -#warning cleanup TODO: keep only PCB_OBJ_* and remove this function +TODO("cleanup: keep only PCB_OBJ_* and remove this function") unsigned long pcb_obj_type2oldtype(pcb_objtype_t type) { switch(type) { Index: trunk/src/hid_cam.c =================================================================== --- trunk/src/hid_cam.c (revision 21406) +++ trunk/src/hid_cam.c (revision 21407) @@ -356,7 +356,7 @@ if (parse_layer_type(curr, &lyt, &offs, &has_offs) != 0) goto err; -#warning TODO: extend the syntax for purpose +TODO("extend the syntax for purpose") parse_layer_supplements(spk, spv, spc, &purpose, &xf, &xf_); vl = pcb_vlayer_get_first(lyt, purpose, -1); Index: trunk/src/library.c =================================================================== --- trunk/src/library.c (revision 21406) +++ trunk/src/library.c (revision 21407) @@ -26,12 +26,13 @@ * */ -#warning cleanup TODO: replace this with genvect +#include "config.h" +TODO("cleanup: replace this with genvect") + #include #include #include -#include "config.h" #include "library.h" #include "macro.h" #include "compat_misc.h" Index: trunk/src/obj_pstk_act.c =================================================================== --- trunk/src/obj_pstk_act.c (revision 21406) +++ trunk/src/obj_pstk_act.c (revision 21407) @@ -183,7 +183,7 @@ } if ((pids == NULL) || (strcmp(pids, "default") == 0)) { -#warning padstack TODO: style default proto +TODO("pstk: style default proto") pid = 0; } else { Index: trunk/src/obj_pstk_op.c =================================================================== --- trunk/src/obj_pstk_op.c (revision 21406) +++ trunk/src/obj_pstk_op.c (revision 21407) @@ -160,7 +160,7 @@ pcb_undo_add_obj_to_clear_poly(PCB_OBJ_PSTK, ps, ps, ps, pcb_false); pcb_poly_restore_to_poly(pcb->Data, PCB_OBJ_PSTK, layer, ps); -#warning TODO: undo +TODO("undo") pcb_pstk_set_thermal(ps, ctx->chgtherm.lid, ctx->chgtherm.style); pcb_undo_add_obj_to_clear_poly(PCB_OBJ_PSTK, ps, ps, ps, pcb_true); Index: trunk/src/obj_pstk_proto.c =================================================================== --- trunk/src/obj_pstk_proto.c (revision 21406) +++ trunk/src/obj_pstk_proto.c (revision 21407) @@ -48,7 +48,7 @@ void pcb_pstk_proto_free_fields(pcb_pstk_proto_t *dst) { -#warning TODO: do a full field free here +TODO("do a full field free here") dst->in_use = 0; } @@ -374,7 +374,7 @@ ly = &dst->Layer[lid]; lyt = pcb_layer_flags_(ly); -#warning layer TODO: make a real scoring mechanism here instead of ly1, ly2, ly3 +TODO("layer: make a real scoring mechanism here instead of ly1, ly2, ly3") /* cheat: pretend boundary layers are mech layers because padstack layer types are interested only in mech layers */ if (lyt & PCB_LYT_BOUNDARY) { lyt &= ~PCB_LYT_BOUNDARY; @@ -873,7 +873,7 @@ int pcb_pstk_proto_change_name(pcb_pstk_proto_t *proto, const char *new_name) { -#warning undo TODO: make this undoable (check how pcb_pstk_proto_change_hole() does it) +TODO("undo: make this undoable (check how pcb_pstk_proto_change_hole() does it)") free(proto->name); if ((new_name == NULL) || (*new_name == '\0')) proto->name = NULL; Index: trunk/src/obj_subc_hash.c =================================================================== --- trunk/src/obj_subc_hash.c (revision 21406) +++ trunk/src/obj_subc_hash.c (revision 21407) @@ -52,7 +52,7 @@ for(lid = 0; lid < sc1->data->LayerN; lid++) { pcb_layer_t *ly1 = &sc1->data->Layer[lid]; pcb_layer_t *ly2 = &sc2->data->Layer[lid]; -#warning TODO: +TODO("todo") /* if (!pcb_layer_eq_bound(ly1, tr1.on_bottom, ly2, tr2.on_bottom)) return 0;*/ if (arclist_length(&ly1->Arc) != arclist_length(&ly2->Arc)) return 0; } @@ -104,7 +104,7 @@ if (!pcb_pstk_eq(&tr1, p1, &tr2, p2)) return 0; p2 = padstacklist_next(p2); } -#warning subc TODO: subc-in-subc eq check +TODO("subc: subc-in-subc eq check") } return 1; @@ -152,7 +152,7 @@ /* hash global objects */ { pcb_pstk_t *ps; -#warning subc TODO: subc in subc: trans in trans +TODO("subc: subc in subc: trans in trans") #if 0 pcb_subc_t *s; subclist_foreach(&sc->data->subc, &it, s) Index: trunk/src/rats_patch.c =================================================================== --- trunk/src/rats_patch.c (revision 21406) +++ trunk/src/rats_patch.c (revision 21407) @@ -253,7 +253,7 @@ case RATP_DEL_CONN: return rats_patch_apply_conn(pcb, patch, 1); case RATP_CHANGE_ATTRIB: -#warning TODO: just check wheter it is still valid +TODO("just check wheter it is still valid") break; } return 0; Index: trunk/src/tool_line.c =================================================================== --- trunk/src/tool_line.c (revision 21406) +++ trunk/src/tool_line.c (revision 21407) @@ -52,7 +52,7 @@ #include "obj_pstk_draw.h" #include "obj_rat_draw.h" -#warning padstack TODO: remove this when via is removed and the padstack is created from style directly +TODO("pstk: remove this when via is removed and the padstack is created from style directly") #include "src_plugins/lib_compat_help/pstk_compat.h" static pcb_layer_t *last_layer; @@ -101,7 +101,7 @@ pcb_crosshair_set_local_ref(pcb_crosshair.X, pcb_crosshair.Y, pcb_true); switch (pcb_crosshair.AttachedLine.State) { case PCB_CH_STATE_FIRST: /* first point */ -#warning subc TODO: this should work on heavy terminals too! +TODO("subc: this should work on heavy terminals too!") if (PCB->RatDraw && pcb_search_screen(pcb_crosshair.X, pcb_crosshair.Y, PCB_OBJ_PSTK | PCB_OBJ_SUBC_PART, &ptr1, &ptr1, &ptr1) == PCB_OBJ_VOID) { pcb_gui->beep(); break; @@ -170,6 +170,7 @@ /* place a via if vias are visible, the layer is in a new group since the last line and there isn't a pin already here */ +TODO("pstk #21: do not work in comp mode, use a pstk proto - scconfig also has TODO #21, fix it there too") if (conf_core.editor.auto_via && PCB->pstk_on && pcb_layer_get_group_(CURRENT) != pcb_layer_get_group_(last_layer) && pcb_search_obj_by_location(PCB_OBJ_CLASS_PIN, &ptr1, &ptr2, &ptr3, @@ -180,7 +181,6 @@ && (pcb_layer_flags_(CURRENT) & PCB_LYT_COPPER) && (pcb_layer_flags_(last_layer) & PCB_LYT_COPPER) && (!PCB->is_footprint) -#warning pdstk TODO #21: do not work in comp mode, use a pstk proto - scconfig also has TODO #21, fix it there too && ((ps = pcb_pstk_new_compat_via(PCB->Data, -1, pcb_crosshair.AttachedLine.Point1.X, pcb_crosshair.AttachedLine.Point1.Y, @@ -261,6 +261,7 @@ /* place a via if vias are visible, the layer is in a new group since the last line and there isn't a pin already here */ +TODO("pstk #21: do not work in comp mode, use a pstk proto - scconfig also has TODO #21, fix it there too") if (PCB->pstk_on && pcb_layer_get_group_(CURRENT) != pcb_layer_get_group_(last_layer) && (pcb_layer_flags_(CURRENT) & PCB_LYT_COPPER) @@ -270,7 +271,6 @@ pcb_crosshair.AttachedLine.Point1.X, pcb_crosshair.AttachedLine.Point1.Y, conf_core.design.via_thickness / 2) == PCB_OBJ_VOID -#warning pdstk TODO #21: do not work in comp mode, use a pstk proto - scconfig also has TODO #21, fix it there too && ((ps = pcb_pstk_new_compat_via(PCB->Data, -1, pcb_crosshair.AttachedLine.Point1.X, pcb_crosshair.AttachedLine.Point1.Y, Index: trunk/src/tool_via.c =================================================================== --- trunk/src/tool_via.c (revision 21406) +++ trunk/src/tool_via.c (revision 21407) @@ -47,7 +47,7 @@ #include "obj_pstk_draw.h" -#warning padstack TODO: remove this when via is removed and the padstack is created from style directly +TODO("padstack: remove this when via is removed and the padstack is created from style directly") #include "src_plugins/lib_compat_help/pstk_compat.h" void pcb_tool_via_notify_mode(void) @@ -63,7 +63,7 @@ return; } -#warning pdstk TODO #21: do not work in comp mode, use a pstk proto - scconfig also has TODO #21, fix it there too +TODO("pstk #21: do not work in comp mode, use a pstk proto - scconfig also has TODO #21, fix it there too") { pcb_pstk_t *ps = pcb_pstk_new_compat_via(PCB->Data, -1, pcb_tool_note.X, pcb_tool_note.Y, conf_core.design.via_drilling_hole, conf_core.design.via_thickness, conf_core.design.clearance, @@ -92,7 +92,7 @@ void pcb_tool_via_draw_attached(void) { -#warning padstack TODO: replace this when route style has a prototype +TODO("pstk: replace this when route style has a prototype") xor_draw_fake_via(pcb_crosshair.X, pcb_crosshair.Y, conf_core.design.via_thickness, 0); if (conf_core.editor.show_drc) { /* XXX: Naughty cheat - use the mask to draw DRC clearance! */