grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] treat multiple slashes like one for JFS


From: Robert Millan
Subject: Re: [PATCH] treat multiple slashes like one for JFS
Date: Sun, 28 Sep 2008 15:32:39 +0200
User-agent: Mutt/1.5.13 (2006-08-11)

On Thu, Sep 25, 2008 at 05:17:06PM +0200, Felix Zielcke wrote:
> Here's a patch to handle things like //boot/// for JFS which doestn't do
> it currently but Linux does it.
> 
> This was already brought up 2004 but I wonder why this isn't included
> http://lists.gnu.org/archive/html/grub-devel/2004-08/msg00043.html
> 
> Probable this can be right now commited but as always I just want to be
> on the safe side.

Assuming you tested it works, I don't see anything wrong in the code.

> +      while (*next == '/')
> +     {
> +       next[0] = '\0';
> +       next++;
> +     }

This seems to incorporate a one-liner of code from Tomas Ebenlendr, I'd
recommend to mention that in the changelog.

-- 
Robert Millan

  The DRM opt-in fallacy: "Your data belongs to us. We will decide when (and
  how) you may access your data; but nobody's threatening your freedom: we
  still allow you to remove your data and not access it at all."




reply via email to

[Prev in Thread] Current Thread [Next in Thread]