Skip to content
GitLab
Menu
Projects
Groups
Snippets
/
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
Menu
Open sidebar
violethaze74
appstream
Commits
5abfc83a
Commit
5abfc83a
authored
Aug 13, 2019
by
Matthias Klumpp
Browse files
validator: Correctly read line numbers in long files
parent
fe2894b8
Changes
5
Hide whitespace changes
Inline
Side-by-side
src/as-validator-issue.c
View file @
5abfc83a
...
...
@@ -41,7 +41,7 @@ typedef struct
gchar
*
fname
;
gchar
*
cid
;
g
int
line
;
g
long
line
;
AsIssueKind
kind
;
/* deprecated */
}
AsValidatorIssuePrivate
;
...
...
@@ -306,7 +306,7 @@ as_validator_issue_set_cid (AsValidatorIssue *issue, const gchar *cid)
*
* Returns: the line number where this issue occured, or -1 if unknown.
**/
g
int
g
long
as_validator_issue_get_line
(
AsValidatorIssue
*
issue
)
{
AsValidatorIssuePrivate
*
priv
=
GET_PRIVATE
(
issue
);
...
...
@@ -321,7 +321,7 @@ as_validator_issue_get_line (AsValidatorIssue *issue)
* Sets the line number where this issue was found.
**/
void
as_validator_issue_set_line
(
AsValidatorIssue
*
issue
,
g
int
line
)
as_validator_issue_set_line
(
AsValidatorIssue
*
issue
,
g
long
line
)
{
AsValidatorIssuePrivate
*
priv
=
GET_PRIVATE
(
issue
);
priv
->
line
=
line
;
...
...
@@ -385,7 +385,7 @@ as_validator_issue_get_location (AsValidatorIssue *issue)
g_string_append_printf
(
location
,
":%s"
,
priv
->
cid
);
if
(
priv
->
line
>=
0
)
{
g_string_append_printf
(
location
,
":%i"
,
priv
->
line
);
g_string_append_printf
(
location
,
":%
l
i"
,
priv
->
line
);
}
return
g_string_free
(
location
,
FALSE
);
...
...
src/as-validator-issue.h
View file @
5abfc83a
...
...
@@ -145,9 +145,9 @@ const gchar *as_validator_issue_get_filename (AsValidatorIssue *issue);
void
as_validator_issue_set_filename
(
AsValidatorIssue
*
issue
,
const
gchar
*
fname
);
g
int
as_validator_issue_get_line
(
AsValidatorIssue
*
issue
);
g
long
as_validator_issue_get_line
(
AsValidatorIssue
*
issue
);
void
as_validator_issue_set_line
(
AsValidatorIssue
*
issue
,
g
int
line
);
g
long
line
);
gchar
*
as_validator_issue_get_location
(
AsValidatorIssue
*
issue
);
...
...
src/as-validator.c
View file @
5abfc83a
...
...
@@ -168,7 +168,7 @@ as_validator_add_issue (AsValidator *validator, xmlNode *node, const gchar *tag,
as_validator_issue_set_cid
(
issue
,
as_component_get_id
(
priv
->
current_cpt
));
if
(
node
!=
NULL
)
as_validator_issue_set_line
(
issue
,
node
->
line
);
as_validator_issue_set_line
(
issue
,
xmlGetLineNo
(
node
)
);
location
=
as_validator_issue_get_location
(
issue
);
id_str
=
g_strdup_printf
(
"%s/%s/%s"
,
...
...
src/as-xml.c
View file @
5abfc83a
...
...
@@ -337,7 +337,7 @@ as_xml_add_description_node_helper (AsContext *ctx, xmlNode *root, xmlNode **des
doc
=
xmlReadMemory
(
xmldata
,
strlen
(
xmldata
),
NULL
,
"utf-8"
,
XML_PARSE_NOBLANKS
|
XML_PARSE_NONET
);
XML_PARSE_NOBLANKS
|
XML_PARSE_NONET
|
XML_PARSE_BIG_LINES
);
if
(
doc
==
NULL
)
{
ret
=
FALSE
;
goto
out
;
...
...
tools/ascli-actions-validate.c
View file @
5abfc83a
...
...
@@ -38,17 +38,17 @@ create_issue_info_print_string (AsValidatorIssue *issue)
const
AsIssueSeverity
severity
=
as_validator_issue_get_severity
(
issue
);
const
gchar
*
tag
=
as_validator_issue_get_tag
(
issue
);
const
gchar
*
cid
=
as_validator_issue_get_cid
(
issue
);
const
gchar
line
=
as_validator_issue_get_line
(
issue
);
const
gchar
*
hint
=
as_validator_issue_get_hint
(
issue
);
glong
line
=
as_validator_issue_get_line
(
issue
);
if
(
cid
==
NULL
)
{
if
(
line
>=
0
)
location
=
g_strdup_printf
(
"~:%i"
,
line
);
location
=
g_strdup_printf
(
"~:%
l
i"
,
line
);
else
location
=
g_strdup
(
"~:~"
);
}
else
{
if
(
line
>=
0
)
location
=
g_strdup_printf
(
"%s:%i"
,
cid
,
line
);
location
=
g_strdup_printf
(
"%s:%
l
i"
,
cid
,
line
);
else
location
=
g_strdup_printf
(
"%s:~"
,
cid
);
}
...
...
Write
Preview
Supports
Markdown
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment