Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1296)

Issue 6332047: Potential fix for bug #31880 (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 10 months ago by nicholasbishop
Modified:
11 years, 10 months ago
Reviewers:
sergey.vfx, bf-codereview
Base URL:
https://svn.blender.org/svnroot/bf-blender/trunk/blender/
Visibility:
Public.

Description

Log: Fixes for modifier data in multi-user meshes. When removing a skin or multires modifier, it skips deletion of the associated CustomData layer if the object has any other modifiers of that type. This check has been extended to all objects that use the object's data. Similarly, deleting higher multires levels and multires subdivision will not update the maximum level of any other multires modifiers on objects that link to the same mesh. Note that modifier_apply_obdata() doesn't need any changes as it does not allow applying to multi-user data. Fixes bug [#31880] instance multiresolution modifier error. http://projects.blender.org/tracker/index.php?func=detail&aid=31880&group_id=9&atid=498

Patch Set 1 #

Patch Set 2 : Fixes for joining multires mesh objects #

Unified diffs Side-by-side diffs Delta from patch set Stats (+137 lines, -33 lines) Patch
source/blender/blenkernel/BKE_multires.h View 1 1 chunk +3 lines, -0 lines 0 comments Download
source/blender/blenkernel/intern/multires.c View 1 3 chunks +9 lines, -5 lines 0 comments Download
source/blender/editors/include/ED_object.h View 1 1 chunk +6 lines, -0 lines 0 comments Download
source/blender/editors/mesh/meshtools.c View 1 2 chunks +11 lines, -1 line 0 comments Download
source/blender/editors/object/object_modifier.c View 1 6 chunks +108 lines, -27 lines 0 comments Download

Messages

Total messages: 6
nicholasbishop
11 years, 10 months ago (2012-06-21 21:26:33 UTC) #1
sergey.vfx
Just sone more possible issue: when you join objects synchronization of subdivision level happens in ...
11 years, 10 months ago (2012-06-22 07:24:08 UTC) #2
nicholasbishop
On 2012/06/22 07:24:08, sergey.vfx wrote: > Just sone more possible issue: when you join objects ...
11 years, 10 months ago (2012-06-22 17:01:12 UTC) #3
nicholasbishop
Fixes for joining multires mesh objects
11 years, 10 months ago (2012-06-22 17:01:45 UTC) #4
sergey.vfx
On 2012/06/22 17:01:45, nicholasbishop wrote: > Fixes for joining multires mesh objects Well, i'm not ...
11 years, 10 months ago (2012-06-24 12:30:10 UTC) #5
nicholasbishop
11 years, 10 months ago (2012-06-24 20:19:17 UTC) #6
On 2012/06/24 12:30:10, sergey.vfx wrote:
> On 2012/06/22 17:01:45, nicholasbishop wrote:
> > Fixes for joining multires mesh objects
> 
> Well, i'm not sure how to prevent changes in instanced objects in a nice way
> too. This could probably be done via making dupliobject real and then joining,
> but that's quite tricky.
> 
> I'd say current patch is fine.

Committed as r48247, thanks for reviewing.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b