efd71aad4f
Cleanup: clang-tidy suppress warnings for PyTypeObject.tp_print
...
Clang-tidy behavior changes from Python 3.7 to 3.8+ so it's simplest
to suppress the warning in this instance.
2020-11-10 09:40:20 +11:00
e118426e46
Fix T82520: error building freestyle with Python3.8
...
Caused by 16732def37
2020-11-09 16:08:29 +11:00
8c846cccd6
Cleanup: clang-format
2020-11-09 15:47:08 +11:00
16732def37
Cleanup: Clang-Tidy modernize-use-nullptr
...
Replace `NULL` with `nullptr` in C++ code.
No functional changes.
2020-11-06 18:08:25 +01:00
af35ada2f3
Cleanup: Clang-Tidy, modernize-use-bool-literals
2020-11-06 14:32:51 +01:00
aa3a4973a3
Cleanup: use ELEM macro
2020-11-06 12:32:54 +11:00
Aaron Carlisle
baa24f1c91
Pydoc: Fix sphinx compile warnings about freestyle
...
Sphinx expects functions and methods with the same name and different
parameters to be written using one directive. See:
https://www.sphinx-doc.org/en/master/usage/restructuredtext/domains.html#basic-markup
Unfortunately this makes giving different descriptions for each harder.
This was already a request for better support for this in sphinx, see:
https://github.com/sphinx-doc/sphinx/issues/7787
Reviewed By: campbellbarton
Differential Revision: https://developer.blender.org/D9170
2020-10-22 17:22:05 -04:00
23f1dea440
Cleanup: spelling
2020-09-09 13:30:05 +10:00
326e137a8c
Cleanup: Freestyle, Clang-Tidy else-after-return fixes
...
This addresses warnings from Clang-Tidy's `readability-else-after-return`
rule in the `source/blender/freestyle` module.
No functional changes.
2020-08-07 13:38:06 +02:00
7283e6fb73
Merge branch 'blender-v2.90-release' into master
2020-08-07 10:04:57 +02:00
91694b9b58
Code Style: use "#pragma once" in source directory
...
This replaces header include guards with `#pragma once`.
A couple of include guards are not removed yet (e.g. `__RNA_TYPES_H__`),
because they are used in other places.
This patch has been generated by P1561 followed by `make format`.
Differential Revision: https://developer.blender.org/D8466
2020-08-07 09:50:34 +02:00
nutti
c442ff0017
Fix RST syntax for freestyle documentation
2020-08-02 18:12:22 +10:00
901ee66ea1
Cleanup: use term init instead of initialize/initialise
...
The abbreviation 'init' is brief, unambiguous and already used
in thousands of places, also initialize is often accidentally
written with British spelling.
2020-08-01 13:51:05 +10:00
a3c6304a27
Cleanup: quiet possible comma misuse warning in feestyle PyAPI
2020-07-29 11:00:32 +10:00
nutti
55c6cc46b5
Fix RST syntax for freestyle documentation
2020-07-29 09:49:08 +10:00
f891d4e2ad
Clang-Tidy: Fix readability-delete-null-pointer warnings
...
Also enable it in the configuration.
2020-07-03 16:32:51 +02:00
3aa1143d57
Cleanup: redundant parenthesis
2020-06-18 14:30:04 +10:00
2d1cce8331
Cleanup: make format after SortedIncludes change
2020-03-19 09:33:58 +01:00
d119e163d0
Fix many typos and other issues in UI messages.
2020-02-17 13:01:07 +01:00
9a9f39e466
Cleanup: remove redundant 'char *' casts
2019-12-20 10:42:57 +11:00
2e6159a494
Curve: CurveMapping Extend Option
...
Extend options are currently stored per curve. This was not clearly
communicated to the user and they expected this to be a setting per
CurveMapping.
This change will move the option from `Curve` to `CurveMapping`. In
order to support this the API had to be changed.
BPY: CurveMap.evaluate is also moved to CurveMapping.evaluate what
breaks Python API. Cycles has been updated but other add-ons have
not. After release of 2.81 we can merge this to master and adapt
the add-ons.
Reviewed By: campbellbarton
Differential Revision: https://developer.blender.org/D6169
2019-11-27 16:05:54 +01:00
00cb31de65
Cleanup: use BKE_ prefix for BKE_colortools.h
2019-08-07 03:38:01 +10:00
760dbd1cbf
Cleanup: misc spelling fixes
...
T68035 by @luzpaz
2019-08-01 14:02:41 +10:00
604fdb6e85
Spelling fixes in comments and descriptions, patch by luzpaz
...
Differential Revision: https://developer.blender.org/D3744
2019-07-31 14:27:35 +02:00
e927ce8acb
Cleanup: avoid line breaks from trailing comments
2019-07-10 14:41:19 +10:00
163996b681
Cleanup: move comments onto own lines to avoid breaking lines
2019-07-02 22:17:22 +10:00
d8dbd49a2f
Cleanup: style, use braces in source/
...
Automated using clang-tidy.
2019-05-31 22:55:15 +10:00
1e8697cd80
Cleanup: comments (long lines) in freestyle
2019-05-01 07:58:27 +10:00
4439e5d0ba
Cleanup: add trailing commas to avoid right shift
2019-04-18 17:19:44 +02:00
e12c08e8d1
ClangFormat: apply to source, most of intern
...
Apply clang format as proposed in T53211.
For details on usage and instructions for migrating branches
without conflicts, see:
https://wiki.blender.org/wiki/Tools/ClangFormat
2019-04-17 06:21:24 +02:00
dbc058301b
Cleanup: trailing commas
2019-04-16 16:40:47 +02:00
de13d0a80c
doxygen: add newline after \file
...
While \file doesn't need an argument, it can't have another doxy
command after it.
2019-02-18 08:22:12 +11:00
eef4077f18
Cleanup: remove redundant doxygen \file argument
...
Move \ingroup onto same line to be more compact and
make it clear the file is in the group.
2019-02-06 15:45:22 +11:00
744f633986
Cleanup: trailing commas
...
Needed for clan-format not to wrap onto one line.
2019-02-03 14:59:11 +11:00
65ec7ec524
Cleanup: remove redundant, invalid info from headers
...
BF-admins agree to remove header information that isn't useful,
to reduce noise.
- BEGIN/END license blocks
Developers should add non license comments as separate comment blocks.
No need for separator text.
- Contributors
This is often invalid, outdated or misleading
especially when splitting files.
It's more useful to git-blame to find out who has developed the code.
See P901 for script to perform these edits.
2019-02-02 01:36:28 +11:00
bd5394f6c0
Fix T57890: freestyle rendering crash.
...
Depsgraph objects are deleted after conversion to Freestyle data structures,
so don't keep a pointer to their name strings.
2019-01-04 17:58:55 +01:00
6d89337257
Fix/cleanup typos and such in UI messages (and some comments).
2018-12-24 15:03:49 +01:00
2083a7e274
Cleanup: style (pointers)
2018-10-11 09:03:39 +11:00
39eb051616
Cleanup: indentation
2018-08-30 01:31:20 +10:00
410880552b
Cleanup: trailing space for freestyle
2018-06-17 17:05:14 +02:00
2e2e6e3bdb
Cleanup: Use BKE_colorband prefix
2017-12-07 15:52:59 +11:00
cc811d1fd6
Cleanup: extract BKE_colorband from BKE_texture
2017-12-07 15:40:11 +11:00
a2a7316d92
Fix T48366: Freestyle will unnecessary exclude some linked objects.
...
Group membership testing for including/excluding feature lines was not
accounting for object names possibly further qualified by library file
paths.
Also fixed a few potential (but unlikely) references of uninitialized
variables.
A big thank to Bastien Montagne for the insight on the cause of the
problem and how to fix it.
2016-08-05 22:25:16 +09:00
9f5621bb4a
Cleanup: comment blocks
2016-07-02 10:08:33 +10:00
2465bd90d5
Cleanup: style, whitespace, doxy filepaths
2016-06-19 06:33:29 +10:00
a269287f36
Cleanup: use single struct for freestyle globals
...
Was shadowing local vars.
2015-11-23 17:40:09 +11:00
52d074ba39
Fix T46604: Crash in ChainPredicateIterator instantiated without predicates.
...
Also fixed a potential crash in the copy constructor case.
2015-10-26 20:55:37 +09:00
5fe9b36aa6
Cleanup: style
2015-09-02 18:39:19 +10:00
be980c4ee4
Freestyle: minor optimization for space by using a pointer to a const char array instead of std::string.
2015-07-20 06:17:50 +09:00
ecb3e0fe73
Cleanup: whitespace & break placement
2015-07-18 19:03:22 +10:00
