Falling through is a dangerous pattern, let's not do it.

git-svn-id: svn+ssh://src.earth.threerings.net/vilya/trunk@116 c613c5cb-e716-0410-b11b-feb51c14d237

build.xml

@@ -169,7 +169,9 @@
       <exclude name="**/ModPlayer.java"/>
       <exclude name="**/MidiPlayer.java"/>
       <exclude name="**/Mp3Player.java"/>
-      <!--<compilerarg value="-Xlint:unchecked"/>-->
+      <compilerarg value="-Xlint"/>
+      <compilerarg value="-Xlint:-serial"/>
+      <compilerarg value="-Xlint:-unchecked"/> <!-- TODO: fix -->
     </javac>
   </target>
 

src/java/com/threerings/stage/tools/editor/PortalTool.java

@@ -68,9 +68,10 @@ public class PortalTool extends MouseInputAdapter
         switch (event.getButton()) {
         case MouseEvent.BUTTON1:
             savePortal();
-            // fall through to BUTTON3
+            break;
 
         case MouseEvent.BUTTON3:
+            savePortal();
             dispose();
             break;
         }