Don't do setter bypass if it's not really necessary.

2019-03-03 12:18:40 +01:00
parent ac908bc82d
commit 56c2ed9642
1 changed files with 5 additions and 5 deletions
--- a/src/tink/macro/Constructor.hx
+++ b/src/tink/macro/Constructor.hx
@@ -96,12 +96,12 @@ class Constructor {
      }
      
    var tmp = MacroApi.tempName();
+    var member = owner.memberByName(name).sure();
    
-    if (options.bypass) {
-      switch owner.memberByName(name) {
-        case Success(member): member.addMeta(':isVar');
-        default:
-      }
+    if (options.bypass && member.kind.match(FProp(_, 'never' | 'set', _, _))) {
+      
+      member.addMeta(':isVar');
+
      addStatement((function () {
        var fields = [for (f in  (macro this).typeof().sure().getClass().fields.get()) f.name => f];