Uncompile-able results due to namespacing

[zwimer]

Uncompile-able results due to namespacing

[lyskov]

Thank you for reporting this @zwimer ! I just pushed 7b47e78 which should fix this.

[zwimer]

I'll add this bug to the known bugs PR: #229

[hogabrie]

Hi @lyskov,

7b47e78 causes a new issue for me.

If I add the following to test/T07.class.hpp:

#include <ostream>
namespace operators{
  struct Operators
  {
    friend std::ostream & operator<<(std::ostream & s, const Operators & a) { return s << "Hi"; }
  };
}

, test compilation fails:

/home/ghottiger/git/binder/build/test/T07_class.cpp: In lambda function:
/home/ghottiger/git/binder/build/test/T07_class.cpp:104:105: error: ‘operator<<’ is not a member of ‘operators’; did you mean ‘std::operator<<’?
  104 | ors::Operators const &o) -> std::string { std::ostringstream s; operators::operator<<(s, o); return s.str(); } );
      |                 

Even though this is perfectly ok C++ code. E.g. run this minimal example:

#include <iostream>

namespace operators {
  struct Operators
  {
    friend std::ostream & operator<<(std::ostream & s, const Operators & a) { return s << a.greetings(); }
    std::string greetings() const { return "Hello"; }
  };
}

int main() {
  operators::Operators s;
  std::cout << s << std::endl;
  return 0;
}

[lyskov]

@hogabrie ah, - i had encounter this or very similar issue before, it became particularly obvious when class and friend function is templated (issue is not specific to operator<<). Solution that i used is to separate declaration of function from it definition. It is a bit verbose but so far this is the only solution that seems to work. Please try following example, it have worked for me:

namespace aaaa {
struct T
{
    friend std::ostream & operator<<(std::ostream & s, T const &);
};

inline std::ostream & operator<<(std::ostream & s, T const &) { return s << "hi..."; }
}

[hogabrie]

Thank you @lyskov for the quick support and the amazing binder project. For now I reverted the commit in our fork. When I find the time I will adapt our huge templated library with this proposal to get the bindings working with the latest binder version.