public inbox for gdb-patches@sourceware.org
 help / color / mirror / Atom feed
* [PATCH 0/3] Convert probe interfaces to C++ (and perform some cleanups)
@ 2017-11-13 17:59 Sergio Durigan Junior
  2017-11-13 17:59 ` [PATCH 3/3] Convert DTrace probe interface " Sergio Durigan Junior
                   ` (3 more replies)
  0 siblings, 4 replies; 23+ messages in thread
From: Sergio Durigan Junior @ 2017-11-13 17:59 UTC (permalink / raw)
  To: GDB Patches; +Cc: Sergio Durigan Junior

This patch series performs the conversion of the probe interfaces
(generic, stap and dtrace) to C++, along with a few cleanups here and
there to make the code conform better to our standards.

This series needs to be committed/tested as one single patch, because
of inter-dependencies.  However, in order to facilitate the review
process, I decided to split things into logical units.

The main changes are the conversion of 'struct probe' to 'class
probe', and 'struct probe_ops' to 'class static_probe_ops'.  Almost
everything else in the patches are adjustments related to these
modifications.

It's important to say that 'class probe' contains the majority of the
methods that were previously living inside 'struct probe_ops'.
However, some methods are special in the sense the they don't act on a
specific probe object, but generically on a probe type (like a
factory).  For that reason the 'class static_probe_ops' had to be
created.

Most methods in 'class probe' are pure virtual and need to be defined
by the probe backend (stap or dtrace, for now).  Other methods (e.g.,
the 'enable' method) are virtual but non-pure and have a default,
dummy version implemented on 'class probe' itself, because the concept
of enabling a probe is not common to all types of probes.

I've also taken the opportunity to convert a few uses of "VEC" to
"std::vector", and to remove annoying spurious newlines from some
places.

The whole patch has been tested on BuildBot, without regressions.

^ permalink raw reply	[flat|nested] 23+ messages in thread
* [PATCH 0/3] Convert probe interfaces to C++ (and perform some cleanups)
@ 2017-11-13 17:53 Sergio Durigan Junior
  2017-11-13 18:02 ` [PATCH 3/3] Convert DTrace probe interface " Sergio Durigan Junior
  0 siblings, 1 reply; 23+ messages in thread
From: Sergio Durigan Junior @ 2017-11-13 17:53 UTC (permalink / raw)
  To: GDB Patches; +Cc: Sergio Durigan Junior

This patch series performs the conversion of the probe interfaces
(generic, stap and dtrace) to C++, along with a few cleanups here and
there to make the code conform better to our standards.

This series needs to be committed/tested as one single patch, because
of inter-dependencies.  However, in order to facilitate the review
process, I decided to split things into logical units.

The main changes are the conversion of 'struct probe' to 'class
probe', and 'struct probe_ops' to 'class static_probe_ops'.  Almost
everything else in the patches are adjustments related to these
modifications.

It's important to say that 'class probe' contains the majority of the
methods that were previously living inside 'struct probe_ops'.
However, some methods are special in the sense the they don't act on a
specific probe object, but generically on a probe type (like a
factory).  For that reason the 'class static_probe_ops' had to be
created.

Most methods in 'class probe' are pure virtual and need to be defined
by the probe backend (stap or dtrace, for now).  Other methods (e.g.,
the 'enable' method) are virtual but non-pure and have a default,
dummy version implemented on 'class probe' itself, because the concept
of enabling a probe is not common to all types of probes.

I've also taken the opportunity to convert a few uses of "VEC" to
"std::vector", and to remove annoying spurious newlines from some
places.

The whole patch has been tested on BuildBot, without regressions.

^ permalink raw reply	[flat|nested] 23+ messages in thread

end of thread, other threads:[~2017-11-23  0:33 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-13 17:59 [PATCH 0/3] Convert probe interfaces to C++ (and perform some cleanups) Sergio Durigan Junior
2017-11-13 17:59 ` [PATCH 3/3] Convert DTrace probe interface " Sergio Durigan Junior
2017-11-15  4:40   ` Simon Marchi
2017-11-16  4:11     ` Sergio Durigan Junior
2017-11-13 17:59 ` [PATCH 1/3] Convert generic " Sergio Durigan Junior
2017-11-15  2:52   ` Simon Marchi
2017-11-15  3:25     ` Simon Marchi
2017-11-15  6:15       ` Sergio Durigan Junior
2017-11-15  6:12     ` Sergio Durigan Junior
2017-11-13 17:59 ` [PATCH 2/3] Convert SystemTap " Sergio Durigan Junior
2017-11-15  3:58   ` Simon Marchi
2017-11-15 22:49     ` Sergio Durigan Junior
2017-11-16  4:37 ` [PATCH v2 0/3] Convert probe interfaces " Sergio Durigan Junior
2017-11-16  4:38   ` [PATCH v2 2/3] Convert SystemTap probe interface " Sergio Durigan Junior
2017-11-16  4:38   ` [PATCH v2 3/3] Convert DTrace " Sergio Durigan Junior
2017-11-16  4:38   ` [PATCH v2 1/3] Convert generic " Sergio Durigan Junior
2017-11-22 20:39     ` Simon Marchi
2017-11-22 22:36       ` Sergio Durigan Junior
2017-11-23  0:01         ` Simon Marchi
2017-11-23  0:15           ` Sergio Durigan Junior
2017-11-23  0:33             ` Sergio Durigan Junior
2017-11-21 16:25   ` [PATCH v2 0/3] Convert probe interfaces " Sergio Durigan Junior
  -- strict thread matches above, loose matches on Subject: below --
2017-11-13 17:53 [PATCH " Sergio Durigan Junior
2017-11-13 18:02 ` [PATCH 3/3] Convert DTrace probe interface " Sergio Durigan Junior

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).