On Wednesday 15 of August 2012 09:37:19 Jiri Moskovcak wrote:
> On 08/14/2012 06:19 PM, Jakub Filak wrote:
> > Signed-off-by: Jakub Filak <[email protected]>
> > ---
> >
> > src/plugins/rhbz.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/src/plugins/rhbz.c b/src/plugins/rhbz.c
> > index 9c9ea34..09de3d6 100644
> > --- a/src/plugins/rhbz.c
> > +++ b/src/plugins/rhbz.c
> > @@ -841,6 +841,13 @@ int rhbz_attach_files(struct abrt_xmlrpc *ax, const
> > char *bug_id,>
> > if (is_in_string_list(name, (char**)g_not_attached_files))
> >
> > continue;
> >
> > + /* Skip empty files */
> > + if (strlen(value->content) == 0)
> > + {
> > + VERB2 log("Skipping empty file '%s'", name);
>
> - please remove the VERB and use just log(_("Skip...
> - to avoid questions like "why isn't this log file attached?"
> - don't forget to mark it as translatable _
>
Thank you for the 'don't forget' note :) I forgot
> > + continue;
> > + }
> > +
> >
> > if (value->flags & CD_FLAG_TXT)
> > {
> >
> > const char *content = value->content;