Implement isinstance #122
Implement isinstance #122
Conversation
Codecov Report
@@ Coverage Diff @@
## master #122 +/- ##
==========================================
- Coverage 72.85% 72.74% -0.12%
==========================================
Files 60 60
Lines 11949 11970 +21
==========================================
+ Hits 8706 8707 +1
- Misses 2709 2729 +20
Partials 534 534
Continue to review full report at Codecov.
|
builtin/builtin.go
Outdated
| var res py.Bool | ||
| var class = args[1].(py.Tuple) | ||
| for idx := range class { | ||
| var new_args []py.Object |
There was a problem hiding this comment.
This would be more efficently written
var new_args = []py.Object{args[0], class[idx]}
builtin/builtin.go
Outdated
| if err != nil { | ||
| return false, err | ||
| } | ||
| res = res || temp |
There was a problem hiding this comment.
This should say
if res {
return res
}to short circuit the rest of the evaluations shouldn't it?
If you do that you can move the definition of res to here.
builtin/builtin.go
Outdated
| } | ||
| res = res || temp | ||
| } | ||
| return res, nil |
There was a problem hiding this comment.
Then this can say
return false, nil
builtin/builtin.go
Outdated
| } | ||
| default: | ||
| { | ||
| if args[0].Type() != py.TypeType { |
There was a problem hiding this comment.
You are checking this for every entry in the args - it would be better checked in builtin_isinstance I think.
|
I think this is looking good now - thank you :-) Apologies for the delay! I'll merge it now. |
#121