Browse Source

run optimization issue tests again

Simon Krajewski 10 years ago
parent
commit
3e015f0025

+ 1 - 1
tests/optimization/run.hxml

@@ -15,5 +15,5 @@
 --macro Macro.register('Test')
 --macro Macro.register('TestJs')
 --macro Macro.register('TestLocalDce')
-#--macro Macro.register('issues')
+--macro Macro.register('issues')
 -dce std

+ 6 - 1
tests/optimization/src/Macro.hx

@@ -47,6 +47,9 @@ class Macro {
 			case TInst(c, _): c.get();
 			case _: Context.error('$className should be a class', Context.currentPos());
 		}
+		#if !js_unflatten
+		className = className.replace(".", "_");
+		#end
 		var fields = [];
 		function checkField(cf:ClassField) {
 			if (cf.meta.has(":js")) {
@@ -62,7 +65,9 @@ class Macro {
 			++tests;
 			if (output != field.js) {
 				++failures;
-				Context.warning('$output should be ${field.js}', field.pos);
+				Context.warning('Test failed', field.pos);
+				Context.warning('Expected: ' + field.js, field.pos);
+				Context.warning('Actual  : ' +output, field.pos);
 			}
 		}
 	}

+ 0 - 1
tests/optimization/src/issues/Issue2236.hx

@@ -38,7 +38,6 @@ class Issue2236 {
 		var _g_pos = 0;
 		while(_g_pos < _g_a.length) {
 			var x = _g_a[_g_pos++];
-			x;
 		}
 	')
 	static function test() {

+ 2 - 1
tests/optimization/src/issues/Issue3713.hx

@@ -4,6 +4,7 @@ private class A<T> {
     public function new() {}
 }
 
+@:native("BImpl")
 private abstract B<T>(A<T>) {
     public function new() this = new A();
     public inline function f():C<T> return new C(this);
@@ -18,7 +19,7 @@ private class C<T> {
 
 class Issue3713 {
 	@:js('
-		var b = issues._Issue3713.B_Impl_._new();
+		var b = BImpl._new();
 		var c_x = 1;
 	')
 	@:analyzer(no_const_propagation, no_local_dce, no_check_has_effect)