Codefix: Fix typos and grammar errors (#389)

Author: Rito13

README.md

@@ -1,6 +1,6 @@
 # NML NewGRF meta language
 
-NML is a a python-based compiler, capable of compiling NML files (along
+NML is a Python-based compiler, capable of compiling NML files (along
 with their associated language, sound and graphic files) into grf
 and / or nfo files.
 

nml/actions/action0.py

@@ -48,7 +48,7 @@ class BlockAllocation:
     @ivar allocated: Mapping of allocated blocks.
     @type allocated: C{dict} of C{int} to allocation information.
 
-    @ivar filled: Mapping of block size to smallest address that may contain free space.
+    @ivar filled: Mapping of block size to the smallest address that may contain free space.
                   Serves as a cache to speed up searches.
     @type filled: C{dict} of C{int}
     """
@@ -72,7 +72,7 @@ def get_num_allocated(self):
 
     def get_max_allocated(self):
         """
-        Return maximum number of allocateable ids.
+        Return maximum number of allocatable ids.
         """
         if self.dynamic_allocation:
             return self.last - self.first + 1
@@ -90,7 +90,7 @@ def in_range(self, addr, length):
         @param length: Number of addresses in the block.
         @type  length: C{int}
 
-        @return: Whether the block fits enitrely in the available address space.
+        @return: Whether the block fits entirely in the available address space.
         @rtype:  C{bool}
         """
         return addr >= 0 and addr + length - 1 <= self.last
@@ -923,7 +923,7 @@ def get_basecost_action(basecost):
 
         num_ids = 1  # Number of values that will be written in one go
         values = []
-        # try to capture as much values as possible
+        # try to capture as many values as possible
         while True:
             cost = basecost.costs[i]
             if isinstance(cost.value, expression.ConstantNumeric):

nml/actions/action0properties.py

@@ -108,7 +108,7 @@ def get_size(self):
 # Summary: If 'string' or 'string_literal' is set, the value should be a
 # string or literal string, else the value is a number. 'unit_type' and
 # 'unit_conversion' are used to convert user-entered values to nfo values.
-# If some more arithmetic is needed to convert the entered value into an nfo
+# If some more arithmetic is needed to convert the entered value into a nfo
 # value, 'value_function' can be used. For even more complicated things,
 # 'custom_function' can be used to create a special mapping of the value to nfo
 # properties, else 'num' and 'size' are used to provide a 'normal' action0.
@@ -214,7 +214,7 @@ def two_byte_property(low_prop, high_prop, low_prop_info=None, high_prop_info=No
 def animation_info(value, loop_bit=8, max_frame=253):
     """
     Convert animation info array of two elements to an animation info property.
-    The first is 0/1, and defines whether or not the animation loops.
+    The first is 0/1, and defines whether the animation loops.
     The second is the number of frames, at most 253 frames.
 
     @param value: Array of animation info.
@@ -440,12 +440,12 @@ def single_or_list(prop_name, single_num, multiple_num, value):
         "unit_conversion": (5000, 1397),
         "adjust_value": lambda val, unit: ottd_display_speed(val, 1, 1, unit)
     },
-    # 09 doesn"t exist
+    # 09 doesn't exist
     "power":                          {"size": 2, "num": 0x0B, "unit_type": "power"},
-    # 0A doesn"t exist
+    # 0A doesn't exist
     "running_cost_factor":            {"size": 1, "num": 0x0D},
     "running_cost_base":              {"size": 4, "num": 0x0E},
-    # 0F -11 don"t exist
+    # 0F -11 don't exist
     "sprite_id":                      {"size": 1, "num": 0x12},
     "dual_headed":                    {"size": 1, "num": 0x13},
     "cargo_capacity":                 {"size": 1, "num": 0x14},
@@ -729,7 +729,7 @@ def aircraft_is_large(value):
 
 def station_platforms_length(value):
     # Writing bitmask(2) to disable platform/length 3 is not very intuitive.
-    # Instead we expect the user will write bitmask(3) and we shift the result.
+    # Instead, we expect the user will write bitmask(3) and we shift the result.
     return nmlop.SHIFT_RIGHT(value, 1, value.pos).reduce()
 
 
@@ -1578,7 +1578,7 @@ def label_list(value, prop_num, description):
 properties[0x11] = {
     "substitute":         {"size": 1, "num": 0x08, "first": None},
     "override":           {"size": 1, "num": 0x09},
-    # 0A - 0D don"t exist (yet?)
+    # 0A - 0D don't exist (yet?)
     # 0E (callback flags) is not set by user
     "animation_info":     {"size": 2, "num": 0x0F, "value_function": animation_info},
     "animation_speed":    {"size": 1, "num": 0x10},

nml/actions/action1.py

@@ -145,7 +145,7 @@ def get_action_list(self):
         Create a list of actions needed to write this collection to the output. This
         will generate a single Action1 and as many realsprites as needed.
 
-        @return: A list of actions needed to represet this collection in a GRF.
+        @return: A list of actions needed to represent this collection in a GRF.
         @rtype: C{list} of L{BaseAction}
         """
         actions = [Action1(self.feature, self.first_set, len(self.spritesets), self.num_sprites_per_spriteset)]

nml/actions/action2.py

@@ -139,7 +139,7 @@ def skip_needed(self):
     def remove_tmp_location(self, location, force_recursive):
         """
         Recursively remove a location from the list of available temporary
-        storage locations. It is not only removed from the the list of the
+        storage locations. It is not only removed from the list of the
         current Action2Var but also from all Action2Var it calls. If an
         Action2Var is referenced as a procedure call, the location is always
         removed recursively, otherwise only if force_recursive is True.
@@ -230,7 +230,7 @@ def make_sprite_group_class(cls_is_spriteset, cls_is_referenced, cls_has_explici
     Metaclass factory which makes base classes for all nodes 'Action 2 graph'
     This graph is made up of all blocks that are eventually compiled to Action2,
     which use the same name space. Spritesets do inherit from this class to make
-    referencing them possible, but they are not part of the refernce graph that
+    referencing them possible, but they are not part of the reference graph that
     is built.
 
     @param cls_is_spriteset: Whether this class represents a spriteset
@@ -244,7 +244,7 @@ def make_sprite_group_class(cls_is_spriteset, cls_is_referenced, cls_has_explici
     @type cls_has_explicit_feature: C{bool}
 
     @param cls_is_relocatable: Whether instances of this class can be freely moved around or whether they need
-                               to to be converted to nfo code at the same location as they are in the nml code.
+                               to be converted to nfo code at the same location as they are in the nml code.
     @type cls_is_relocatable: C{bool}
 
     @return: The constructed class
@@ -262,7 +262,7 @@ class ASTSpriteGroup(base_statement.BaseStatement):
         Child classes should do the following:
             - Implement their own __init__ method
             - Call BaseStatement.__init__
-            - Call initialize, pre_process and perpare_output (in that order)
+            - Call initialize, pre_process and prepare_output (in that order)
             - Implement collect_references
             - Call set_action2 after generating the corresponding action2 (if applicable)
 
@@ -294,8 +294,8 @@ class ASTSpriteGroup(base_statement.BaseStatement):
         def __init__(self):
             """
             Subclasses should implement their own __init__ method.
-            This method should not be called, because calling a method on a meta class can be troublesome.
-            Instead, call initialize(..).
+            This method should not be called, because calling a method on a metaclass can be troublesome.
+            Instead, call initialize(...).
             """
             raise NotImplementedError(
                 (

nml/actions/action2layout.py

@@ -117,7 +117,7 @@ def write_flags(self, file):
             flags |= 1 << 2
         if self.palette_from_action1:
             flags |= 1 << 3
-        # for building sprites: bit 4 => xoffset+yoffset, bit 5 => zoffset (x and y always set totgether)
+        # for building sprites: bit 4 => xoffset+yoffset, bit 5 => zoffset (x and y always set together)
         # for child sprites: bit 4 => xoffset, bit 5 => yoffset
         if self.type == Action2LayoutSpriteType.BUILDING:
             assert (self.get_register("xoffset") is not None) == (self.get_register("yoffset") is not None)

nml/actions/action2random.py

@@ -105,7 +105,7 @@ def __init__(self, result, prob, comment):
 
 def parse_randomswitch_type(random_switch):
     """
-    Parse the type of a random switch to determine the type and random bits to use.
+    Parse the type of random switch to determine the type and random bits to use.
 
     @param random_switch: Random switch to parse the type of
     @type random_switch: L{RandomSwitch}
@@ -203,7 +203,7 @@ def parse_randomswitch_dependencies(random_switch, start_bit, bits_available, nr
 
     @return: A tuple of two values:
                 - The first random bit to use
-                - The number of random choices to use. This may be higher the the original amount passed as paramter
+                - The number of random choices to use. This may be higher than the original amount passed as parameter
     @rtype: C{tuple} of (C{int}, C{int})
     """
     # Dependent random chains
@@ -240,7 +240,7 @@ def parse_randomswitch_dependencies(random_switch, start_bit, bits_available, nr
     else:
         randbit = -1
 
-    # INdependent random chains
+    # Independent random chains
     possible_mask = ((1 << bits_available) - 1) << start_bit
     for indep in random_switch.independent:
         act2 = lookup_random_action2(indep)

nml/actions/action2var_variables.py

@@ -298,7 +298,7 @@ def vehicle_tramtype(name, args, pos, info):
 varact2vars_stations = {
     **varact2vars_base_stations,
     # Vars 40, 41, 46, 47, 49 are implemented as 60+x vars,
-    # except for the 'tile type' part which is always the same anyways
+    # except for the 'tile type' part which is always the same anyway
     'tile_type'                : {'var': 0x40, 'start': 24, 'size': 8},
     'terrain_type'             : {'var': 0x42, 'start':  0, 'size': 8},
     'track_type'               : {'var': 0x42, 'start':  8, 'size': 8},

nml/actions/action3.py

@@ -474,7 +474,7 @@ def parse_graphics_block_single_id(
         # Special case for vehicle cb 36, maps var10 values to spritegroups
         cb36_mapping = {}
         cb36_buy_mapping = {}
-        # Sspecial case for industry production CB, maps var18 values to spritegroups
+        # Special case for industry production CB, maps var18 values to spritegroups
         prod_cb_mapping = {}
 
         for cb_info, gfx in callbacks:

nml/actions/action3_callbacks.py

@@ -29,7 +29,7 @@
 general_vehicle_cbs = {
     'default'                 : {'type': 'cargo', 'num': None},
     'purchase'                : {'type': 'cargo', 'num': 0xFF},
-    'random_trigger'          : {'type': 'cb', 'num': 0x01}, # Almost undocumented, but really neccesary!
+    'random_trigger'          : {'type': 'cb', 'num': 0x01}, # Almost undocumented, but really necessary!
     'loading_speed'           : {'type': 'cb', 'num': 0x36, 'var10': 0x07},
     'cargo_subtype_text'      : {'type': 'cb', 'num': 0x19, 'flag_bit': 5},
     'additional_text'         : {'type': 'cb', 'num': 0x23, 'purchase': 2},