Jump to conversation
Unresolved conversations (3)
@ximion ximion Jul 19, 2017
I wonder whether we should iterate over `wantedIconSizes` here, instead of defining another array (minor thing though).
src/asgen/engine.d
@ximion ximion Jul 19, 2017
You could make this just `themedir["scale"].get!int;`, D doesn't need the brackets here (that particular code is rather old from when I didn't know that much about D yet ^^) But that's just a style-thing, no need to change it.
src/asgen/handlers/iconhandler.d
@ximion ximion Jul 16, 2017
This doesn't look right... That was an 32x32@2 size will be ordered before an 128x128 size. I don't think this has practical implications, but we should likely resolve that so nobody wonders about this in future.
Outdated
src/asgen/utils.d
Resolved conversations (0)