Jan Kuhlmann
Jan Kuhlmann
Do make sure to destroy the allocator prior to the device according to deinitialization order as mentioned by @adam-sawicki-a ; not doing so is only case where I seem to...
I think I've found a good signature to use and wanted to ask your take on it, as it's similar but not exactly the same as what you had: ```cpp...
The bar width was basically just a placeholder for a graph-specific option. Thought it might be nice to draw each histogram bar 2 characters wide in some cases. > Should...
They won't cause any conflict, since I've not yet implemented proper drawing. Gonna use those once I do! I've put some more thought into the intermediate representation and decided to...
The hex value of each dot really threw me off.. I had falsely assumed it was linear, i.e. `0x1 0x2 0x4 0x8` for the first column.. Anyways, the two functions...
> I can add some inline comments, if you are ready for that. Sounds good! As a side note, do you know why exactly the test on ubuntu 20.04 with...
`VULKAN_HPP_DISPATCH_LOADER_DYNAMIC` is now also a cmake generation time option https://github.com/KhronosGroup/Vulkan-Hpp/blob/efa6cf3a255e138d24e0bf408f93d1a893129a20/CMakeLists.txt#L52 which is why I bring this up. Every option checking for `VULKAN_HPP_SAMPLES_BUILD_ONLY_DNYAMIC` and so on could essentially be replaced by...
> One change I have been considering is replacing cout with an instance of [ostringstream](https://en.cppreference.com/w/cpp/io/basic_ostringstream) and then doing a just single cout at the end. I do not expect that...
> I updated the libraries to replace cout with ostringstream, as well as added a new output stream option to change the output destination. That looks pretty good! > Note...
> Note that char_index would not work, as there are multiple arrays it is indexing into depending on the configured options and not all of the strings that are used...