Factor wrap_in_para_itemlist_markup() out of generate_markup().
authorFabian Keil <fk@fabiankeil.de>
Sun, 31 Oct 2010 13:27:03 +0000 (13:27 +0000)
committerFabian Keil <fk@fabiankeil.de>
Sun, 31 Oct 2010 13:27:03 +0000 (13:27 +0000)
utils/changelog2doc.pl

index bfa23b0..6ed30c0 100755 (executable)
@@ -1,6 +1,6 @@
 #!/usr/bin/perl
 
 #!/usr/bin/perl
 
-# $Id: changelog2doc.pl,v 1.3 2010/10/31 13:25:47 fabiankeil Exp $
+# $Id: changelog2doc.pl,v 1.4 2010/10/31 13:26:07 fabiankeil Exp $
 # $Source: /cvsroot/ijbswa/current/utils/changelog2doc.pl,v $
 
 # Filter to parse the ChangeLog and translate the changes for
 # $Source: /cvsroot/ijbswa/current/utils/changelog2doc.pl,v $
 
 # Filter to parse the ChangeLog and translate the changes for
@@ -50,20 +50,24 @@ sub create_listitem_markup($) {
            "  </listitem>\n";
 }
 
            "  </listitem>\n";
 }
 
+sub wrap_in_para_itemlist_markup($) {
+    my $content = shift;
+    my $markup = "<para>\n" .
+                 " <itemizedlist>\n" .
+                 "  $content" .
+                 " </itemizedlist>\n" .
+                 "</para>\n";
+    return $markup;
+}
+
 sub generate_markup() {
     my $markup = '';
 
 sub generate_markup() {
     my $markup = '';
 
-    $markup .= "<para>\n" .
-               " <itemizedlist>\n";
-
     foreach my $entry (@entries) {
         $markup .= create_listitem_markup(\%{$entry});
     }
 
     foreach my $entry (@entries) {
         $markup .= create_listitem_markup(\%{$entry});
     }
 
-    $markup .= " </itemizedlist>\n" .
-               "</para>\n";
-
-    print $markup;
+    print wrap_in_para_itemlist_markup($markup);
 }
 
 sub main () {
 }
 
 sub main () {