Patch: fix for bug #4436 with associated fixes for stream
reference counting
Dr J A Gow
J.A.Gow at furrybubble.co.uk
Mon Feb 20 14:08:42 CST 2006
Thanks for the comments. Patch now resubmitted as two chunks as suggested.
John.
Mike McCormack wrote:
>
> Dr J A Gow wrote:
>> FROM: Dr J A Gow
>> PATCH: Provides fix for bug 4436, some reference counting fixes, and
>> now the IStorage correctly handles the following:
>> - Stream opening when storage was opened in transacted mode
>> - Stream methods called after parent object has been closed
>> correctly return STG_E_REVERTED
>> - Adds conformance test for stream opening in transacted
>> mode
>
> Looks great, however we usually try to keep seperate fixes seperate so
> that regression testing is easier, and to reduce the size and complexity
> of each patch.
>
> If you could split these two patches into one for the transacted
> storage/stream permissions problem, and one for the reference counting
> problem I'm think they'll be committed without further trouble.
>
> thanks,
>
> Mike
More information about the wine-devel
mailing list