Welcome to mirror list, hosted at ThFree Co, Russian Federation.

git.blender.org/blender.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
path: root/source/blender/editors/curve/editcurve.c
AgeCommit message (Collapse)Author
2020-08-21Fix T79987: Crash when joining objectsJulian Eisel
Mistake in b077de086e14. I did the same fix for a few operators there, but missed the object "Join" one. The joining operator changes the layer content. So it must send a notifier for that. Before b077de086e14 that didn't cause a noticeable issue, because the Outliner happened to listen to other notifiers (active/selection changes) the operator sent and fully rebuilt its tree in response. Now missing these notifiers can be more problematic, since we try to avoid more rebuilds. Added comments to the notifier types to avoid at least this pitfall.
2020-08-08Cleanup: use array syntax for sizeof with fixed valuesCampbell Barton
Also order sizeof(..) first to promote other values to size_t.
2020-08-07Cleanup: pass arrays const where possibleCampbell Barton
2020-08-01Cleanup: use term init instead of initialize/initialiseCampbell Barton
The abbreviation 'init' is brief, unambiguous and already used in thousands of places, also initialize is often accidentally written with British spelling.
2020-07-20UI: use "Recalculate" instead of "Recalc"Yevgeny Makarov
2020-07-14Fix T78883: New bezier curve points to uninitialized memoryCampbell Barton
2020-07-03Cleanup: Editors, Clang-Tidy else-after-return fixesSybren A. Stüvel
This addresses warnings from Clang-Tidy's `readability-else-after-return` rule in the `source/blender/editors` module. No functional changes.
2020-06-17Cleanup: name mesh join functions using ED_{type} prefixCampbell Barton
2020-06-16Cleanup: use explicit enum type for duplicate option of `BKE_object_duplicate`Bastien Montagne
Using enum type itself in implementations, and uint in headers (as using enums types in headers is a pain when enum are not defined and used in a single same header file...).
2020-06-05Code Cleanup: fcurve function namingJeroen Bakker
2020-05-27Cleanup: remove unused Main struct from snap contextCampbell Barton
2020-05-26Curves: Implement Handles for selected points onlyAntonio Vazquez
When editing a complex curve is very annoying to have all handles at a time. Also, this is a requirement for the current GSoC Edit Grease Pencil using curves. I have seen that this improvement can be used in any other area of blender, so I have decided to publish the option in the overlay panel.. Reviewed By: fclem, #user_interface, billreynish, Severin Differential Revision: https://developer.blender.org/D7754
2020-05-02Merge branch 'blender-v2.83-release'Nathan Craddock
2020-05-02Revert "Outliner: Fix selection sync for various operators"Nathan Craddock
This reverts commit 92d62148bed1cf68ed674b4a9c6ba6440a697ca8. When merging in from blender-v2.83-release the merge was somehow rebased after viewing the log.
2020-05-02Outliner: Fix selection sync for various operatorsNathan Craddock
Add missing outliner selection sync tagging for various non-outliner operators. * Curve separate * Grease Pencil separate * Mesh separate * Make instances real * 3D view paste * Sequencer paste * Armature delete, dissolve, separate, duplicate, subdivide, extrude, click extrude, primitive add * Pose Group select, delete Resolves T71404
2020-05-02Outliner: Fix selection sync for various operatorsNathan Craddock
Add missing outliner selection sync tagging for various non-outliner operators. * Curve separate * Grease Pencil separate * Mesh separate * Make instances real * 3D view paste * Sequencer paste * Armature delete, dissolve, separate, duplicate, subdivide, extrude, click extrude, primitive add * Pose Group select, delete Resolves T71404
2020-05-02Fix T76062: Interpolate Radius in Curve Subdivide Special CaseHans Goudey
Differential Revision: https://developer.blender.org/D7523
2020-04-14Fix T75733: Curve extrusion does not include endpointsGermano Cavalcante
This error only occurs when the end points are part of a sequence of selected points.
2020-04-03Code Quality: Replace for loops with LISTBASE_FOREACHDalai Felinto
Note this only changes cases where the variable was declared inside the for loop. To handle it outside as well is a different challenge. Differential Revision: https://developer.blender.org/D7320
2020-04-03Cleanup: Animation, move AnimData API to `anim_data.c`/`BKE_anim_data.h`Sybren A. Stüvel
The `BKE_animsys.h` and `anim_sys.c` files already had a an "AnimData API" section. The code in that section has now been split off, and placed into `BKE_anim_data.h` and `anim_data.c`. All files that used to include `BKE_animsys.h` have been adjusted to only include the animation headers they need (sometimes none). No functional changes.
2020-04-03Cleanup: use abbreviated names for unsigned types in editorsCampbell Barton
2020-04-03Cleanup: replace static list with argument for curve mergingCampbell Barton
2020-04-03Cleanup: move curve picking functions into 'editcurve_query.c'Campbell Barton
2020-04-01Fix problem extruding curve segments with selected handlesCampbell Barton
Issue introduced in 38685b5a39769
2020-04-01Cleanup: use doxy sectionsCampbell Barton
2020-03-31Edit Curve: Improve Curve extrudeGermano Cavalcante
The original code has two logics, extrude the end points or duplicate points (making new splines). Now all the logic has been redone by extruding contiguous selected segments. Fix T47169 Reviewed By: campbellbarton Differential Revision: https://developer.blender.org/D6982
2020-03-19Cleanup: `make format` after SortedIncludes changeDalai Felinto
2020-03-09Transform Snap Object: Remove depsgraph when creating contextGermano Cavalcante
Currently, this change does not bring functional changes. But it is necessary to extend the use of the snap system for gizmos, since, after a Undo, the `depsgraph` pointed by the `snap_context` has its memory invalidated. Reviewed By: campbellbarton Differential Revision: https://developer.blender.org/D7013
2020-03-06Cleanup: Rename ARegion variables from ar to regionJulian Eisel
The old convention was easy to confuse with ScrArea. Part of https://developer.blender.org/T74432. This is mostly a batch rename with some manual fixing. Only single word variable names are changed, no prefixed/suffixed names. Brecht van Lommel and Campbell Barton both gave me a green light for this convention change. Also ran clan clang format on affected files.
2020-03-03Cleanup: formatting, spellingCampbell Barton
2020-03-03Separate operators (mesh/curve/armature/gpencil): take user preferencesPhilipp Oeser
into account for duplicating actions Previously actions remained linked after duplication, now this is based on the User Preferences (PreferencesEdit.use_duplicate_action). note: default is ON here, so this changes default behavior of separate operators. First intuition was to respect _all_ preferences here (e.g. also duplicating materials if chosen in the User Preferences) but after consideration this is probably not what the User would expect from such 'modeling' opertions (e.g. separate by loose parts resulting in possibly many duplicate materials) Fixes T71038 Maniphest Tasks: T71038 Differential Revision: https://developer.blender.org/D6120
2020-02-10Cleanup/refactor: Rename `BKE_library` files to `BKE_lib`.Bastien Montagne
Note that `BKE_library.h`/`library.c` were renamed to `BKE_lib_id.h`/`lib_id.c` to avoid having a too generic name here. Part of T72604.
2020-02-04Improve error message converting nurb to bezierPhilipp Oeser
BKE_nurb_type_convert now takes r_err_msg and is more specific in the error message... ref T71672. Maniphest Tasks: T71672 Differential Revision: https://developer.blender.org/D6275
2019-12-17Fix T68665: FCurve group disappear on Curve/Surface object dataSybren A. Stüvel
When going from EDIT to OBJECT mode, Blender updates the object data from the edit-mode data. This took care of renaming FCurves that animate Curve control points when control points are added/removed, but this didn't keep the FCurve groups intact. Since the FCurve groups are tightly connected to the Action channels, it's hard to keep the group pointers intact during this process. Instead of making the code even more complex in an attempt to do that, I implemented a function (`BKE_action_groups_reconstruct()`) that rebuilds the group channel pointers. The call to `action_groups_add_channel()` had to be removed because it updates the the next/prev pointers of the FCurve while we're looping over them, causing infinite loops.
2019-12-17Cleanup: Animation: mark function parameter as `const`Sybren A. Stüvel
`fcurve_path_rename(..., rna_path, ...)` doesn't change `rna_path` at all, so it can be marked as `const char *`. No functional changes.
2019-11-24Cleanup: doxygen commentsCampbell Barton
Also correct some outdated symbol references, add missing 'name' commands.
2019-09-23Cleanup: remove unimplemented texture space rotation variablesBrecht Van Lommel
2019-09-21Cleanup: unused headers in editorsCampbell Barton
2019-09-18Fix excessive dependency graph evaluation while painting strokesBrecht Van Lommel
Particularly noticeable when vertex painting with a subsurf modifier. In some cases every sculpt or paint stroke step would evaluate the dependency graph. This should only happen for redraws. Now more selectively choose if the dependency graph should be evaluated to initialize the view context. Doing it in the view context evaluation is somewhat hidden, now it's more explicit. Differential Revision: https://developer.blender.org/D5844
2019-09-07Cleanup: use post increment/decrementCampbell Barton
When the result isn't used, prefer post increment/decrement (already used nearly everywhere in Blender).
2019-08-21Transform: New Snap Option: Edge Perpendicularmano-wii
Part of T66420 Option for snapping to the nearest point of a reference coordinate. The patch also adds Edge Center and Perpendicular snaps to the ruler. {F7675906} Reviewers: campbellbarton, brecht Reviewed By: campbellbarton Differential Revision: https://developer.blender.org/D5543
2019-08-19Fix T68779: "Match Texture Space" causes blender to crashPhilipp Oeser
need to access curve_cache from evaluated object. thx @sergey for pointing that out. Reviewers: sergey Maniphest Tasks: T68779 Differential Revision: https://developer.blender.org/D5526
2019-08-14Cleanup: move trailing comments to avoid wrapping codeCampbell Barton
Some statements were split across multiple lines because of their trailing comments. In most cases it's clearer to put the comments above.
2019-08-08Fix T68375: Polyline: can not make segment (cyclic)Philipp Oeser
Reviewers: campbellbarton Maniphest Tasks: T68375 Differential Revision: https://developer.blender.org/D5438
2019-07-31Refactor access to dependency graphSergey Sharybin
This change ensures that operators which needs access to evaluated data first makes sure there is a dependency graph. Other accesses to the dependency graph made it more explicit about whether they just need a valid dependency graph pointer or whether they expect the graph to be already evaluated. This replaces OPTYPE_USE_EVAL_DATA which is now removed. Some general rules about usage of accessors: - Drawing is expected to happen from a fully evaluated dependency graph. There is now a function to access it, which will in the future control that dependency graph is actually evaluated. This check is not yet done because there are some things to be taken care about first: for example, post-update hooks might leave scene in a state where something is still tagged for update. - All operators which needs to access evaluated state must use CTX_data_ensure_evaluated_depsgraph(). This function replaces OPTYPE_USE_EVAL_DATA. The call is generally to be done in the very beginning of the operator, prior other logic (unless this is some comprehensive operator which might or might not need access to an evaluated state). This call is never to be used from a loop. If some utility function requires evaluated state of dependency graph the graph is to be passed as an explicit argument. This way it is clear that no evaluation happens in a loop or something like this. - All cases which needs to know dependency graph pointer, but which doesn't want to actually evaluate it can use old-style function CTX_data_depsgraph_pointer(), assuming that underlying code will ensure dependency graph is evaluated prior to accessing it. - The new functions are replacing OPTYPE_USE_EVAL_DATA, so now it is explicit and local about where dependency graph is being ensured. This commit also contains some fixes of wrong usage of evaluation functions on original objects. Ideally should be split out, but in reality with all the APIs being renamed is quite tricky. Fixes T67454: Blender crash on rapid undo and select Speculation here is that sometimes undo and selection operators are sometimes handled in the same event loop iteration, which leaves non-evaluated dependency graph. Fixes T67973: Crash on Fix Deforms operator Fixes T67902: Crash when undo a loop cut Reviewers: brecht Reviewed By: brecht Subscribers: lichtwerk Maniphest Tasks: T67454 Differential Revision: https://developer.blender.org/D5343
2019-05-31Cleanup: style, use braces in source/ (include disabled blocks)Campbell Barton
2019-05-28Fix T64829: Active point of Curve objects is not displayed correctlyPhilipp Oeser
three issues here: - when curves had multiple nurbs, the active vert is per nurb [curve_create_edit_data_and_handles() wasnt taking that into account] - code could go wrong when points where hidden - upon selection, tag curve ID_RECALC_COPY_ON_WRITE for batch cache update Reviewers: brecht, fclem, sergey Maniphest Tasks: T64829 Differential Revision: https://developer.blender.org/D4943
2019-05-28Cleanup: clang formatCampbell Barton
2019-05-24Fix T65053: unhide curve does not refresh the viewport properlyPhilipp Oeser
Reviewers: sergey Maniphest Tasks: T65053 Differential Revision: https://developer.blender.org/D4937
2019-05-22Fix T64974: misisng multi-object edit for some curve operators, like smoothBrecht Van Lommel
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-21 23:48:27 +0300