[Bf-blender-cvs] [fc37238b17e] master: Cleanup: unused args/vars/imports in bl_operators

Campbell Barton noreply at git.blender.org
Thu May 9 05:13:22 CEST 2019


Commit: fc37238b17ed1a8c80bf5eaf849ab3fc4af59069
Author: Campbell Barton
Date:   Thu May 9 09:15:01 2019 +1000
Branches: master
https://developer.blender.org/rBfc37238b17ed1a8c80bf5eaf849ab3fc4af59069

Cleanup: unused args/vars/imports in bl_operators

===================================================================

M	release/scripts/startup/bl_operators/add_mesh_torus.py
M	release/scripts/startup/bl_operators/anim.py
M	release/scripts/startup/bl_operators/clip.py
M	release/scripts/startup/bl_operators/constraint.py
M	release/scripts/startup/bl_operators/file.py
M	release/scripts/startup/bl_operators/freestyle.py
M	release/scripts/startup/bl_operators/image.py
M	release/scripts/startup/bl_operators/mask.py
M	release/scripts/startup/bl_operators/object.py
M	release/scripts/startup/bl_operators/object_quick_effects.py
M	release/scripts/startup/bl_operators/object_randomize_transform.py
M	release/scripts/startup/bl_operators/presets.py
M	release/scripts/startup/bl_operators/rigidbody.py
M	release/scripts/startup/bl_operators/userpref.py
M	release/scripts/startup/bl_operators/uvcalc_follow_active.py
M	release/scripts/startup/bl_operators/uvcalc_lightmap.py
M	release/scripts/startup/bl_operators/uvcalc_smart_project.py
M	release/scripts/startup/bl_operators/vertexpaint_dirt.py
M	release/scripts/startup/bl_operators/view3d.py
M	release/scripts/startup/bl_operators/wm.py

===================================================================

diff --git a/release/scripts/startup/bl_operators/add_mesh_torus.py b/release/scripts/startup/bl_operators/add_mesh_torus.py
index e4495127ce7..a22161fc1ae 100644
--- a/release/scripts/startup/bl_operators/add_mesh_torus.py
+++ b/release/scripts/startup/bl_operators/add_mesh_torus.py
@@ -130,7 +130,7 @@ class AddTorus(Operator, object_utils.AddObjectHelper):
     bl_label = "Add Torus"
     bl_options = {'REGISTER', 'UNDO', 'PRESET'}
 
-    def mode_update_callback(self, context):
+    def mode_update_callback(self, _context):
         if self.mode == 'EXT_INT':
             self.abso_major_rad = self.major_radius + self.minor_radius
             self.abso_minor_rad = self.major_radius - self.minor_radius
@@ -196,7 +196,7 @@ class AddTorus(Operator, object_utils.AddObjectHelper):
         default=True,
     )
 
-    def draw(self, context):
+    def draw(self, _context):
         layout = self.layout
 
         col = layout.column(align=True)
@@ -241,7 +241,7 @@ class AddTorus(Operator, object_utils.AddObjectHelper):
             col.label(text="Interior Radius")
             col.prop(self, "abso_minor_rad", text="")
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         object_utils.object_add_grid_scale_apply_operator(self, context)
         return self.execute(context)
 
diff --git a/release/scripts/startup/bl_operators/anim.py b/release/scripts/startup/bl_operators/anim.py
index 4f2642bbe7c..1470aed4d55 100644
--- a/release/scripts/startup/bl_operators/anim.py
+++ b/release/scripts/startup/bl_operators/anim.py
@@ -191,7 +191,7 @@ class ANIM_OT_keying_set_export(Operator):
 
         return {'FINISHED'}
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         wm = context.window_manager
         wm.fileselect_add(self)
         return {'RUNNING_MODAL'}
@@ -285,7 +285,7 @@ class NLA_OT_bake(Operator):
 
         return {'FINISHED'}
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         scene = context.scene
         self.frame_start = scene.frame_start
         self.frame_end = scene.frame_end
@@ -309,10 +309,10 @@ class ClearUselessActions(Operator):
     )
 
     @classmethod
-    def poll(cls, context):
+    def poll(cls, _context):
         return bool(bpy.data.actions)
 
