Rename PanelType* variables from pt to panel_type
Details
Diff Detail
- Repository
- rB Blender
- Branch
- Rename_PanelType_variables (branched from master)
- Build Status
Buildable 21409 Build 21409: arc lint + arc unit
Event Timeline
| source/blender/editors/interface/interface.c | ||
|---|---|---|
| 4400–4403 | panel_type cannot be both const char* and PanelType you have to rename one of these two variables. | |
| source/blender/editors/include/UI_interface.h | ||
|---|---|---|
| 2798 | Since "panel_type" is much longer than "pt" some long lines like this need reformatting. Clang format can take care of that for you: https://wiki.blender.org/wiki/Tools/ClangFormat People sometimes forget to run clang format before committing, so it can happen that areas you haven't touched get reformatted as well. If that happens, make sure to not include those changes in your patch - they will be taken care of by someone in a cleanup commit. ;) | |
| source/blender/editors/include/UI_interface.h | ||
|---|---|---|
| 2798 | In the new developers' guide it is written that the clang format is already available in the c/c++ tools in vscode, and installing the separate extension is the old procedure and not needed now, which is the method specified in the link you have suggested. I am already using clang format and format on save in vscode, and the formatting seemed alright to me. If there is anything wrong with my setup, please guide me on how to fix it. | |
| source/blender/editors/include/UI_interface.h | ||
|---|---|---|
| 2798 | Well, applying your patch and running make format in the console changes a bunch of stuff: P2879 (Not only your stuff, btw...) Since I don't use vscode a lot, I unfortunately can't help you much, except suggest to recheck, if you setup everything according to the description you mentioned. I wasn't aware that the setup for clang format changed. So that's good to know, thanks! | |