On 04/10/2012 12:46 PM, Manuel López-Ibáñez wrote:
+ max_width = context->caret_max_width; + if (max_width<= 0) + max_width = INT_MAX;
I don't think we need the test here; diagnostic_set_caret_max_width should make sure caret_max_width is set sensibly. Otherwise, yes, thanks.
On 04/10/2012 02:52 PM, Gabriel Dos Reis wrote:
There is a novelty in this new version that I don't think we discussed before: automatic expansion of tabs to 8 hard space characters. That number should not be hardcoded as there is no reason to believe a tab character always expands to 8 space characters. You should check the environment first; if not present the default expansion number should be a symbolic constant as opposed to a magic number sprinkled all over the places.
Hmm. I don't know if there's any reliable way to query tab stops; the "it" termcap/terminfo capability tells you what it was originally (presumably 8), but it might have changed.
Jason