[MapServer-dev] Whole code base reformatting

Steve Lime sdlime at gmail.com
Thu Sep 28 11:53:56 PDT 2023


No objections from me. Anything we can do to drive consistency across the
codebase is always welcome. Thanks for taking care of this! Now to adopt
standard naming conventions for variables, functions, etc…

+1

On Thu, Sep 28, 2023 at 9:24 AM Even Rouault via MapServer-dev <
mapserver-dev at lists.osgeo.org> wrote:

> Hi,
>
> Below text from https://github.com/MapServer/MapServer/pull/6937:
>
> I propose to apply to Mapserver the same set of tools that have been
> applied to GDAL and PROJ recently to have consistent code formatting for
> C/C++ and .py files
>
> This uses the pre-commit tool that installs hooks run by git commit to
> automatically apply the formatting rules:
>
> - clang-format for C/C++ code using a .clang-format style based on the
> LLVMStyle with a minimal customization to do 2-space indentation (which
> is the general practice in Mapserver), and not sorting includes (as this
> breaks compilation due to includes unfortunately not being
> clean/standalone, with inconsistencies like "a.h needs a type of b.h,
> but b.h includes a.h"). Third-party code is excluded from reformating
> (in .pre-commit-config.yaml)
>
> - black and isort to format Python code
>
> - flake8 to enforce some rules on Python code (with tons of exceptions
> in .flake8, directly borrowed from the ones of GDAL). I had to manually
> fixup a few things in the .py files
>
> Cf https://gdal.org/development/rfc/rfc69_cplusplus_formatting.html on
> how to install pre-commit (one time operation to do), and
> https://gdal.org/development/dev_practices.html#commit-hooks
>
> A .git-blame-ignore-revs file is also added to identify the reformatting
> commit as to be ignored for the purpose of "git blame". It needs to be
> manually declared to git with "git config blame.ignoreRevsFile
> .git-blame-ignore-revs", for local use (one time operation) but it is
> automatically pickedup by GitHub.
>
> We should probably apply that to the stable branch as well to make
> backports easier.
>
> This also adds a .github/workflows/code_checks.yml GitHub Action that
> runs pre-commit on pull requests to detect violations. Such sitaution
> shouldn't happen for developers that have already set it up locally, but
> useful to catch issues coming from casual contributors (e.g. simulation
> of a commit that would break formatting:
>
> https://github.com/rouault/mapserver/actions/runs/6340096023/job/17220740975
> )
>
> Upon acceptance, I'll add a "Code formatting rules" section to
> https://mapserver.org/development/index.html to gather all above
> information
>
> Even
>
> --
> http://www.spatialys.com
> My software is free, but my time generally not.
>
> _______________________________________________
> MapServer-dev mailing list
> MapServer-dev at lists.osgeo.org
> https://lists.osgeo.org/mailman/listinfo/mapserver-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/mapserver-dev/attachments/20230928/6f4558be/attachment.htm>


More information about the MapServer-dev mailing list