Code

clang-format implementation#1740

Code

Merged

Ralim merged 45 commits intoRalim:devRalim/IronOS:devfrom ia:clang-formatia/IronOSf:clang-formatCopy head branch name to clipboardJul 16, 2023

+1,555-1,359

Commits

Commits on Jul 11, 2023

• Testing clang-format style check using github CI

  ia

  committed
  94c2f43

• github/push: implement check-style for clang-format as a separate build step

  ia

  committed
  b24d945

• github/push: add missing packages for check-style/clang-format build step

  ia

  committed
  1103b50

Commits on Jul 12, 2023

• source/Makefile: check-style - reduce files of interest; update .clang-format to keep enums init

  ia

  committed
  9bd3501

• source/Makefile: empty lines, spaces & tabs refactoring to unify style - part 1 out of N

  ia

  committed
  6306e9b

• source/Makefile: fix formatting for multi-line variables

  ia

  committed
  5bb5afe

• source/Makefile: update formatting for multi-line variables

  ia

  committed
  5a0f346

• source/Makefile: remove spaces on vars assignments to unify style

  ia

  committed
  38bea98

• source/Makefile: remove unused target style

  ia

  committed
  703498d

• source/Makefile: implement exclude vars for clang-format related files

  ia

  committed
  20e2da7

• source/Makefile: exclude configuration.h from clang-format check

  ia

  committed
  af0200a

• Dockerfile: add diffutils in a container to make check-style target using advanced version of diff to get more advanced output to parse & navigate log more easily

  ia

  committed
  96e7c99

• source/Makefile: implement parser for clang-format inside check-style target to make output compatible with gcc-like error compilation format for compatibility with IDEs/editors for easy navigation…

  Show description for a1ef959
  ia

  committed
  a1ef959

• source/Makefile: probably final touches on unifying style

  ia

  committed
  60cd7a6

• source/Makefile: implement check-style-list target to only list affected file names with wrong code style for debug purposes

  ia

  committed
  a60ca8b

• source/Makefile: fix missed spaces

  ia

  committed
  680c781

• deploy.sh: add helper routine to deal with clang-format error output logging from makefile

  ia

  committed
  791fec8

• gitignore: add clang-format log explicitly

  ia

  committed
  baa709a

• Refactoring for clang-format compiance

  ia

  committed
  ad50dc2

• Dockerfile: add sed

  ia

  committed
  ec60601

• Dockerfile: false alarm - remove sed since busybox-sed seems fine

  ia

  committed
  4c03e06

• source/Makefile: reduce calls of clang-format & make error log more clean, clear, and tidy

  ia

  committed
  3560e8c

• deploy.sh:check_style() - add removal of DOS EOLs for generated log

  ia

  committed
  7df60b0

Commits on Jul 13, 2023

• source/Makefile:check-style: add more empty lines between blocks with errors for readability when suggestion is too long & heavy

  ia

  committed
  18ce274

• source/Makefile: add STOP var to check-style for exit on first failed file

  ia

  committed
  396f082

• source/Makefile: check-style: make log looks more like traditional diff/patch output

  ia

  committed
  a39dab4

• source/Core/BSP/Pinecilv2/MemMang/heap_5.c: clang-format refactoring using reasonable advises ... and then disable it in Makefile from scanning by clang-format

  ia

  committed
  d0f87e3

• Return headers include order

  ia

  committed
  3058f6e

• clang-format config: disable warnings about non-alphabetic include order

  ia

  committed
  5365168

• clang-format refactoring

  ia

  committed
  15970a9

• clang-format refactoring, part 2

  ia

  committed
  ab1a7b3

• clang-format refactoring, part 3

  ia

  committed
  2c3c447

• settingsGUI.cpp: refactoring, part 1

  ia

  committed
  8c387a8

• settingsGUI.cpp: refactoring, part 2

  ia

  committed
  1e00a5b

• settingsGUI.cpp: refactoring, part 3

  ia

  committed
  5204478

• settingsGUI.cpp: refactoring, part 4

  ia

  committed
  b30722d

• clang-format should be happy now

  ia

  committed
  da5534d

• workflows/push: put readme check into separate build step & update style

  ia

  committed
  7daf912

• clang-format: giving SortIncludes option second chance by tweaking a couple of headers a bit

  ia

  committed
  ff8df64

Commits on Jul 14, 2023

• source/Makefile: check-style: add homebrew parser to check for { } in conditional blocks

  ia

  committed
  7f8e70e

• homebrew-format: add { } for if/else, while, and for & unify some comments style; left two errors intentionally to debug & improve parser

  ia

  committed
  f53b7db

• source/Makefile: homebrew-format: fix false negative trigger for multi-line condition in if-s

  ia

  committed
  970f3c0

• Sleep.cpp: unify style & comments

  ia

  committed
  a492c3c

• source/Makefile: remove unused debug target

  ia

  committed
  382ae15

Commits on Jul 15, 2023

• Merge dev branch to clang-format branch

  ia

  committed
  a63d5d0