On 09/08/2016 10:39, Gerry Boland wrote: > Review: Needs Fixing > > +EventBuilder::EventBuilder() > +{ > + m_eventInfoVector.resize(10); > +} > can do: > EventBuilder::EventBuilder() > : m_eventInfoVector(10) > {}
Done.
> There is a lack of tests for EventBuilder.
Added a test.
« Back to merge proposal
On 09/08/2016 10:39, Gerry Boland wrote: :EventBuilder( ) or.resize( 10); :EventBuilder( ) or(10)
> Review: Needs Fixing
>
> +EventBuilder:
> +{
> + m_eventInfoVect
> +}
> can do:
> EventBuilder:
> : m_eventInfoVect
> {}
Done.
> There is a lack of tests for EventBuilder.
Added a test.