[PATCH 2/3] wininet: Use parent/child handle framework to track
parent/child handle relationships.
Misha Koshelev
mk144210 at bcm.edu
Sat Sep 8 18:35:08 CDT 2007
Jacek Caban wrote:
> Hi Misha,
>
> Misha Koshelev wrote:
> > We have a lot of parent/child relationships in wininet that are addressed on an individual basis.
> > Now we can just address them within the framework since we need that framework for closing handles
> > too. The only thing I don't like is that we have to make lots of casts now, but it doesn't really make
> > much sense to me to store the "parent" object handle in two different places.
>
> Sorry for late response.
>
> This patch reverses what I've done some time ago. There is no reason to
> have parent object in header - you need only a child list to handle
> closing handles. Also casts make the code much less readable.
>
> Thanks,
> Jacek
Ok, that sounds fair. I didn't really like the casts either, that was my biggest problem
with this patchset. I am leaving the country tomorrow for 9 days but will change the patches/resubmit
as you suggested after I come back.
Thanks for your comments.
Misha
More information about the wine-devel
mailing list