On Sun, Apr 28, 2013 at 08:12:31AM -0500, Vladimir Támara Patiño wrote: > On Fri, Apr 26, 2013 at 01:04:26PM +0200, Stefan Sperling wrote: > >Do you want to try implementing this idea? > >I believe this idea would result in a smaller diff, too. > > Attached,
Thanks, very nice. I've committed the parts which fix the mb_cur_max issue. The diff looked almost exactly as I expected, minus some redundant commas and whitespace issues that I cleaned up before commit. > with reordering of #includes in setrunelocale.c to comply > better with style (9). I'll look at these #include changes later. I'd prefer if you kept patches stricly on-topic, and started a new thread for any unrelated changes. That makes it easier for us to review and keep track of your contributions. I know that CVS and diff(1) can sometimes make it hard to keep unrelated changes in separate patches. So if you have any questions about how to do this best in particular cases feel free to ask for help.