-    def execute(self, context):
+    def execute(self, _context):
         removed = 0
 
         for action in bpy.data.actions:
diff --git a/release/scripts/startup/bl_operators/clip.py b/release/scripts/startup/bl_operators/clip.py
index 87c103ea150..664684cce6d 100644
--- a/release/scripts/startup/bl_operators/clip.py
+++ b/release/scripts/startup/bl_operators/clip.py
@@ -906,7 +906,7 @@ class CLIP_OT_setup_tracking_scene(Operator):
         return None
 
     @staticmethod
-    def _createLight(scene):
+    def _createLight():
         light = bpy.data.lights.new(name="Light", type='POINT')
         lightob = bpy.data.objects.new(name="Light", object_data=light)
 
@@ -951,7 +951,7 @@ class CLIP_OT_setup_tracking_scene(Operator):
 
         # Create sample light if there is no lights in the scene.
         if not has_light:
-            light = self._createLight(scene)
+            light = self._createLight()
             fg_coll.objects.link(light)
             bg_coll.objects.link(light)
 
diff --git a/release/scripts/startup/bl_operators/constraint.py b/release/scripts/startup/bl_operators/constraint.py
index 811218f5e09..213122952ae 100644
--- a/release/scripts/startup/bl_operators/constraint.py
+++ b/release/scripts/startup/bl_operators/constraint.py
@@ -18,7 +18,6 @@
 
 # <pep8-80 compliant>
 
-import bpy
 from bpy.types import (
     Operator,
 )
diff --git a/release/scripts/startup/bl_operators/file.py b/release/scripts/startup/bl_operators/file.py
index 5a127cf2fe3..43fbd381cb2 100644
--- a/release/scripts/startup/bl_operators/file.py
+++ b/release/scripts/startup/bl_operators/file.py
@@ -92,7 +92,7 @@ class WM_OT_previews_batch_generate(Operator):
         description="Keep a backup (.blend1) version of the files when saving with generated previews",
     )
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         context.window_manager.fileselect_add(self)
         return {'RUNNING_MODAL'}
 
@@ -202,7 +202,7 @@ class WM_OT_previews_batch_clear(Operator):
         description="Keep a backup (.blend1) version of the files when saving with cleared previews",
     )
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         context.window_manager.fileselect_add(self)
         return {'RUNNING_MODAL'}
 
@@ -291,7 +291,7 @@ class WM_OT_blend_strings_utf8_validate(Operator):
                     changed |= self.validate_strings(it, done_items)
         return changed
 
-    def execute(self, context):
+    def execute(self, _context):
         changed = False
         done_items = set()
         for prop in bpy.data.bl_rna.properties:
diff --git a/release/scripts/startup/bl_operators/freestyle.py b/release/scripts/startup/bl_operators/freestyle.py
index 7196c53098b..b600df8fb84 100644
--- a/release/scripts/startup/bl_operators/freestyle.py
+++ b/release/scripts/startup/bl_operators/freestyle.py
@@ -212,13 +212,13 @@ class SCENE_OT_freestyle_module_open(bpy.types.Operator):
         view_layer = context.view_layer
         return view_layer and view_layer.freestyle_settings.mode == 'SCRIPT'
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         self.freestyle_module = context.freestyle_module
         wm = context.window_manager
         wm.fileselect_add(self)
         return {'RUNNING_MODAL'}
 
-    def execute(self, context):
+    def execute(self, _context):
         text = bpy.data.texts.load(self.filepath, self.make_internal)
         self.freestyle_module.script = text
         return {'FINISHED'}
diff --git a/release/scripts/startup/bl_operators/image.py b/release/scripts/startup/bl_operators/image.py
index d07546d1773..4f0b574ced9 100644
--- a/release/scripts/startup/bl_operators/image.py
+++ b/release/scripts/startup/bl_operators/image.py
@@ -89,7 +89,7 @@ class EditExternally(Operator):
 
         return {'FINISHED'}
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         import os
         sd = context.space_data
         try:
@@ -121,7 +121,7 @@ class SaveDirty(Operator):
     bl_label = "Save Dirty"
     bl_options = {'REGISTER', 'UNDO'}
 
