[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