Avoid code duplication and using throwaway variables

some uses of str.format remain, but converting them would be more trouble than it's worth

game/scripts/inventory/potions/_potions_.txt

@@ -331,15 +331,15 @@ init -1 python:
 
         def get_store_menu_item(self, disabled=False):
             if disabled:
-                return gui.menu_item('-{}-'.format(self.name), None, style="disabled")
+                return gui.menu_item(f'-{self.name}-', None, style="disabled")
             else:
-                return gui.menu_item('-{}-'.format(self.name), self)
+                return gui.menu_item(f'-{self.name}-', self)
 
         def get_craft_menu_item(self, disabled=False):
             if disabled:
-                return gui.menu_item('-Craft: "{}"-'.format(self.name), self.ingredients, style="disabled")
+                return gui.menu_item(f'-Craft: "{self.name}"-', self.ingredients, style="disabled")
             else:
-                return gui.menu_item('-Craft: "{}"-'.format(self.name), self.id)
+                return gui.menu_item(f'-Craft: "{self.name}"-', self.id)
 
         def get_mix_text(self):
             return ">You mix the {i}" + potion_lib.get_name_by_id(self.ingredients[0]) \

game/scripts/utility/devtools.rpy

@@ -42,7 +42,7 @@ python early:
                     orphaned.append(i)
 
             if orphaned:
-                raise Exception("Orphaned compiled scripts detected, please delete them before continuing:\n{}".format(orphaned))
+                raise Exception(f"Orphaned compiled scripts detected, please delete them before continuing:\n{orphaned}")
 
         detect_orphaned_rpyc_files()
 
@@ -91,7 +91,7 @@ init -2 python:
             return Image(file + ".png")
 
         if config.developer:
-            raise IOError("Missing image: {}".format(path))
+            raise IOError(f"Missing image: {path}")
         return Image("images/blank.webp")
 
     def missing_label_func(name):

game/scripts/utility/editor.rpy

@@ -86,7 +86,7 @@ init python:
             elif what == "temp_attr":
                 setattr(node, "temporary_attributes", tuple(contents))
             else:
-                raise TypeError("Type {!r} is not implemented.".format(what))
+                raise TypeError(f"Type {what!r} is not implemented.")
 
         def replace_expression(self, expr, val):
             node = self.node
@@ -97,14 +97,14 @@ init python:
             # We need to make sure not to add quotes
             # to expressions or variables.
             if isinstance(val, str):
-                val = "\"{}\"".format(val)
+                val = repr(val)
 
             # Insert new expression
             d = self.get_expressions_active(who)
             d[expr] = val
 
             # Convert to list of tuples
-            # l = [(k, "\"{}\"".format(v)) for k, v in d.items() if not v is None] # This is faster, but not robust enough.
+            # l = [(k, repr(v)) for k, v in d.items() if not v is None] # This is faster, but not robust enough.
 
             l = _list()
             l2 = _list()

game/scripts/utility/engine.rpy

@@ -39,13 +39,13 @@ init python:
 
                     if cache_size >= cache_limit:
                         if config.developer:
-                            print("Cache limit reached, purging cache... ({}/{})\n{}".format(cache_size, cache_limit, renpy.get_filename_line()))
+                            print(f"Cache limit reached, purging cache... ({cache_size}/{cache_limit})\n{renpy.get_filename_line()}")
 
                         cache.clear()
 
                     if renpy.game.interface is not None:
                         if config.developer:
-                            print("Statements limit reached, cleaning textures... ({})\n{}".format(self.limit, renpy.get_filename_line()))
+                            print(f"Statements limit reached, cleaning textures... ({self.limit})\n{renpy.get_filename_line()}")
 
                         renpy.game.interface.full_redraw = True
                         renpy.game.interface.restart_interaction = True

game/scripts/utility/lint.rpy

@@ -137,7 +137,7 @@ init -1 python:
                                     renpy.lint.report_node = node
                                     has_failed = True
 
-                                msg = "{!r} requires an integer, or a pre-defined named position, not {!r}".format(key, val)
+                                msg = f"{key!r} requires an integer, or a pre-defined named position, not {val!r}"
                                 renpy.lint.report(msg)
 
                             # This would require fixing hundreds of calls. Might postpone it...
@@ -161,7 +161,7 @@ init -1 python:
 
                                 if key == "emote":
                                     msg = repr(key)
