diff options
author | David Schleef <ds@schleef.org> | 2005-02-09 20:50:35 +0000 |
---|---|---|
committer | David Schleef <ds@schleef.org> | 2005-02-09 20:50:35 +0000 |
commit | 6b55d8f1eb59ac5c780295da2bcfe84aa6d15488 (patch) | |
tree | 7a19677776b1a1eaad178c6fcf78770267f1a743 /tests/old/testsuite | |
parent | c3118a7382a90a0698d257c7cc815f2dbd11a2c0 (diff) |
testsuite/gst-lint: Check for non-statically scoped parent_class variables. This won't be a problem once plugins are...
Original commit message from CVS:
* testsuite/gst-lint: Check for non-statically scoped
parent_class variables. This won't be a problem once
plugins are loaded with RTLD_LOCAL.
Diffstat (limited to 'tests/old/testsuite')
-rwxr-xr-x | tests/old/testsuite/gst-lint | 17 |
1 files changed, 16 insertions, 1 deletions
diff --git a/tests/old/testsuite/gst-lint b/tests/old/testsuite/gst-lint index 4b99a1da9..a924f052d 100755 --- a/tests/old/testsuite/gst-lint +++ b/tests/old/testsuite/gst-lint @@ -48,6 +48,7 @@ sub check_explicit_caps(); sub check_signals(); sub check_gettext(); sub check_padtemplate(); +sub check_parent_class(); sub m_check_plugindir(); sub m_check_interfaces(); @@ -78,6 +79,7 @@ foreach $filename (<FIND>) { check_signals(); check_gettext(); check_padtemplate(); + check_parent_class(); # less important stuff check_license(); @@ -548,7 +550,20 @@ sub check_padtemplate() { foreach $line (@lines){ if ($line =~ /GstStaticPadTemplate/ && !($line =~ /static/)) { - print "W: pad template definitions should be static\n" + print "W: pad template definitions should be static\n"; + return; + } + } +} + +# +# Check that parent_class is statically scoped +# +sub check_parent_class() +{ + foreach $line (@lines){ + if ($line =~ /Gst.*\*\s*parent_class/ && !($line =~ /static/)) { + print "E: parent_class definitions should be static\n"; return; } } |