Comment 3 for bug 2052578

Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Hi Chris,

Thanks for providing a patch. I noticed that you're suggesting that we move that entire chunk of code from a header file to a specific source file. Without really testing it, I'm thinking whether such a change isn't too invasive. It will mean that other files including back-ldbm.h won't benefit from the code in question.

I also took some time to look if upstream has experienced this problem, but I couldn't find any reports there. But what happens if you reorder the header files on bdb_layer.c instead? I haven't had the time/chance to give it a try, but maybe we can find a simpler approach to this problem.

Thanks.