Adam Retter

Results 574 comments of Adam Retter

> OK, so then have a look at the test in this PR as it is changed to address your review. I'm not seeing that in this PR.

> Well it is the other changed file. No it isn't. It does not fully address my review and is still missing several things.

@reinhapa just to make you aware that the native binaries would need to be compiled and added to this PR before it is merged

@PieterLamers This is definitely a bug and not an enhancement; I have modified the labels. I can also confirm that this happens in `7.0.0-SNAPSHOT` also. I also note that BaseX...

@nverwer If I remember correctly there are some issues with Lazy Evaluation of variables in eXist-db. Additionally, this should really be caught in the static analysis phase; that would be...

@nverwer The cache that your traces are showing is that of compiled XQuery Modules (and not the Cache XQuery Extension Module that is available via the `cache:*` functions). When eXist-db...

So I took a quick look into this. Unfortunately the root of the problem is that eXist-db's parser and XQuery engine does not correctly understand `.` i.e. the Context Item,...

@wolfgangmm any comments, perhaps you see an alternative path for fixing this?

As Erik mentions, the temporary workaround is: `count((true(), false(), false())[. eq false()])`

I fixed a lot of these issues sometime ago now. Should all work, what exactly is the issue?