Details
-
Task
-
Status: Closed (View Workflow)
-
Major
-
Resolution: Fixed
-
None
Description
It's a proposal, not a bug.
The main reason to use clang-format: it increases productivity. It will be easy to write code and more importantly to read a well-formatted code afterwards.
clang-format is available on win and *nix.
It has a nice integration with popular editors (e.g. https://github.com/llvm-mirror/clang/blob/master/tools/clang-format/clang-format.el, http://llvm.org/builds/)
It allows to easily enforce some coding style.
It's easy to format whole code base once (producing a huge patch though) and it's possible to add a pre-commit hook to invoke clang-format.
Introductory video:
https://www.youtube.com/watch?v=s7JmdCfI__c
Official documentation:
http://clang.llvm.org/docs/ClangFormat.html
http://clang.llvm.org/docs/ClangFormatStyleOptions.html
/sql config could look like this:
BreakBeforeBraces: Allman
|
SpaceBeforeAssignmentOperators: false
|
/storage/innobase config could look like this:
UseTab: Always
|
TabWidth: 8
|
IndentWidth: 8
|
Attachments
Activity
Field | Original Value | New Value |
---|---|---|
Issue Type | Bug [ 1 ] | Task [ 3 ] |
Assignee | Vicentiu Ciorbaru [ cvicentiu ] |
Assignee | Vicentiu Ciorbaru [ cvicentiu ] | Sergey Vojtovich [ svoj ] |
Component/s | Scripts & Clients [ 11002 ] | |
Fix Version/s | 5.5.65 [ 23402 ] | |
Fix Version/s | 10.1.41 [ 23406 ] | |
Fix Version/s | 10.2.25 [ 23408 ] | |
Fix Version/s | 10.3.16 [ 23410 ] | |
Fix Version/s | 10.4.6 [ 23412 ] | |
Fix Version/s | 10.0.38 [ 23211 ] | |
Resolution | Fixed [ 1 ] | |
Status | Open [ 1 ] | Closed [ 6 ] |
Workflow | MariaDB v3 [ 82238 ] | MariaDB v4 [ 133332 ] |