octave-maintainers
[Top][All Lists]
Advanced

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

Re: [Changeset] new bzip2 function


From: Jaroslav Hajek
Subject: Re: [Changeset] new bzip2 function
Date: Mon, 5 Jan 2009 10:43:09 +0100

On Thu, Jan 1, 2009 at 11:32 AM, Thorsten Meyer <address@hidden> wrote:
> Hi,
>
> John W. Eaton wrote:
>> +      error (sprintf("%s: extension has to be a string with finite length",
>> +                     commandname));
>>
>> There's no need to use sprintf here since error already understands
>> format strings.  So you should write
>>
>>   error ("%s: extension has to be a string with finite length", commandname);
>>
> Thanks for the hint.
>
> Attached, you will find a little patch which fixes the error messages in
> __xzip__ and adds a few tests of the error handling.
>
> regards
>
> Thorsten
>

Applied.

thanks


-- 
RNDr. Jaroslav Hajek
computing expert
Aeronautical Research and Test Institute (VZLU)
Prague, Czech Republic
url: www.highegg.matfyz.cz


reply via email to

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