diff options
| author | Thomas Haller <thaller@redhat.com> | 2019-12-16 17:41:16 +0100 |
|---|---|---|
| committer | Thomas Haller <thaller@redhat.com> | 2019-12-16 17:42:23 +0100 |
| commit | ec0adbfaf0175fc7f54b3f4a6035240a837651e6 (patch) | |
| tree | f660a87f157c1779eb42a1028cb264441274c9c2 | |
| parent | 9e02a676196300370487b39cbcff4e28d6c5f27c (diff) | |
| download | NetworkManager-ec0adbfaf0175fc7f54b3f4a6035240a837651e6.tar.gz | |
checkpatch: catch "gs_free GError *" declations
| -rwxr-xr-x | contrib/scripts/checkpatch.pl | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/contrib/scripts/checkpatch.pl b/contrib/scripts/checkpatch.pl index bf1ccb1ccc..98693f8dc3 100755 --- a/contrib/scripts/checkpatch.pl +++ b/contrib/scripts/checkpatch.pl @@ -189,6 +189,7 @@ complain ("Don't add Emacs editor formatting hints to source files") if $line_no complain ("XXX marker are reserved for development while work-in-progress. Use TODO or FIXME comment instead?") if $line =~ /\bXXX\b/; complain ("This gtk-doc annotation looks wrong") if $line =~ /\*.*\( *(transfer-(none|container|full)|allow none) *\) *(:|\()/; complain ("Prefer nm_assert() or g_return*() to g_assert*()") if $line =~ /g_assert/ and not $filename =~ /\/tests\//; +complain ("Use gs_free_error with GError variables") if $line =~ /\bgs_free\b +GError *\*/; new_hunk if $_ eq ''; my ($this_indent) = /^(\s*)/; |