[GBra-4-669]

Fixes #3488.

An Objective-C method that declares C-style parameters is accepted by clang, with
-Wdeprecated-declarations:

- (int)take:(int)t, int extra;

Both parameters land in clang's parameter list, but the selector is still take:. format_method_call
asserted that the two counts agreed. bindgen aborted:

panicked at bindgen/ir/objc.rs:297:9:
assertion `left == right` failed: Incorrect method name or arguments for objc method

msg_send! dispatches on the selector. A method with a parameter count that disagrees with its selector
cannot be represented and emitting it would produce a call with the wrong argument list.
Skip the method and warn instead of aborting.
The crate already does this for other constructs it cannot support (ir/var.rs).

The output for valid Objective-C is unchanged. Across the 19 objc*.h test headers, 18 produce
byte-identical output. The only difference is objc_c_style_params.h, which previously aborted
(exit 101) and now succeeds (exit 0).

Two extra things:

• With the logging feature off, warn! emits nothing. A skipped method is dropped silently.
• The warning names the selector but not the owning class. The same selector declared in two places
  produces two identical warnings.

cargo test -p bindgen-tests: 638 passed, 2 failed. Both fail identically on the base commit.

[emilio]

Fair enough!

View changes since this review