I have merged the change into master. I forgot to reference the 'CID-123719' in the commit note though.
*Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Fri, Sep 5, 2014 at 10:02 AM, Will Stevens <wstev...@cloudops.com> wrote: > Ok, thanks... > > ws > > > *Will STEVENS* > Lead Developer > > *CloudOps* *| *Cloud Solutions Experts > 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 > w cloudops.com *|* tw @CloudOps_ > > > On Fri, Sep 5, 2014 at 10:02 AM, Daan Hoogland <daan.hoogl...@gmail.com> > wrote: > >> go ahead and cherry-pick I will merge in 4.4. >> >> >> On Fri, Sep 5, 2014 at 3:52 PM, Will Stevens <wstev...@cloudops.com> >> wrote: >> >>> Yes, I am fine with that change. Should I add it to master? >>> >>> Cheers, >>> >>> Will >>> >>> >>> *Will STEVENS* >>> Lead Developer >>> >>> *CloudOps* *| *Cloud Solutions Experts >>> 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 >>> w cloudops.com *|* tw @CloudOps_ >>> >>> >>> On Fri, Sep 5, 2014 at 8:54 AM, Daan Hoogland <daan.hoogl...@gmail.com> >>> wrote: >>> >>>> H Will, coverity complained about a few lines of yours. please have a >>>> look at hotfix/4.4-CID1237195 and tell me you agree or not. I think it >>>> should go in master as well. >>>> >>>> -- >>>> Daan >>>> >>> >>> >> >> >> -- >> Daan >> > >
