Revision 392dc4de
Added by Ken McDonell over 13 years ago
- ID 392dc4deeeba11bee4024af9ea55de77eeced3c8
- added
- modified
- copied
- renamed
- deleted
- configure.in (diff)
- src
- cpan
- dbpmda
- include
- GNUmakefile (diff)
- buildrules (diff)
- impl.h (diff)
- platform_defs.h.in
- platform_header.h.in
- platform_trailer.in
- libpcp
- libpcp_gui
- libpcp_mmv
- libpcp_pmda
- libpcp_trace
- pmcd
- pmdas
- pmdumplog
- pmie
- pmlogextract
- pmlogger
- pmlogger.c (diff)
- ports.c (diff)
- preamble.c (diff)
- pmloglabel
- pmlogreduce
- pmproxy
- procmemstat
- win32ctl
Cleanup pid_t usage
A lot of small changes across many source files to cleanup how we
handle pid_t data types and process ids hiding in integer variables.
Driven by compilation warnings on some platforms, but most of these
are latent problems that were just lurking and waiting to bite us in
the future.
Most critical changes is to enlist configure assistance to determine the
correct printf format specification for a value of type pid_t. This
requires splitting platform_dev.in into platform_header.h.in and
platform_trailer.in, and typical printf construct now looks like
printf("... pid=%" FMT_PID " more fmt", ..., pid, ...);
Changes involve making the printf changes above and explicit casting
when int -> pid_t or pid_t -> int conversions are being done ...