BogdanNagirniak/blender
0
0
Fork 3
forked from blender/blender
Code Pull Requests Activity

BLEN-358: Error in console after closing Blender #22

Merged
Bogdan Nagirniak merged 4 commits from BLEN-358_1 into hydra-render 2023-04-06 11:58:30 +02:00
Conversation 0 Commits 4 Files Changed 1 +7 -6
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit cc46fc1a0a - Show all commits

13
scripts/modules/bpy_hydra.py
View File

@ -106,12 +106,7 @@ def export_mtlx(material):
try:
import materialx.utils as mx_utils
try:
doc = mx_utils.export(material, None)
except:
traceback.print_exc()
return ""
doc = mx_utils.export(material, None)
if not doc:
return ""
DagerD marked this conversation as resolved Outdated
Bogdan Nagirniak commented 2023-04-05 15:48:57 +02:00
Outdated
Review
Copy Link

Move this to end of function with comment like "Important: code shouldn't raise any exception ...."

Move this to end of function with comment like "Important: code shouldn't raise any exception ...."
@ -122,4 +117,10 @@ def export_mtlx(material):
except ImportError:
print("ERROR: no MaterialX addon available")
# This is a placeholder, this code will be moved to C part later
# This code shouldn't raise any exception due to breaking refcounts on RenderEngine
except:
traceback.print_exc()
return ""
DagerD marked this conversation as resolved
Bogdan Nagirniak commented 2023-04-06 11:50:53 +02:00
Review
Copy Link

remove return "" from here, move comment under except:

remove `return ""` from here, move comment under `except:`
return ""