From 8d9f7ba8a42ec1cfd39e6c249aef15e9295fe0a1 Mon Sep 17 00:00:00 2001
From: Matej Habrnal <mhabrnal@redhat.com>
Date: Tue, 13 Jan 2015 19:23:08 -0500
Subject: [PATCH 1001/1015] lib: created a new lib file for reporters
Moved some functions from rhbz.c to src/lib/reporters.c and src/lib/strbuf.c.
Related to #272
Signed-off-by: Matej Habrnal <mhabrnal@redhat.com>
---
po/POTFILES.in | 1 +
src/include/Makefile.am | 3 +-
src/include/internal_libreport.h | 5 +++
src/include/reporters.h | 36 ++++++++++++++++++
src/lib/Makefile.am | 3 +-
src/lib/reporters.c | 80 ++++++++++++++++++++++++++++++++++++++++
src/lib/strbuf.c | 60 ++++++++++++++++++++++++++++++
src/plugins/rhbz.c | 78 +--------------------------------------
src/plugins/rhbz.h | 2 -
9 files changed, 188 insertions(+), 80 deletions(-)
create mode 100644 src/include/reporters.h
create mode 100644 src/lib/reporters.c
diff --git a/po/POTFILES.in b/po/POTFILES.in
index 4246e06..003e686 100644
--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -27,6 +27,7 @@ src/lib/parse_options.c
src/lib/problem_data.c
src/lib/problem_report.c
src/lib/reported_to.c
+src/lib/reporters.c
src/lib/run_event.c
src/plugins/abrt_rh_support.c
src/plugins/report_Bugzilla.xml.in.in
diff --git a/src/include/Makefile.am b/src/include/Makefile.am
index 87e5e60..4d8c6a5 100644
--- a/src/include/Makefile.am
+++ b/src/include/Makefile.am
@@ -15,7 +15,8 @@ libreport_include_HEADERS = \
file_obj.h \
internal_libreport.h \
internal_abrt_dbus.h \
- xml_parser.h
+ xml_parser.h \
+ reporters.h
if BUILD_UREPORT
libreport_include_HEADERS += ureport.h
diff --git a/src/include/internal_libreport.h b/src/include/internal_libreport.h
index cf5730c..a867649 100644
--- a/src/include/internal_libreport.h
+++ b/src/include/internal_libreport.h
@@ -99,6 +99,7 @@ int vdprintf(int d, const char *format, va_list ap);
#include "workflow.h"
#include "file_obj.h"
#include "libreport_types.h"
+#include "reporters.h"
#ifdef __cplusplus
extern "C" {
@@ -108,6 +109,10 @@ extern "C" {
int prefixcmp(const char *str, const char *prefix);
#define suffixcmp libreport_suffixcmp
int suffixcmp(const char *str, const char *suffix);
+#define trim_all_whitespace libreport_trim_all_whitespace
+char *trim_all_whitespace(const char *str);
+#define shorten_string_to_length libreport_shorten_string_to_length
+char *shorten_string_to_length(const char *str, unsigned length);
#define strtrim libreport_strtrim
char *strtrim(char *str);
#define strtrimch libreport_strtrimch
diff --git a/src/include/reporters.h b/src/include/reporters.h
new file mode 100644
index 0000000..d415b7f
--- /dev/null
+++ b/src/include/reporters.h
@@ -0,0 +1,36 @@
+/*
+ Copyright (C) 2014 ABRT team
+ Copyright (C) 2014 RedHat Inc
+
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License along
+ with this program; if not, write to the Free Software Foundation, Inc.,
+ 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+*/
+
+#ifndef REPORTERS_H
+#define REPORTERS_H
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+#define is_comment_dup libreport_is_comment_dup
+int is_comment_dup(GList *comments, const char *comment);
+#define comments_find_best_bt_rating libreport_comments_find_best_bt_rating
+unsigned comments_find_best_bt_rating(GList *comments);
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif
diff --git a/src/lib/Makefile.am b/src/lib/Makefile.am
index c11a42d..d41e543 100644
--- a/src/lib/Makefile.am
+++ b/src/lib/Makefile.am
@@ -58,7 +58,8 @@ libreport_la_SOURCES = \
xml_parser.c \
libreport_init.c \
global_configuration.c \
- uriparser.c
+ uriparser.c \
+ reporters.c
libreport_la_CPPFLAGS = \
-I$(srcdir)/../include \
diff --git a/src/lib/reporters.c b/src/lib/reporters.c
new file mode 100644
index 0000000..e3305ca
--- /dev/null
+++ b/src/lib/reporters.c
@@ -0,0 +1,80 @@
+/*
+ String buffer implementation
+
+ Copyright (C) 2015 RedHat inc.
+
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2 of the License, or
+ (at your option) any later version.
+
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ GNU General Public License for more details.
+
+ You should have received a copy of the GNU General Public License along
+ with this program; if not, write to the Free Software Foundation, Inc.,
+ 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+*/
+
+#include "internal_libreport.h"
+
+int
+is_comment_dup(GList *comments, const char *comment)
+{
+ char * const trim_comment = trim_all_whitespace(comment);
+ bool same_comments = false;
+
+ for (GList *l = comments; l && !same_comments; l = l->next)
+ {
+ const char * const comment_body = (const char *) l->data;
+ char * const trim_comment_body = trim_all_whitespace(comment_body);
+ same_comments = (strcmp(trim_comment_body, trim_comment) == 0);
+ free(trim_comment_body);
+ }
+
+ free(trim_comment);
+ return same_comments;
+}
+
+unsigned
+comments_find_best_bt_rating(GList *comments)
+{
+ if (comments == NULL)
+ return 0;
+
+ unsigned best_rating = 0;
+ for (GList *l = comments; l; l = l->next)
+ {
+ char *comment_body = (char *) l->data;
+
+ char *start_rating_line = strstr(comment_body, FILENAME_RATING": ");
+ if (!start_rating_line)
+ {
+ log_debug(_("Note does not contain rating"));
+ continue;
+ }
+
+ start_rating_line += strlen(FILENAME_RATING": ");
+
+ errno = 0;
+ char *e;
+ long rating = strtoul(start_rating_line, &e, 10);
+ /*
+ * Note: we intentionally check for '\n'. Any other terminator
+ * (even '\0') is not ok in this case.
+ */
+ if (errno || e == start_rating_line || (*e != '\n' && *e != '\r') || (unsigned long)rating > UINT_MAX)
+ {
+ /* error / no digits / illegal trailing chars */
+ continue;
+ }
+
+ if (rating > best_rating)
+ best_rating = rating;
+ }
+
+ return best_rating;
+}
+
diff --git a/src/lib/strbuf.c b/src/lib/strbuf.c
index ef8bda8..f0cd1b8 100644
--- a/src/lib/strbuf.c
+++ b/src/lib/strbuf.c
@@ -37,6 +37,66 @@ int suffixcmp(const char *str, const char *suffix)
return strcmp(str + len_minus_suflen, suffix);
}
+char *trim_all_whitespace(const char *str)
+{
+ char *trim = xzalloc(sizeof(char) * strlen(str) + 1);
+ int i = 0;
+ while (*str)
+ {
+ if (!isspace(*str))
+ trim[i++] = *str;
+ str++;
+ }
+
+ return trim;
+}
+
+/* If str is longer than max allowed length then
+ * try to find first ' ' from the end of acceptable long str string
+ *
+ * If ' ' is found replace string after that by "..."
+ *
+ * If ' ' is NOT found in maximal allowed range, cut str string on
+ * lenght (MAX_SUMMARY_LENGTH - strlen("...")) and append "..."
+ *
+ * If MAX_LENGTH is 15 and max allowed cut is 5:
+ *
+ * 0123456789ABCDEF -> 0123456789AB...
+ * 0123456789 BCDEF -> 0123456789 ...
+ * 012345 789ABCDEF -> 012345 789AB...
+ */
+char *
+shorten_string_to_length(const char *str, unsigned length)
+{
+ char *dup_str = xstrdup(str);
+ if (strlen(str) > length)
+ {
+ char *max_end = dup_str + (length - strlen("..."));
+
+ /* maximal number of characters to cut due to attempt cut dup_str
+ * string after last ' '
+ */
+ int max_cut = 16;
+
+ /* start looking for ' ' one char before the last possible character */
+ char *buf = max_end - 1;
+ while (buf[0] != ' ' && max_cut--)
+ --buf;
+
+ if (buf[0] != ' ')
+ buf = max_end;
+ else
+ ++buf;
+
+ buf[0] = '.';
+ buf[1] = '.';
+ buf[2] = '.';
+ buf[3] = '\0';
+ }
+
+ return dup_str;
+}
+
/*
* Trims whitespace characters both from left and right side of a string.
* Modifies the string in-place. Returns the trimmed string.
diff --git a/src/plugins/rhbz.c b/src/plugins/rhbz.c
index a227c62..fdcfff9 100644
--- a/src/plugins/rhbz.c
+++ b/src/plugins/rhbz.c
@@ -133,41 +133,6 @@ static GList *rhbz_comments(struct abrt_xmlrpc *ax, int bug_id)
return g_list_reverse(comments);
}
-static char *trim_all_whitespace(const char *str)
-{
- func_entry();
-
- char *trim = xzalloc(sizeof(char) * strlen(str) + 1);
- int i = 0;
- while (*str)
- {
- if (!isspace(*str))
- trim[i++] = *str;
- str++;
- }
-
- return trim;
-}
-
-int is_comment_dup(GList *comments, const char *comment)
-{
- func_entry();
-
- char * const trim_comment = trim_all_whitespace(comment);
- bool same_comments = false;
-
- for (GList *l = comments; l && !same_comments; l = l->next)
- {
- const char * const comment_body = (const char *) l->data;
- char * const trim_comment_body = trim_all_whitespace(comment_body);
- same_comments = (strcmp(trim_comment_body, trim_comment) == 0);
- free(trim_comment_body);
- }
-
- free(trim_comment);
- return same_comments;
-}
-
static unsigned find_best_bt_rating_in_comments(GList *comments)
{
func_entry();
@@ -553,46 +518,7 @@ int rhbz_new_bug(struct abrt_xmlrpc *ax,
if (!duphash) duphash = problem_data_get_content_or_NULL(problem_data,
"global_uuid");
- /* If summary is longer than max allowed summary length then
- * try to find first ' ' from the end of acceptable long summary string
- *
- * If ' ' is found replace string after that by "..."
- *
- * If ' ' is NOT found in maximal allowed range, cut summary string on
- * lenght (MAX_SUMMARY_LENGTH - strlen("...")) and append "..."
- *
- * If MAX_SUMMARY_LENGTH is 15 and max allowed cut is 5:
- *
- * 0123456789ABCDEF -> 0123456789AB...
- * 0123456789 BCDEF -> 0123456789 ...
- * 012345 789ABCDEF -> 012345 789AB...
- */
- char *summary = NULL;
- if (strlen(bzsummary) > MAX_SUMMARY_LENGTH)
- {
- summary = xstrdup(bzsummary);
- char *max_end = summary + (MAX_SUMMARY_LENGTH - strlen("..."));
-
- /* maximal number of characters to cut due to attempt cut summary
- * string after last ' '
- */
- int max_cut = 16;
-
- /* start looking for ' ' one char before the last possible character */
- char *buf = max_end - 1;
- while (buf[0] != ' ' && max_cut--)
- --buf;
-
- if (buf[0] != ' ')
- buf = max_end;
- else
- ++buf;
-
- buf[0] = '.';
- buf[1] = '.';
- buf[2] = '.';
- buf[3] = '\0';
- }
+ char *summary = shorten_string_to_length(bzsummary, MAX_SUMMARY_LENGTH);
char *status_whiteboard = xasprintf("abrt_hash:%s", duphash);
@@ -604,7 +530,7 @@ int rhbz_new_bug(struct abrt_xmlrpc *ax,
abrt_xmlrpc_params_add_string(&env, params, "product", product);
abrt_xmlrpc_params_add_string(&env, params, "component", component);
abrt_xmlrpc_params_add_string(&env, params, "version", version);
- abrt_xmlrpc_params_add_string(&env, params, "summary", (summary ? summary : bzsummary));
+ abrt_xmlrpc_params_add_string(&env, params, "summary", summary);
abrt_xmlrpc_params_add_string(&env, params, "description", bzcomment);
abrt_xmlrpc_params_add_string(&env, params, "status_whiteboard", status_whiteboard);
diff --git a/src/plugins/rhbz.h b/src/plugins/rhbz.h
index 15e7699..86632a3 100644
--- a/src/plugins/rhbz.h
+++ b/src/plugins/rhbz.h
@@ -105,8 +105,6 @@ int rhbz_attach_blob(struct abrt_xmlrpc *ax, const char *bug_id,
int rhbz_attach_fd(struct abrt_xmlrpc *ax, const char *bug_id,
const char *att_name, int fd, int flags);
-int is_comment_dup(GList *comments, const char *comment);
-
GList *rhbz_bug_cc(xmlrpc_value *result_xml);
struct bug_info *rhbz_bug_info(struct abrt_xmlrpc *ax, int bug_id);
--
1.8.3.1