A coworker noticed this snipped of code in target.c:
case TCFG_COREID:
if (goi->isconfigure) {
e = Jim_GetOpt_Wide(goi, &w);
if (e != JIM_OK)
return e;
target->coreid = (int32_t)w;
} else {
if (goi->argc != 0)
goto no_params;
}
Jim_SetResult(goi->interp, Jim_NewIntObj(goi->interp,
target->working_area_size));
/* loop for more */
break;He remarked that the result should probably be target->coreid instead of target->working_area_size, and I'm inclined to agree. But since I don't really understand this code, nor know how to really test it, I'm not comfortable making the change or even declaring it a real bug. But now at least it's been passed up the chain. If this means something to somebody, it can be fixed. :-) Tim
_______________________________________________ OpenOCD-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openocd-devel
