blender
/
blender
133
398
Fork 579
Code Issues 6.3k Pull Requests 747 Projects 19 Wiki Activity

Initial Grease Pencil 3.0 stage #106848

Merged
Falk David merged 224 commits from filedescriptor/blender:grease-pencil-v3 into main 2023-05-30 11:14:22 +02:00
Conversation 8 Commits 224 Files Changed 94 +1 -1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 8f689e43f4 - Show all commits

2
source/blender/makesdna/DNA_grease_pencil_types.h
View File

@ -82,7 +82,7 @@ typedef struct GreasePencilDrawingReference {
/**
* A reference to another GreasePencil data-block.
* If the data-block has multiple drawings, this drawing references all of them sequentially.
* See the note in `GreasePencilLayer->frames()` for a detailed expelantion of this.
* See the note in `GreasePencilLayer->frames()` for a detailed explanation of this.
Hans Goudey commented 2023-04-25 04:49:05 +02:00
Outdated
Review
Copy Link

I wonder if this user count could be runtime data? Or if not, maybe it's worth mentioning why it isn't in a comment.

I wonder if this user count could be runtime data? Or if not, maybe it's worth mentioning why it isn't in a comment.
Falk David commented 2023-04-25 17:28:39 +02:00
Outdated
Review
Copy Link

I thought about this, but I don't think it can be runtime data. Since there will be keyframe instances, we need to make the user count is saved.

I thought about this, but I don't think it can be runtime data. Since there will be keyframe instances, we need to make the user count is saved.
Hans Goudey commented 2023-04-25 17:59:54 +02:00
Outdated
Review
Copy Link

The connection between keyframe instances and saving the user count isn't super clear to me. Maybe a comment about what the user count is used for here would help?

The connection between keyframe instances and saving the user count isn't super clear to me. Maybe a comment about what the user count is used for here would help?
Brecht Van Lommel commented 2023-04-25 21:19:31 +02:00
Outdated
Review
Copy Link

Maybe it's still better to have this as runtime data. If there is ever a bug where this doesn't get counted correctly, at least a file read could fix it. It should be possible to read this as zero, and increment it for every user?

This is easy to change afterwards though.

Maybe it's still better to have this as runtime data. If there is ever a bug where this doesn't get counted correctly, at least a file read could fix it. It should be possible to read this as zero, and increment it for every user? This is easy to change afterwards though.
Falk David commented 2023-04-25 21:23:23 +02:00
Outdated
Review
Copy Link

I see, re-creating the user count when reading could work indeed.

I see, re-creating the user count when reading could work indeed.
*/
struct GreasePencil *id_reference;
} GreasePencilDrawingReference;