drop post-processing support of "export *"

Summary:
Sickle now processes "export *" at input type, before JS emit.

If we end up generating JS that uses __export(), we've already
lost, because Closure doesn't know about the types of all the
exported variables.

Part of issue #111.

Reviewers: rkirov, alexeagle

Reviewed By: alexeagle

Subscribers: typescript-eng

Differential Revision: https://reviews.angular.io/D32

Author: evmar

src/sickle.ts

@@ -988,18 +988,17 @@ export function convertCommonJsToGoogModule(
   // All require statements must be assigned.
   // import 'z'          ==> var unused_xxx = require('z')
   // import {x} from 'y' ==> var ... = require('y')
-  // export * from 'x'   ==> __export(require('x'))
   let unusedIdx = 0;
   content = content.replace(
-      /(((?:^|;)\s*)|= |__export\()require\(["']([^'";]+)['"]\)(\))?;/gm,
-      (match, prefix, leading, modName, suffix = '') => {
+      /(((?:^|;)\s*)|= )require\(["']([^'";]+)['"]\);/gm,
+      (match, prefix, leading, modName) => {
         modName = pathToModuleName(fileName, modName);
         referencedModules.push(modName);
         if (prefix === leading) {
           // No prefix ==> side effect style "import 'foo';".
           prefix = prefix + 'var unused_' + unusedIdx++ + '_ = ';
         }
-        return `${prefix}goog.require('${modName}')${suffix};`;
+        return `${prefix}goog.require('${modName}');`;
       });
 
   return {output: content, referencedModules};

test/sickle_test.ts

@@ -189,11 +189,6 @@ require('req/mod');`)
 var unused_0_ = goog.require('req$mod');`);
   });
 
-  it('converts export * statements', () => {
-    expectCommonJs('a.js', `__export(require('req/mod'));`)
-        .to.equal(`goog.module('a');__export(goog.require('req$mod'));`);
-  });
-
   it('resolves relative module URIs', () => {
     // See below for more fine-grained unit tests.
     expectCommonJs('a/b.js', `var r = require('./req/mod');`)