Respond to PR comments

Author: alan412

src/blocks/mrc_mechanism.ts

@@ -66,7 +66,7 @@ const MECHANISM_FUNCTION = {
     * Returns the state of this block as a JSON serializable object.
     */
   saveExtraState: function (this: MechanismBlock): MechanismExtraState {
-    let extraState: MechanismExtraState = {
+    const extraState: MechanismExtraState = {
     };
     extraState.params = [];
     this.mrcArgs.forEach((arg) => {
@@ -103,7 +103,7 @@ const MECHANISM_FUNCTION = {
      */
     updateBlock_: function(this: MechanismBlock): void {
       // Add input sockets for the arguments.
-      for (let i = 0; i < this.mrcArgs.length; i++) {
+      for (const i = 0; i < this.mrcArgs.length; i++) {
         const input = this.appendValueInput('ARG' + i)
             .setAlign(Blockly.inputs.Align.RIGHT)
             .appendField(this.mrcArgs[i].name);
@@ -129,15 +129,15 @@ export const pythonFromBlock = function (
   let code = 'self.mechanisms["' + mechanismBlock.getFieldValue('NAME') + '"] = '
     + mechanismBlock.getFieldValue('TYPE') + '('
 
-  for (let i = 0; i < mechanismBlock.mrcArgs.length; i++) {
-      let fieldName = 'ARG' + i;
+  for (const i = 0; i < mechanismBlock.mrcArgs.length; i++) {
+      const fieldName = 'ARG' + i;
       if(i != 0){
         code += ', '
       }
       code += mechanismBlock.mrcArgs[i].name + ' = ' + generator.valueToCode(mechanismBlock, fieldName, Order.NONE);
     }
 
-  code += ')' + "\n"
+  code += ')' + "\n";
 
   return code
 }

src/editor/extended_python_generator.ts

@@ -178,7 +178,11 @@ export class ExtendedPythonGenerator extends PythonGenerator {
         definitions.push(def);
       }
     }
-    // Call Blockly.CodeGenerator's finish.
+    // Call Blockly.CodeGenerator's finish.  This is required to be done this way
+    // because we derive from PythonGenerator which dervies from CodeGenerator
+    // This section except for the class_def part is all copied from Blockly's
+    // PythonGenerator.  It can't be derived because it needs the class insertion
+    // in the middle.
     code = Blockly.CodeGenerator.prototype.finish(code);
     this.isInitialized = false;
 
@@ -189,7 +193,8 @@ export class ExtendedPythonGenerator extends PythonGenerator {
 
     this.nameDB_!.reset();
     const allDefs = imports.join('\n') + '\n\n' + definitions.join('\n\n');
-    return allDefs.replace(/\n\n+/g, '\n\n').replace(/\n*$/, '\n\n\n') + class_def + this.prefixLines(code, this.INDENT);
+    return allDefs.replace(/\n\n+/g, '\n\n').replace(/\n*$/, '\n\n\n') + class_def + 
+            this.prefixLines(code, this.INDENT);
   }
 }