Peter Hill comments

Results 535 comments of


                                            Peter Hill

[F2008] F2008 constructs/statements don't work in blocks

Yes, looks like this works now

Can't parse preprocessed files

This turns out to be basically equivalent to the `#line` directive, but is from the [output of the preprocessor](https://gcc.gnu.org/onlinedocs/gcc-10.2.0/cpp/Preprocessor-Output.html). Playing about with different compilers on godbolt, it looks like this...

Pull out separate functions for reading files in SetupModel.cpp

Rebased to resolve conflicts

Add support for clang-tidy-17 and cmake 3.27

I am a little bit wary of this, because I'd prefer to keep the base image as a LTS OS if possible, as that would give the widest compatibility. We...

Python fixes for running on multiple platforms and outside of docker

@bwrsandman Apologies, I missed this at the time! Will take a look this week.

Python fixes for running on multiple platforms and outside of docker

LGTM, thanks @bwrsandman !

Option to not use line filters

I think in that case, it's probably best to run `clang-tidy` manually

Option to not use line filters

Hmm, that would be useful, yeah. I guess you could turn off posting comments and just have it upload the review as an artefact so that it works on plain...

Investigate clang-format action for ideas on customisable docker base image

I'd really really like to implement this, I just don't have time to investigate properly at the moment! If there's anyone reading this who's interested, please feel free to give...

Errors running clang-tidy itself still gives a green tick in actions

I think other people had requested that it doesn't fail on errors, so maybe that should be a user option