Re: [PATCH v3 0/8] MR4817: shell32: Implement some functions of IFileOperation.

[Esme Povirk (＠madewokherd)]

I'm not sure how much it matters, but I would have split the patches like this:

First, implement PerformOperations along with one other method, so that PerformOperations can do something. Then implement the other methods in separate commits. That way, all of the commits make a functional change, and the related code for each function can be added in each commit. For example, the commit that implements NewItem can also introduce the new_item function. Since the way new_item works depends on the item that NewItem creates, it'd be easier to review them together.