Re: [basex-talk] Bug merging maps with option duplicates:combine?

2020-01-10 Thread Christian Grün
Hi Leo, that’s awesome, thanks a lot! And I can clarify that the problem was introduced by me and the intent to provide support for the duplicates option three years ago. Your original map implementation has been rock solid since its very beginnings. @Johannes: A new stable snapshot will be avail

Re: [basex-talk] Bug merging maps with option duplicates:combine?

2020-01-09 Thread Johannes Echterhoff
, Johannes -Ursprüngliche Nachricht- Von: Leonard Wörteler [mailto:leonard.woerte...@uni-konstanz.de] Gesendet: Donnerstag, 9. Januar 2020 23:40 An: Bridger Dyson-Smith ; Johannes Echterhoff Cc: BaseX Betreff: Re: [basex-talk] Bug merging maps with option duplicates:combine? Hi Bridger

Re: [basex-talk] Bug merging maps with option duplicates:combine?

2020-01-09 Thread Leonard Wörteler
Hi Bridger and Johannes, wow, this is a really old and obscure bug in the `map:merge(...)` logic you found there. It only occurs if both maps to be merged contain hash collisions on the same key, and the two collision buckets are merged together. This is not a common scenario. In a quick searc

Re: [basex-talk] Bug merging maps with option duplicates:combine?

2020-01-09 Thread Bridger Dyson-Smith
Hi all, While I don't have any answer to this question, I can corroborate Johannes' finding. Also, Johannes, did you find a magic string? I tried some longer keys and they seem to work :) Best, Bridger On Thu, Jan 9, 2020 at 11:51 AM Johannes Echterhoff < echterh...@interactive-instruments.de> w

[basex-talk] Bug merging maps with option duplicates:combine?

2020-01-09 Thread Johannes Echterhoff
Hello, When executing: map:merge(( map { "DENW21AL100077Hs": "a", "DENW21AL100076i5": "a" }, map { "DENW21AL100077Hs": "b", "DENW21AL100076i5": "b", "DENW21AL100076hT": "b" } ), map {'duplicates': 'combine'}) in the BaseX GUI I get: Error: Improper use? Potential bug? Your feedback is welcome: C