[Toybox] [PATCH] ls: Ensure file names are separated by 2 spaces

[Andrew Ilijic]

These two patches should resolve the 2 spaces issue.
https://www.mail-archive.com/toybox@lists.landley.net/msg06039.html
https://www.mail-archive.com/toybox@lists.landley.net/msg06040.html

~Andrew

On Tue, Oct 22, 2019 at 9:40 AM Andrew Ilijic <ilijic.andrew at gmail.com> wrote:
>
> Hi Rob,
>
> I have not submitted a second patch yet. My next patch will include
> the two new tests mentioned previously and a fix for the trailing
> spaces. I had planned to do it yesterday but then found out that I
> have a job interview coming up and I need to study and prepare for it.
>
> Respectfully,
> Andrew
>
> On Mon, Oct 21, 2019 at 6:12 PM Rob Landley <rob at landley.net> wrote:
> >
> > On 10/18/19 2:59 PM, Andrew Ilijic wrote:
> > > Hi Enh,
> > >
> > > (Below are the new tests and the failure output)
> > > I created 2 test cases to do a basic test of `-x` and `-C`. These
> > > tests pass for the host `ls` but not for Toybox's `ls`. You cant see
> > > the issue in `diff` but I recreated the test and `hexdump -C
> > > <test_file>` shows a trailing 1 - 2 spaces, depending on if you are on
> > > `master` or my patch. I need to modify my patch so that it does not
> > > add the 2 trailing spaces.
> >
> > I committed your patch yesterday, it should be in master already?
> >
> > https://github.com/landley/toybox/commit/0b34a5637d91
> >
> > (And then I left this window open over the weekend and now it's monday. I don't
> > remember when I committed your patch.)
> >
> > > How should I submit these changes, all at once or multiple patches?
> >
> > I haven't seen your patch to remove the trailing spaces? (Maybe it's in the
> > email I haven't read yet, but you didn't send it before this email...)
> >
> > Ro