Cy 'kkm' Katsnelson
Cy 'kkm' Katsnelson
> I wonder if that kPath test is really constexpr because it seems that it's both the condition and it's negation is evaluated as true If you look closer at...
> IIRC that was something they had before but for some reason, they changed to enable_if semantics (perhaps saving a couple of instructions?. That cannot save any this way, if...
I am getting this error in 3 headers from compiling this one file. Should not be a big patch. When you compile full fstscript, are there more than this?
Let me come up with a "minimal" patch, so that it would be easy to maintain.
Looks like enable_if is used only in these 3 includes (the 4th use in util.h is rather valid).
There. That file compiled. Please give the change a glance over if you have time: https://github.com/kkm000/openfst/commit/7b7db521175b956210a2ed197b408ef2ae224731?diff=unified I went for half-indent of the if's to keep most code lines intact.
This is a new change in 1.6.5; here's a relevant comment from the old version: ``` // TODO(kbg): Make this a compile-time static_assert once: // 1) All weight properties are...
Thanks, quite an interesting post! I need to go through it carefully. A very interesting note in the SO post that you sent me, exactly [this answer](https://stackoverflow.com/a/26753794/1149924) about templates having...
Oh, and what they *really* want is probably just `static_assert` :)