David Bears

I ran into this issue, and I am surprised that it is actually intended behavior since it is not intuitive. What worked for me is to create a method with...

Will [`Traversable.toMap`](https://www.javadoc.io/static/io.vavr/vavr/1.0.0-alpha-4/io/vavr/collection/Traversable.html#toMap-java.util.function.Function-java.util.function.Function-java.util.function.BiFunction-) do what you want?

By this I assume you mean [`Traversable.groupBy`](https://www.javadoc.io/static/io.vavr/vavr/1.0.0-alpha-4/io/vavr/collection/Traversable.html#groupBy-java.util.function.Function-). Since this returns a `Map`, which doesn't necessarily guarantee order, I would assume that any apparent ordering is a coincidence. Furthermore, I would...

This is also an issue when using the time-keeping feature. ``` # tmp.ldg i 2024-02-02 00:00:00 A o 2024-02-02 01:00:00 2024-02-02 A = 0h B ``` ``` $ ledger --args-only...

It is possible that copying the cost should go in post_t::copy_details instead of in xact_t::extend_xact. But it is not entirely clear to me what the copy_details method is supposed to...

Hey, I'm not sure why this PR never got reviewed and also PR #2274 , but I'd like to bring them to the surface. @afh

@tdenny What issue are you seeing exactly? From your example, it doesn't look like you are using automated transactions.

@tdenny > I'm seeing the error `transaction does not balance` when I used commodities in an explicit transaction. I'm fairly certain my problems are a result of trying to use...

Maybe a silly question, but why do we want to compile with C++11 by default? Instead of manually adding the -std flag with `add_compile_options`, we could use the [`CXX_STANDARD`](https://cmake.org/cmake/help/latest/prop_tgt/CXX_STANDARD.html) parameter....

Another minor annoyance to highlight what I believe is the atrocity of `trimQuotes` enabled. Suppose an end user wants an entire option value to not be split and it includes...