Incorrect order of arguments

[drmeister]

I'm pretty sure there is a bug in the order of arguments passed here:

https://github.com/luabind/luabind/blob/master/src/inheritance.cpp#L193

I think it should be:

m_cache.put(src, target, dynamic_id, object_offset, -1, cache::invalid);

The cache::invalid value only makes sense as a std::ptrdiff_t and the compiler accepts the wrong order because of implicit type casting of integer types.

I've copied a lot of luabind code for my own binding library 'clbind' in clasp (https://github.com/clasp-developers/clasp.git) - a Common Lisp that interoperates with C++.

[keinstein]

As far as I know the signature is

 void cache::put(
      class_id src, class_id target, class_id dynamic_id
    , std::ptrdiff_t object_offset, std::ptrdiff_t offset, int distance)

with

     static std::ptrdiff_t const unknown;
      static std::ptrdiff_t const invalid;

So your suggestion casts -1 to std::ptrdiff_t and cache::invalid from std::ptrdiff_t to int.