Bindings to overloaded functions aren't supported anymore

[fitzgen]

The old servo/rust-bindgen supported them by appending 1, 2, 3, ... to each overload, but the new one does not.

I'll make a fix.

Test case (this should generate bindings for each overload):

struct JSContext;
struct ReadOnlyCompileOptions;
struct SourceBufferHolder;
struct AutoObjectVector;
struct char16_t;

#define JS_PUBLIC_API(t) t

namespace JS {

struct MutableHandleValue {};

/**
 * Evaluate the given source buffer in the scope of the current global of cx.
 */
extern JS_PUBLIC_API(bool)
Evaluate(JSContext* cx, const ReadOnlyCompileOptions& options,
         SourceBufferHolder& srcBuf, JS::MutableHandleValue rval);

/**
 * As above, but providing an explicit scope chain.  envChain must not include
 * the global object on it; that's implicit.  It needs to contain the other
 * objects that should end up on the script's scope chain.
 */
extern JS_PUBLIC_API(bool)
Evaluate(JSContext* cx, AutoObjectVector& envChain, const ReadOnlyCompileOptions& options,
         SourceBufferHolder& srcBuf, JS::MutableHandleValue rval);

/**
 * Evaluate the given character buffer in the scope of the current global of cx.
 */
extern JS_PUBLIC_API(bool)
Evaluate(JSContext* cx, const ReadOnlyCompileOptions& options,
         const char16_t* chars, int length, JS::MutableHandleValue rval);

/**
 * As above, but providing an explicit scope chain.  envChain must not include
 * the global object on it; that's implicit.  It needs to contain the other
 * objects that should end up on the script's scope chain.
 */
extern JS_PUBLIC_API(bool)
Evaluate(JSContext* cx, AutoObjectVector& envChain, const ReadOnlyCompileOptions& options,
         const char16_t* chars, int length, JS::MutableHandleValue rval);

/**
 * Evaluate the given byte buffer in the scope of the current global of cx.
 */
extern JS_PUBLIC_API(bool)
Evaluate(JSContext* cx, const ReadOnlyCompileOptions& options,
         const char* bytes, int length, JS::MutableHandleValue rval);

/**
 * Evaluate the given file in the scope of the current global of cx.
 */
extern JS_PUBLIC_API(bool)
Evaluate(JSContext* cx, const ReadOnlyCompileOptions& options,
         const char* filename, JS::MutableHandleValue rval);

} // namespace JS

But we only get bindings for the first:

/* automatically generated by rust-bindgen */

extern "C" {
    /**
 * Evaluate the given source buffer in the scope of the current global of cx.
 */
    #[link_name =
          "_ZN2JS8EvaluateEP9JSContextRK22ReadOnlyCompileOptionsR18SourceBufferHolderNS_18MutableHandleValueE"]
    pub fn Evaluate(cx: *mut JSContext,
                    options: *const ReadOnlyCompileOptions,
                    srcBuf: *mut SourceBufferHolder, rval: MutableHandleValue)
     -> bool;
}

[emilio]

Fix should be easy, we should store symbols instead of names in the functions seen we track in codegen.