Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Instrumentation to Generated Msg Code #21

Open
1 task
wjwwood opened this issue Jun 15, 2015 · 1 comment
Open
1 task

Add Instrumentation to Generated Msg Code #21

wjwwood opened this issue Jun 15, 2015 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@wjwwood
Copy link
Member

wjwwood commented Jun 15, 2015

It would be useful to have some kind of optional instrumentation on the generated messages so that we could more easily track things like construction, destruction, and coping of the messages. Other kinds of data might be useful as well, but I can't think of any off the top of my head.

AC:

  • Messages generate optional (compile time enabled) mechanism for tracking calls to the message's constructor, destructor, and any copy/move constructors.

Ideally this could be disabled/enabled by the consuming package (like with a #define before including the message's header), and that if you do not use the instrumentation then it does not affect performance at all. But importantly, the enabling/disabling of the instrumentation cannot affect the behavior of the message class.

@wjwwood wjwwood self-assigned this Jun 15, 2015
@wjwwood wjwwood added the ready Work is about to start (Kanban column) label Jun 15, 2015
@gerkey
Copy link
Member

gerkey commented Jul 14, 2015

This is handy as extra proof of correct zero-copy behavior, but for now we'll rely on people understanding how the intraprocess system works (and possibly instrument it somewhere, with manual verification). We can come back to this extra feature later.

@gerkey gerkey removed the ready Work is about to start (Kanban column) label Jul 14, 2015
@mikaelarguedas mikaelarguedas added the enhancement New feature or request label Aug 7, 2017
@clalancette clalancette modified the milestone: untargeted Feb 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants