[libvirt] [PATCH 17/30] remote: Remove virConnectPtr from error/errorf

[Eric Blake]

On 04/12/2010 09:03 AM, Dave Allan wrote:
>>>> +        remoteError(VIR_ERR_INVALID_ARG, "%s",
>>>> +                    _("transport methods unix, ssh and ext are not supported "
>>>> +                      "under Windows"));
>>>
>>> I see why you broke this line, to fit 80 columns, but that can impact
>>> grep-ability of the original message.  Is there any policy on this?
>>>
>>
>> I'm not aware of any policy for this, but we have many error messages
>> split into multiple lines already.
> 
> I agree that it impacts grep-ability in the code, but very long lines
> impact the readability of the code, so in general, I'm in favor of
> splitting lines.

Good to know; thanks for the answers.

Meanwhile, I've seen this paradigm in some other projects (glibc,
coreutils, ...), to at least reduce the number of splits due to long
messages:

         remoteError(VIR_ERR_INVALID_ARG, "%s", _("\
transport methods unix, ssh and ext are not supported under Windows"));

By using the \-newline combo, and starting the message at column 1,
you've bought yourself quite a bit of line length, without any change to
the resulting message either on screen or in the .pot file.

But I'm not bothered enough by the current situation to spend any of my
time reformatting existing messages to a new format, so for now, I'm
happy leaving things as they are.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 323 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100412/89e6b4ac/attachment-0001.sig>