-                                    fn = "characters/{}/emote/{}.webp".format(SAYERS.get(who), val)
+                                    fn = f"characters/{SAYERS.get(who)}/emote/{val}.webp"
 
                                     if not has_failed:
                                         # Avoid repeating node destination

game/scripts/utility/translation.rpy

@@ -75,7 +75,7 @@ init python in cli:
 
         for i, s in enumerate(strings):
             n = round(float(i)/(nstrings)*100)
-            print("\rGenerating strings for {} ... Total progress:{} % ... Stage 2/2".format(language, n), end="")
+            print(f"\rGenerating strings for {language} ... Total progress:{n} % ... Stage 2/2", end="")
 
             tlfn = renpy.translation.generation.translation_filename(s)
 
@@ -96,17 +96,17 @@ init python in cli:
             tlfn = os.path.join(renpy.config.gamedir, renpy.config.tl_directory, language, tlfn)
 
             f = open_tl_file(tlfn, mode="w")
-            f.write(u"translate {} strings:\n".format(language))
+            f.write(f"translate {language} strings:\n")
-            f.write(u"\n")
+            f.write("\n")
 
             for s in sl:
                 original = s.text
                 translation = stl.translate(s.text) # Keeps translated strings
 
-                f.write(u"    # {}:{}\n".format(elide_filename(s.filename), s.line))
+                f.write(f"    # {elide_filename(s.filename)}:{s.line}\n")
-                f.write(u"    old \"{}\"\n".format(quote_unicode(original)))
+                f.write(f"    old \"{quote_unicode(original)}\"\n")
-                f.write(u"    new \"{}\"\n".format(quote_unicode(translation)))
+                f.write(f"    new \"{quote_unicode(translation)}\"\n")
-                f.write(u"\n")
+                f.write("\n")
 
     def retranslate():
         translator = renpy.game.script.translator
@@ -135,7 +135,7 @@ init python in cli:
 
             for language in translator.languages:
 
-                print("\rGenerating dialogues for {} ... Total progress:{} % ... Stage 1/2".format(language, n), end="")
+                print(f"\rGenerating dialogues for {language} ... Total progress:{n} % ... Stage 1/2", end="")
 
                 for _, trans in translator.file_translates[filepath]:
 
@@ -159,9 +159,9 @@ init python in cli:
 
                     f = open_tl_file(fp, mode="w")
 
-                    f.write(u"# {}:{}\n".format(trans.filename, trans.linenumber))
+                    f.write(f"# {trans.filename}:{trans.linenumber}\n")
-                    f.write(u"translate {} {}:\n".format(language, trans.identifier.replace(".", "_")))
+                    f.write("translate {} {}:\n".format(language, trans.identifier.replace(".", "_")))
-                    f.write(u"\n")
+                    f.write("\n")
 
                     for n in trans.block:
                         f.write(u"    # " + n.get_code() + "\n")
@@ -196,7 +196,7 @@ init python in cli:
                             f.close()
 
                         if args.dry:
-                            print("Removal required: {}".format(filepath))
+                            print(f"Removal required: {filepath}")
                         else:
                             os.unlink(filepath)
                             os.unlink(filepath + "c")

game/scripts/utility/updater.rpy

@@ -214,7 +214,7 @@ init python:
             delattr(store, "achievements_dict")
 
         if current > latest:
-            raise Exception("Loaded save file is incompatible. (Save Version: {}, Game Version: {})".format(current, latest))
+            raise Exception(f"Loaded save file is incompatible. (Save Version: {current}, Game Version: {latest})")
 
         if current < latest:
             setattr(store, "version", latest)
@@ -228,7 +228,7 @@ init python:
 
     def version_logo():
         url = UPDATE_URL
-        filename = "logo_{}.webp".format(UPDATE_VER)
+        filename = f"logo_{UPDATE_VER}.webp"
         path = os.path.join(config.basedir, "update", filename)
 
         # Read file if exists

game/scripts/wardrobe/studio.rpy

@@ -203,7 +203,7 @@ screen studio():
     $ bg_blur = studio.choices["background"]["blur"]
     $ bg_matrix = bg_hue*bg_saturation*bg_brightness
     $ bg_image = studio.choices["background"]["list"][studio.choices["background"]["image"]]
-    $ bg_image = "images/rooms/_bg_/{}.webp".format(bg_image)
+    $ bg_image = f"images/rooms/_bg_/{bg_image}.webp"
     $ bg = Transform(bg_image, matrixcolor=bg_matrix, blur=bg_blur)
 
     $ ov_hue = HueMatrix(studio.choices["overlay"]["hue"])
