Skip client namespace checks for methods with inline body.

[DutChen18]

The client namespace checks are skipped for functions that have bodies, if FunctionDecl::hasBody() returns true. For the following function, FunctionDecl::hasBody() returns false.

		template<class T, class = std::enable_if_t<cheerp::CanCast<_Function<cheerp::FunctionType<T>>, _Function<F>>>>
		_Function(T&& func) : Function(cheerp::Callback(func)) {
		}

hasInlineBody() does return true, and also skipping the check when hasInlineBody() returns true "fixes" the issue and allows the above code to compile.

I don't really understand why hasBody() returns false here, or why hasInlineBody() does not imply hasBody().

[yuri91]

yeah it's peculiar. Even if there is no "body", there is still code that needs to be generated, since arbitrary stuff can be called in the initializer list...

It's probably because that flag is set from a purely syntactical point of view, which sees the body as empty, even if there is an initializer list