Code review comment for lp:~tmaesaka/drizzle/drizzle-fix-drizzleslap

Revision history for this message
Toru Maesaka (tmaesaka) wrote :

Jay, Stewart

Hi!

Fair enough I'll get rid of my_open() and resubmit the patch.

> Toru, I can tell you didn't want to go back to my_close()...is there another
> reason you did that?

What I had in mind was to refactor drizzleslap later on and get rid of mysys dependencies then . But I guess I was just being lazy :(

Cheers,
Toru

> Hi Toru!
>
> I think we've been trying to get rid of calls to my_open() and my_close() in
> favour of the standard open()/close() methods... so I'd prefer to see the
> calls to my_open() disappear than going back to my_close() :)
>
> Toru, I can tell you didn't want to go back to my_close()...is there another
> reason you did that?
>
> -jay

« Back to merge proposal