Minifying javascripts is evil. Let's not do it.

The microformats stuff was unused

plugins/Mapstraction/Makefile

@@ -1,14 +0,0 @@
-.fake: clean all
-
-TARGETS=usermap-mxn-openlayers.min.js
-
-CORE=js/mxn.js js/mxn.core.js
-USERMAP=usermap.js
-
-all: $(TARGETS)
-
-clean:
-	rm -f $(TARGETS)
-
-usermap-mxn-openlayers.min.js: $(CORE) js/mxn.openlayers.core.js $(USERMAP)
-	cat $+ | yui-compressor -o $@ --type=js

plugins/Mapstraction/MapstractionPlugin.php

@@ -116,20 +116,10 @@ class MapstractionPlugin extends Plugin
         default:
             return true;
         }
-
-        if ($this->provider == 'openlayers') {
-            // We have an optimized path for our default case.
-            //
-            // Note that OpenLayers.js needs to be separate, or it won't
-            // be able to find its UI images and styles.
-            $action->script($this->path('usermap-mxn-openlayers.min.js'));
-        } else {
-            $action->script(sprintf('%s?(%s)',
-                                    $this->path('js/mxn.js'),
-                                    $this->provider));
-
-            $action->script($this->path('usermap.js'));
-        }
+        $action->script(sprintf('%s?(%s)',
+                                $this->path('js/mxn.js'),
+                                $this->provider));
+        $action->script($this->path('usermap.js'));
 
         $action->inlineScript(sprintf('var _provider = "%s";', $this->provider));
 

plugins/Mapstraction/usermap.js

@@ -28,15 +28,6 @@ function scrapeUser()
      };
 }
 
-function getMicroformatValue(element)
-{
-    if(element[0].tagName.toLowerCase() == 'abbr'){
-        return element.attr('title');
-    }else{
-        return element.text();
-    }
-}
-
 function getNoticeFromElement(noticeElement)
 {
     var notice = {};