Re: API Review of adding missing TSTextLogObjectRollingSizeMbSet() function

2014-09-15 Thread Brian Rectanus
On Tue, Sep 9, 2014 at 10:42 AM, James Peach wrote: > > On Sep 9, 2014, at 6:28 AM, Brian Rectanus wrote: > > > On Monday, September 8, 2014, Leif Hedstrom > > wrote: > > > >> > >> On Sep 8, 2014, at 9:43 PM, James Peach wrote: > >> > >>> On Sep 5, 2014, at 2:59 PM, Brian Rectanus wrote: > >>

Re: API Review of adding missing TSTextLogObjectRollingSizeMbSet() function

2014-09-09 Thread James Peach
On Sep 9, 2014, at 6:28 AM, Brian Rectanus wrote: > On Monday, September 8, 2014, Leif Hedstrom > wrote: > >> >> On Sep 8, 2014, at 9:43 PM, James Peach wrote: >> >>> On Sep 5, 2014, at 2:59 PM, Brian Rectanus wrote: >>> All, I have a pull request in that is a trivial patc

API Review of adding missing TSTextLogObjectRollingSizeMbSet() function

2014-09-09 Thread Brian Rectanus
On Monday, September 8, 2014, Leif Hedstrom > wrote: > > On Sep 8, 2014, at 9:43 PM, James Peach wrote: > > > On Sep 5, 2014, at 2:59 PM, Brian Rectanus wrote: > > > >> All, > >> > >> I have a pull request in that is a trivial patch to add a long missing > >> TSTextLogObjectRollingSizeMbSet(int

Re: API Review of adding missing TSTextLogObjectRollingSizeMbSet() function

2014-09-08 Thread Leif Hedstrom
On Sep 8, 2014, at 9:43 PM, James Peach wrote: > On Sep 5, 2014, at 2:59 PM, Brian Rectanus wrote: > >> All, >> >> I have a pull request in that is a trivial patch to add a long missing >> TSTextLogObjectRollingSizeMbSet(int rolling_size_mb) API function to allow >> setting the rolling size o

Re: API Review of adding missing TSTextLogObjectRollingSizeMbSet() function

2014-09-08 Thread James Peach
On Sep 5, 2014, at 2:59 PM, Brian Rectanus wrote: > All, > > I have a pull request in that is a trivial patch to add a long missing > TSTextLogObjectRollingSizeMbSet(int rolling_size_mb) API function to allow > setting the rolling size of custom log objects. This API just wraps the > existing T

API Review of adding missing TSTextLogObjectRollingSizeMbSet() function

2014-09-05 Thread Brian Rectanus
All, I have a pull request in that is a trivial patch to add a long missing TSTextLogObjectRollingSizeMbSet(int rolling_size_mb) API function to allow setting the rolling size of custom log objects. This API just wraps the existing TextLogObject::set_rolling_size_mb(int rolling_size_mb) which is