public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* Put "static" C code snippets in their own runtime C header files
@ 2011-07-29 11:47 Mark Wielaard
  0 siblings, 0 replies; only message in thread
From: Mark Wielaard @ 2011-07-29 11:47 UTC (permalink / raw)
  To: systemtap

Hi,

While working on the stack backtrace cleanup I had to change parts of
the "static" C code that defines the session state and probe context a
couple of times. Although these C snippets weren't dependent on the
current session state they were still "dynamically" created in
translate.cxx. This made editing and documenting some of this pieces a
bit inconvenient. I separated the out the common (static) session state,
probe context and common defines into their own runtime C header files:
runtime/common_probe_context.h, runtime/common_session_state.h and
runtime/runtime_defines.h. So translate.cxx can just output one
"#include" string.

I also tried to add some more documentation for the common probe
context. Please let me know if I misunderstood something or horribly
broke someones setup by doing this.

Thanks,

Mark

commit a34a9fe036c9c6f06452eb893590a6984949dc95
Author: Mark Wielaard <mjw@redhat.com>
Date:   Fri Jul 29 13:35:06 2011 +0200

Put common probe and session context state definitions in their own
files.
    
translate.cxx contained code to produce the common probe and session
context state as C code snippets. Large parts were not dependent on the
session state at all, so they really were just static blobs of C code
wrapped in c++ io stream operators. These have been put in their own
C runtime header files (common_session_state.h, common_probe_context.h
and runtime_defines.h) to make it easier to edit and document.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-07-29 11:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-07-29 11:47 Put "static" C code snippets in their own runtime C header files Mark Wielaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).