@@ -214,9 +214,8 @@ screen studio():
     $ ov_matrix = ov_hue*ov_saturation*ov_brightness
     $ ov_image = studio.choices["overlay"]["list"][studio.choices["overlay"]["image"]]
 
-    if not ov_image is None:
+    if ov_image is not None:
-        $ ov_image = "images/rooms/overlays/{}.webp".format(ov_image)
+        $ ov = Transform(f"images/rooms/overlays/{ov_image}.webp", matrixcolor=ov_matrix, blur=ov_blur, alpha=ov_alpha)
-        $ ov = Transform(ov_image, matrixcolor=ov_matrix, blur=ov_blur, alpha=ov_alpha)
     else:
         $ ov = None
 
@@ -337,7 +336,7 @@ screen studio():
                                 selected v[1] text_color ("#009900" if active else "#f9d592")
                                 text_hover_color "#fff"
                                 text_first_indent 20
-                                background Transform("interface/icons/head/{}.webp".format(k), size=(16, 16), offset=(22, 3))
+                                background Transform(f"interface/icons/head/{k}.webp", size=(16, 16), offset=(22, 3))
 
         vbox:
             align (1.0, 1.0)

game/scripts/wardrobe/wardrobe.rpy

@@ -638,22 +638,16 @@ screen wardrobe_menu(xx, yy):
             yspacing 18
 
             for i, category in enumerate(wardrobe_categories):
-                if wardrobe_check_category(category):
-                    $ icon = Fixed(icon_bg, Transform("interface/wardrobe/icons/categories/{}/{}.webp".format(states.active_girl, category), zoom=0.45, anchor=(0.5, 0.5), align=(0.5, 0.5)), icon_frame)
-                else:
-                    $ icon = Fixed(icon_bg, Transform("interface/wardrobe/icons/categories/{}/{}.webp".format(states.active_girl, category), zoom=0.45, anchor=(0.5, 0.5), align=(0.5, 0.5), matrixcolor=SaturationMatrix(0.0)), icon_frame)
-                $ icon_xoffset = -18 if (i % 2) == 0 else 18
-
                 button:
                     focus_mask None
                     xysize (72, 72)
-                    background lock_wardrobe_icon(icon)
+                    background lock_wardrobe_icon(Fixed(icon_bg, Transform(f"interface/wardrobe/icons/categories/{states.active_girl}/{category}.webp", zoom=0.45, anchor=(0.5, 0.5), align=(0.5, 0.5), matrixcolor=(IdentityMatrix() if wardrobe_check_category(category) else SaturationMatrix(0.0))), icon_frame))
                     activate_sound "sounds/scroll.ogg"
                     tooltip category
                     sensitive (not bool(DollThread._count))
                     action Return(["category", category])
                     if current_category == category:
-                        xoffset icon_xoffset
+                        xoffset -18 if (i % 2) == 0 else 18
 
         # Outfits and Studio
         hbox:
@@ -777,7 +771,7 @@ screen wardrobe_menuitem(xx, yy):
             pos (12, 108)
 
             for subcategory in category_items.keys():
-                $ icon = lock_wardrobe_icon("interface/wardrobe/icons/{}.webp".format(subcategory))
+                $ icon = lock_wardrobe_icon(f"interface/wardrobe/icons/{subcategory}.webp")
 
                 button:
                     focus_mask None
@@ -830,7 +824,7 @@ screen wardrobe_outfit_menuitem(xx, yy):
             pos (8, 108)
 
             for subcategory in category_items.keys():
-                $ icon = lock_wardrobe_icon("interface/wardrobe/icons/{}.webp".format(subcategory))
+                $ icon = lock_wardrobe_icon(f"interface/wardrobe/icons/{subcategory}.webp")
                 $ action = Return(["subcategory", subcategory])
 
                 if subcategory == "schedule" and not get_character_scheduling(states.active_girl):
@@ -903,7 +897,7 @@ screen wardrobe_schedule_menuitem(item):
                 spacing 0
                 for i in wardrobe_outfit_schedule:
                     $ boolean = "" if item.schedule[i] else "Not "
-                    $ caption = "{}worn during the {}".format(boolean, i) if i in ("day", "night") else "{}worn in {} weather".format(boolean, i)
+                    $ caption = f"{boolean}worn during the {i}" if i in ("day", "night") else f"{boolean}worn in {i} weather"
                     textbutton i:
                         style gui.theme("dropdown")
                         tooltip caption