Hi all,

Last night, I committed patches to hide the definition of the
target_type_s structure from out-of-module users.  This was done over
the course of a few series of patches.  I apologize if any problems crop
up, but I will report my own status in a new thread following this one.
Suffice it to say that I believe it unlikely there will be problems.

The first and second series of patches added trivial target_* wrappers
for its fields and callback functions.  They culminated in the removal
of the structure definition and replacement with a forward declaration,
unless the DEFINE_TARGET_TYPE_S symbol is defined by the source file
that requires it.

The third "series" turned out to be one patch to move the structure into
its own header file, which allows it to be documented without bloating
the main target.h header.  That header will remain fairly hefty (after
being documented), even after removing this particular fatty lump.

As this sentiment suggests, I was tempted to make some other changes
along the way to clean up the target APIs, but I wanted to remain true
to the intent of the line of development (indicated in the Subject line
of this message).  Now that most of that work is done, I will post a new
thread to list the things that bothered me when I was knee-deep in code.

Cheers,

Zach

_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to