Use clang-format to format API code about wasm-c-api HOT 5 OPEN

Yeah, I understand the suggestion, but I'm sorry to say that my prior exposure has elicited an allergy to clang-format. Symptoms are cringing pain and curling toe nails when exposed to algorithmic stupor steam-rolling over aesthetics. :)

Is the amount of code contributions to expect for this repo large enough that formatting would likely be a problem?

from wasm-c-api.

Yeah, I understand the suggestion, but I'm sorry to say that my prior exposure has elicited an allergy to clang-format. Symptoms are cringing pain and curling toe nails when exposed to algorithmic stupor steam-rolling over aesthetics. :)

I understand your feeling, and after using clang-format for a while on WAVM, am still disappointed with the way it formats certain things. But it's also liberating to accept the automatic formatting and focus on more important issues.

Is the amount of code contributions to expect for this repo large enough that formatting would likely be a problem?

I don't know, but this repo will need to continue to evolve with WebAssembly.

from wasm-c-api.

I understand your feeling, and after using clang-format for a while on WAVM, am still disappointed with the way it formats certain things. But it's also liberating to accept the automatic formatting and focus on more important issues.

Big +1 to this.

Is the amount of code contributions to expect for this repo large enough that formatting would likely be a problem?

For me this is the only thing that matters; making collaboration easier is much more important than any one person's sense of aesthetics. Obviously, whether formatting is a "problem" could also be subjective; but the cost is small, and adding formatting early is easier than doing it later (e.g. in response to disagreement) when there's a lot of existing code.

from wasm-c-api.

Could you upload a PR with the changes that result for formatting with BasedOnStyle: Chromium so we can see how intrusive it would be?

from wasm-c-api.

Could you upload a PR with the changes that result for formatting with BasedOnStyle: Chromium so we can see how intrusive it would be?

@sbc100 See #115.

from wasm-c-api.