[Merge] lp:~morphis/aethercast/add-lttng-support into lp:aethercast

[Jim Hodapp]

Diff comments:

> 
> === added file 'src/mcs/report/lttng/packetizerreport.cpp'
> --- src/mcs/report/lttng/packetizerreport.cpp	1970-01-01 00:00:00 +0000
> +++ src/mcs/report/lttng/packetizerreport.cpp	2016-03-07 13:37:21 +0000
> @@ -0,0 +1,34 @@
> +/*
> + * Copyright (C) 2016 Canonical, Ltd.
> + *
> + * This program is free software: you can redistribute it and/or modify it
> + * under the terms of the GNU General Public License version 3, as published
> + * by the Free Software Foundation.
> + *
> + * This program is distributed in the hope that it will be useful, but
> + * WITHOUT ANY WARRANTY; without even the implied warranties of
> + * MERCHANTABILITY, SATISFACTORY QUALITY, or FITNESS FOR A PARTICULAR
> + * PURPOSE.  See the GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License along
> + * with this program.  If not, see <http://www.gnu.org/licenses/>.
> + *
> + */
> +
> +#include "mcs/report/lttng/packetizerreport.h"
> +
> +#define TRACEPOINT_DEFINE
> +#define TRACEPOINT_PROBE_DYNAMIC_LINKAGE
> +#include "mcs/report/lttng/packetizerreport_tp.h"
> +
> +namespace mcs {
> +namespace report {
> +namespace lttng {
> +
> +void PacketizerReport::PacketizedFrame(const TimestampUs timestamp) {

Unless the type is the same size as a void*, you'll still want to pass by const reference for efficiency and to avoid a copy.

> +    mcs_tracepoint(aethercast_packetizer, packetized_frame, timestamp);
> +}
> +
> +} // namespace lttng
> +} // namespace report
> +} // namespace mcs


-- 
https://code.launchpad.net/~morphis/aethercast/add-lttng-support/+merge/288291
Your team Ubuntu Phablet Team is requested to review the proposed merge of lp:~morphis/aethercast/add-lttng-support into lp:aethercast.