-    def execute(self, context):
+    def execute(self, _context):
         unique_paths = set()
         for image in bpy.data.images:
             if image.is_dirty:
@@ -227,7 +227,7 @@ class ProjectApply(Operator):
     bl_label = "Project Apply"
     bl_options = {'REGISTER'}
 
-    def execute(self, context):
+    def execute(self, _context):
         image_name = ProjectEdit._proj_hack[0]  # TODO, deal with this nicer
 
         try:
diff --git a/release/scripts/startup/bl_operators/mask.py b/release/scripts/startup/bl_operators/mask.py
index 3ce1abad9b8..395423f745f 100644
--- a/release/scripts/startup/bl_operators/mask.py
+++ b/release/scripts/startup/bl_operators/mask.py
@@ -25,7 +25,7 @@ class MASK_MT_add(Menu):
     bl_idname = "MASK_MT_add"
     bl_label = "Add"
 
-    def draw(self, context):
+    def draw(self, _context):
         layout = self.layout
 
         layout.operator_context = 'INVOKE_REGION_WIN'
diff --git a/release/scripts/startup/bl_operators/object.py b/release/scripts/startup/bl_operators/object.py
index c05ee01c72d..f68b59192b8 100644
--- a/release/scripts/startup/bl_operators/object.py
+++ b/release/scripts/startup/bl_operators/object.py
@@ -103,7 +103,7 @@ class SelectPattern(Operator):
         wm = context.window_manager
         return wm.invoke_props_popup(self, event)
 
-    def draw(self, context):
+    def draw(self, _context):
         layout = self.layout
 
         layout.prop(self, "pattern")
@@ -610,7 +610,6 @@ class MakeDupliFace(Operator):
             rot = matrix.to_3x3()  # also contains scale
 
             return [(rot @ b) + trans for b in base_tri]
-        scene = context.scene
         linked = defaultdict(list)
         for obj in context.selected_objects:
             if obj.type == 'MESH':
@@ -895,7 +894,7 @@ class LoadImageAsEmpty:
     def poll(cls, context):
         return context.mode == 'OBJECT'
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         context.window_manager.fileselect_add(self)
         return {'RUNNING_MODAL'}
 
diff --git a/release/scripts/startup/bl_operators/object_quick_effects.py b/release/scripts/startup/bl_operators/object_quick_effects.py
index 798dd0bdd6a..87e88010b80 100644
--- a/release/scripts/startup/bl_operators/object_quick_effects.py
+++ b/release/scripts/startup/bl_operators/object_quick_effects.py
@@ -304,7 +304,7 @@ class QuickExplode(ObjectModeOperator, Operator):
 
         return {'FINISHED'}
 
-    def invoke(self, context, event):
+    def invoke(self, context, _event):
         self.frame_start = context.scene.frame_current
         self.frame_end = self.frame_start + self.frame_duration
         return self.execute(context)
diff --git a/release/scripts/startup/bl_operators/object_randomize_transform.py b/release/scripts/startup/bl_operators/object_randomize_transform.py
index bfac9ccb751..a9a6819cb13 100644
--- a/release/scripts/startup/bl_operators/object_randomize_transform.py
+++ b/release/scripts/startup/bl_operators/object_randomize_transform.py
@@ -18,13 +18,12 @@
 
 # <pep8-80 compliant>
 
-import bpy
 from bpy.types import Operator
 from mathutils import Vector
 
 
 def randomize_selected(context, seed, delta,
-                       loc, rot, scale, scale_even, scale_min):
+                       loc, rot, scale, scale_even, _scale_min):
 
     import random
     from random import uniform
diff --git a/release/scripts/startup/bl_operators/presets.py b/release/scripts/startup/bl_operators/presets.py
index c7f516f61a1..8ca08938fe5 100644
--- a/release/scripts/startup/bl_operators/presets.py
+++ b/release/scripts/startup/bl_operators/presets.py
@@ -209,10 +209,10 @@ class AddPresetBase:
 
         return {'FINISHED'}
 
-    def check(self

@@ Diff output truncated at 10240 characters. @@



More information about the Bf-blender-cvs mailing list