On 10/19/2012 10:00 PM, john skaller wrote: > Judy didn't select the right type for Windows, but the choice is in a > single place. All the rest of the code is correct AFAIK. That may be the case, but I did find some functions and objects that have Word_t variables right next to some signed "long" variables. (The BranchB object in JudyCount.c, for instance.) So in Windows, that's compiling as just a 32-bit int, while Doug & Alan wrote & tested Judy with it as a 64-bit int.
It may be that a 32-bit int is actually enough even when Word_t is 64-bit, and it's somewhat reassuring that you haven't run into any problems in your use, John. (And memory is a concern for me, so I'd love to avoid doubling the size of the member variable ints.) But I'll be more comfortable ensuring that every variable is the same size as when Judy was being designed & tested--unless Doug or Alan remember the nitty-gritties well enough to express confidence that leaving the signed ints as 32-bit shouldn't be a problem. --- Tim Margheim ------------------------------------------------------------------------------ Everyone hates slow websites. So do we. Make your web apps faster with AppDynamics Download AppDynamics Lite for free today: http://p.sf.net/sfu/appdyn_sfd2d_oct _______________________________________________ Judy-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/judy-devel
