optimize compiler warnings
This commit is contained in:
+10
-10
@@ -100,7 +100,7 @@ class BindExt {
|
|||||||
try {
|
try {
|
||||||
c = warnPrepareChain(start, macro $i { zeroListener }, pre, true);
|
c = warnPrepareChain(start, macro $i { zeroListener }, pre, true);
|
||||||
} catch (e:bindx.Error) {
|
} catch (e:bindx.Error) {
|
||||||
Context.warning('${start.toString()} is not bindable.', e.pos);
|
//Context.warning('${start.toString()} is not bindable.', e.pos);
|
||||||
//e.contextWarning();
|
//e.contextWarning();
|
||||||
}
|
}
|
||||||
if (c != null) {
|
if (c != null) {
|
||||||
@@ -123,24 +123,23 @@ class BindExt {
|
|||||||
var i = 0;
|
var i = 0;
|
||||||
while (i < keys.length) {
|
while (i < keys.length) {
|
||||||
var k = keys[i];
|
var k = keys[i];
|
||||||
var j = i;
|
|
||||||
var remove = false;
|
var remove = false;
|
||||||
while (++j < keys.length) if (keys[j].startsWith(k)) {
|
var j = i;
|
||||||
remove = true;
|
while (!remove && ++j < keys.length) remove = keys[j].startsWith(k);
|
||||||
break;
|
|
||||||
}
|
|
||||||
if (remove) keys = keys.splice(i, 1); else i++;
|
if (remove) keys = keys.splice(i, 1); else i++;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
var msg = [];
|
||||||
for (k in keys) {
|
for (k in keys) {
|
||||||
var data = binded.get(k);
|
var data = binded.get(k);
|
||||||
Context.warning('Bind ${data.c.expr.toString()}', data.c.expr.pos);
|
msg.push(data.c.expr.toString());
|
||||||
chain.bind.unshift(data.prebind);
|
chain.bind.unshift(data.prebind);
|
||||||
var c = data.c;
|
var c = data.c;
|
||||||
chain.init = chain.init.concat(c.init);
|
chain.init = chain.init.concat(c.init);
|
||||||
chain.bind = chain.bind.concat(c.bind);
|
chain.bind = chain.bind.concat(c.bind);
|
||||||
chain.unbind = chain.unbind.concat(c.unbind);
|
chain.unbind = chain.unbind.concat(c.unbind);
|
||||||
}
|
}
|
||||||
|
Context.warning('Bind \'${msg.join("', '")}\'', expr.pos);
|
||||||
|
|
||||||
var zeroListener = listenerName(0, "");
|
var zeroListener = listenerName(0, "");
|
||||||
|
|
||||||
@@ -172,12 +171,14 @@ class BindExt {
|
|||||||
var prevField = {e:first.e, field:first.field, error:null};
|
var prevField = {e:first.e, field:first.field, error:null};
|
||||||
var fields:Array<FieldExpr> = [ { field:first.field, bindable:first.error == null, e:first.e } ];
|
var fields:Array<FieldExpr> = [ { field:first.field, bindable:first.error == null, e:first.e } ];
|
||||||
|
|
||||||
while (true) {
|
var end = false;
|
||||||
|
while (!end) {
|
||||||
|
end = true;
|
||||||
var field = Bind.checkField(prevField.e);
|
var field = Bind.checkField(prevField.e);
|
||||||
if (field.field != null) {
|
if (field.field != null) {
|
||||||
fields.push( { field:field.field, bindable:field.error == null, e:field.e } );
|
fields.push( { field:field.field, bindable:field.error == null, e:field.e } );
|
||||||
|
end = false;
|
||||||
} else if (field.error != null) {
|
} else if (field.error != null) {
|
||||||
var end = true;
|
|
||||||
switch (prevField.e.expr) {
|
switch (prevField.e.expr) {
|
||||||
case ECall(e, params):
|
case ECall(e, params):
|
||||||
field = Bind.checkField(e);
|
field = Bind.checkField(e);
|
||||||
@@ -186,7 +187,6 @@ class BindExt {
|
|||||||
end = false;
|
end = false;
|
||||||
case _:
|
case _:
|
||||||
}
|
}
|
||||||
if (end) break;
|
|
||||||
}
|
}
|
||||||
else if (field.e == null) {
|
else if (field.e == null) {
|
||||||
throw new FatalError('${prevField.e.toString()} is not bindable.', prevField.e.pos);
|
throw new FatalError('${prevField.e.toString()} is not bindable.', prevField.e.pos);
|
||||||
|
|||||||
Reference in New Issue
Block a user