grub-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] create variables when exporting them


From: Vladimir 'φ-coder/phcoder' Serbinenko
Subject: Re: [PATCH] create variables when exporting them
Date: Mon, 21 Dec 2009 15:21:37 +0100
User-agent: Mozilla-Thunderbird 2.0.0.22 (X11/20091109)

Andreas Born wrote:
> I just noticed that I forgot the header line of the diff. Sorry for that.
> New double checked version attached.
>
Applied with stylistic fixes
> The Changelog is still:
> 2009-12-15 Andreas Born  <address@hidden>
>
>      * kern/env.c (grub_env_export): Create nonexistent variables
> before exporting.
>
> Andreas Born schrieb:
>> Same here. Nothing changed as of r1917 and I didn't get any further
>> feedback either.
>> I've attached a rediffed version, too.
>>
>> Andreas Born schrieb:
>>> Here you go:
>>> ChangeLog:
>>> 2009-10-15  Andreas Born  <address@hidden>
>>>
>>>       * kern/env.c (grub_env_export): Create nonexistent variables
>>> before exporting.
>>>
>>>
>>> The mixed indentation (tabs and spaces) in kern/env.c was a bit
>>> confusing.
>>>
>>> Andreas
>>>
>>> Colin Watson schrieb:
>>>> On Thu, Oct 15, 2009 at 08:40:37PM +0200, Andreas Born wrote:
>>>>  
>>>>> This patch changes grub_env_export to create variables with empty
>>>>> value,  if necessary.
>>>>> This makes it possible to export variables before actually
>>>>> assigning  them any value and is the way bash behaves.
>>>>>     
>>>>
>>>> This makes sense to me (although bash is "too big and too slow" and
>>>> we'll never implement anything close to all of it, this is a pretty
>>>> cheap way to reduce confusion due to differences), although
>>>> probably for
>>>> post-1.97. A couple of nits:
>>>>
>>>>  
>>>>>        * kern/env.c (grub_env_export): Create inexistent
>>>>> variables  before exporting.
>>>>>     
>>>>
>>>> "nonexistent"
>>>>
>>>>  
>>>>> +  if (! var)
>>>>> +  {
>>>>> +    if (grub_env_set (name, "") != GRUB_ERR_NONE)
>>>>> +        return grub_errno;
>>>>> +    var = grub_env_find (name);
>>>>> +  }        
>>>>
>>>> GNU brace style involves indenting the braces as well, thus:
>>>>
>>>>   if (! var)
>>>>     {
>>>>       if (grub_env_set (name, "") != GRUB_ERR_NONE)
>>>>           return grub_errno;
>>>>       var = grub_env_find (name);
>>>>     }     
>>>
>>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Grub-devel mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/grub-devel


-- 
Regards
Vladimir 'φ-coder/phcoder' Serbinenko


Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

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