On 2016-04-12 12:41:05 -0700, Kevin J. McCarthy wrote:
> I'm attaching a slightly revised part1 patch, and adding another patch
> to the series.  The first patch is changed so that the spacing can be
> invoked when pad=0.  (There still might be a spacing issue for that case.)
> 
> The second patch fixes softpad when the pad character itself is multicolumn.

In this second patch:

+              pad = ((signed)(destlen - wlen - len)) / pl;

The cast to "signed" seems useless and confusing: what is its goal?

-- 
Vincent Lefèvre <[email protected]> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / AriC project (LIP, ENS-Lyon)

Reply via email to