On 22 Mar 2016, at 14:12, Bhargava Abhyankar
<Bhargava.Abhyankar(a)radisys.com> wrote:
The copy constructor of GprsCodingScheme class object used in
uplink tbf path is replaced by the reference variable.
Appropriate function prototypes and function definition are
changed.
yes, it is good style to mark them as const (and pass them as reference).
This patch will increase efficiency of the code.
Given the size of the member and that the constructor is inlined my first assumption is
that for this specific type the runtime cost of copying m_scheme and a pointer will be
about the same.
kind regards
holger