template classes are not imperatively exported

3nids · 3nids · commit 8157cf6e4467 · 2017-05-24T07:22:59.000+02:00
followup a14ad32 fixes win build
diff --git a/scripts/sipify.pl b/scripts/sipify.pl
@@ -410,7 +410,11 @@ sub detect_comment_block{
         do {no warnings 'uninitialized';
             push @CLASSNAME, $3;
             dbg_info("class: ".$CLASSNAME[$#CLASSNAME]);
-            $EXPORTED[-1]++ if $line =~ m/\b[A-Z]+_EXPORT\b/;
+            if ($line =~ m/\b[A-Z]+_EXPORT\b/ || $#CLASSNAME != 0 || $lines[$line_idx-2] =~ m/^\s*template</){
+                # class should be exported except those not at top level or template classes
+                # if class is not exported, then its methods should be (checked whenever leaving out the class)
+                $EXPORTED[-1]++;
+            }
         };
         $line = "$1 $3";
         # Inheritance
@@ -490,7 +494,7 @@ sub detect_comment_block{
                     pop(@global_bracket_nesting_index);
                     pop(@ACCESS);
                     die "Class $CLASSNAME[$#CLASSNAME] in $headerfile at line $line_idx should be exported with appropriate [LIB]_EXPORT macro. If this should not be available in python, wrap it in a `#ifndef SIP_RUN` block."
-                        if $EXPORTED[-1] == 0 && $#CLASSNAME == 0;
+                        if $EXPORTED[-1] == 0;
                     pop @EXPORTED;
                 }
                 pop(@CLASSNAME);
diff --git a/src/core/qgsfeatureiterator.h b/src/core/qgsfeatureiterator.h
@@ -180,7 +180,7 @@ class CORE_EXPORT QgsAbstractFeatureIterator
  * \note not available in Python bindings (although present in SIP file)
 */
 template<typename T>
-class CORE_EXPORT QgsAbstractFeatureIteratorFromSource : public QgsAbstractFeatureIterator
+class QgsAbstractFeatureIteratorFromSource : public QgsAbstractFeatureIterator
 {
   public:
     QgsAbstractFeatureIteratorFromSource( T *source, bool ownSource, const QgsFeatureRequest &request )

Original file line number	Diff line number	Diff line change
`@@ -180,7 +180,7 @@ class CORE_EXPORT QgsAbstractFeatureIterator`
`180`	`180`	`* \note not available in Python bindings (although present in SIP file)`
`181`	`181`	`*/`
`182`	`182`	`template<typename T>`
`183`		`-class CORE_EXPORT QgsAbstractFeatureIteratorFromSource : public QgsAbstractFeatureIterator`
	`183`	`+class QgsAbstractFeatureIteratorFromSource : public QgsAbstractFeatureIterator`
`184`	`184`	`{`
`185`	`185`	`public:`
`186`	`186`	`QgsAbstractFeatureIteratorFromSource( T *source, bool ownSource, const QgsFeatureRequest &request )`