SMB3 Multi-Channel (Re: What I'm working on)
Michael Adam
obnox at samba.org
Tue Aug 5 06:09:37 MDT 2014
On 2014-08-05 at 14:03 +0200, Stefan (metze) Metzmacher wrote:
> Hi Volker,
>
> >>>> Attached find a diff between the old and new patchset
> >>>> together with the new patchset.
> >>>>
> >>>> If you're fine with the changes we can finally push them.
> >>>
> >>> Reviewing the 85 patches piece by piece a third time now. I
> >>> wonder how many times this in the end will have taken.
> >>
> >> Why? Just look at cmp.diff and the commits which still have
> >> the TODO/REVIEW.
> >>
> >> Thanks for looking at it!
> >
> > This time it certainly went quicker. Sometimes the diff of a
> > diff can be pretty confusing, so I hope I did not miss a
> > patch in the attached patchset.
>
> Yes, a diff of a diff is mostly useful to to verify that nothing changed.
> Or to have an indication where something changed to reduce the further work.
>
> If you have ideas how to make such a review easier I'd like to hear them:-)
>
> > In the current version of this patchset I don't see any problems anymore,
> > but I am not good at reviewing such a large patchset. So before pushing
> > you might ask someone else to also take a look.
>
> Jeremy, Michael: Do you want to have a look and push this?
Yes, I will start to do it this afternoon.
Cheers - Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20140805/09b14535/attachment.pgp>
More information about the samba-technical
mailing list