Chad Condon

Results 119 comments of Chad Condon

> A column limit of `0` means that there is no column limit. In this case, clang-format will respect the input’s line breaking decisions within statements unless they contradict other...

> I suggest setting the limit to 180. done

I'm not willing to spend any more time on this. I thought autoformatting would help normalize style and improve the readability of all the long, dense lines, but I don't...

I think what you're looking for is ```yaml AllowShortFunctionsOnASingleLine: Inline ColumnLimit: 0 ```

I'd prefer if we could just [add a badge](https://github.com/microsoft/vcpkg/discussions/17783) instead of duplicating your own docs.

I don't think [this patch](https://github.com/microsoft/vcpkg/blob/master/ports/cpputest/fix-arm-build-error.patch) is necessary since #1629.

GItHub won't show my inline comments for some reason. I suggested, in the README.md additions, to replace ``` * git clone https://github.com/Microsoft/vcpkg.git * cd vcpkg * ./bootstrap-vcpkg.sh # ./bootstrap-vcpkg.bat for...

can/should we add C++20 to the basic CI builds?

The default runners in GitHub seem sufficiently recent: https://github.com/actions/runner-images/blob/main/images/linux/Ubuntu2004-Readme.md#language-and-runtime.

`volatile` itself isn't deprecated; using assignment operators (e.g `*=`, `/=`) on `volatile`s is.