(0024537)
neil (administrator)
2013-11-30 00:34
edited on: 2013-11-30 00:37
|
I like the idea; several comments on the patch:
First, the display is a little ugly with the text unindented. What about, as dpeg suggested in Tavern, removing the parenthetical description. Then you could let the list take up to two lines, and keep the second line indented. Something like:
Recite against which type of sinner?
[a] - Chaotic (2: a green ugly thing, a white very ugly thing)
[b] - Heretic (7: an orc, an orc sorcerer, an orc priest,
an orc warrior, an orc, an orc)
The repeated "an orc" is kind of ugly. Perhaps identical names could be collapsed into something like "an orc x3"? Doing genus factoring (as in come-into-view messages) might be a bit too much, as it would give just "7 orcs".
Also, use strwidth(s.c_str()) rather than s.length(), or it won't handle unicode properly: length() returns the number of bytes, which these days has little to do with how much space it takes to print. It's not so much of an issue now (no monsters have non-ASCII names), but will be when we implement more translation.
It would be nice to not hardcode a width of 80, so tiles and bigterm players don't have wasted space. Unfortunately there's not a simple way to get the message window width. msgwin_line_length() would do it, but it would need to be made non-static and added to a header.
Finally, please use spaces instead of tabs; checkwhite can do the conversion, but I don't think it handles your four-space tabs properly, so it would be better to configure your editor to do it for you if possible.
|
(0024545)
Sandman25 (reporter)
2013-11-30 03:03
|
Sorry, I am not a C++ programmer so the code can be very ugly, for example I used google to find how C++ arrays are created and string length is detemined. Please let me know if you want me to continue working on the patch, I am not sure how much time it can take. |
recite.patch [^] (1,733 bytes) 2013-11-27 22:00 [Show Content]
