erroneous hdfdump output of multiline (heredoc) elements

[Narcelio Filho <narcelio+filho@…>]

Multi-line HDF elements are being generated wrong for hdfdump. HDFWrapper generates this:

foo {
 bar = << EOM
line1
line2
EOM
}

when should generate this:

foo {
 bar << EOM
line1
line2
EOM
}

(without the equal sign)

Clearsilver tool chokes when feed with first example:

$ cs -h test.hdf
Traceback (innermost last):
  File "neo_hdf.c", line 1840, in hdf_read_file()
  File "neo_hdf.c", line 1678, in _hdf_read_string()
    In file test.hdf:3
  File "neo_hdf.c", line 1739, in _hdf_read_string()
ParseError: [test.hdf:3] Unable to parse line line1

Reference: ​ClearSilver HDF Dataset

Working patch (with doctest testcase):

trac/web/clearsilver.py

@@ -259 +259 @@
         add_value(name, value)
 
     def __str__(self):
+        """
+        Returns the HDF dataset
+
+        >>> hdf = HDFWrapper()
+        >>> hdf['test.multiline'] = '''line1
+        ... line2'''
+        >>> print hdf
+        test {
+          multiline << EOM
+        line1
+        line2
+        EOM
+        }
+        """
         from StringIO import StringIO
         buf = StringIO()
         def hdf_tree_walk(node, prefix=''):
@@ -270 +284 @@
                     if value.find('\n') == -1:
                         buf.write(' = %s' % value)
                     else:
-                        buf.write(' = << EOM\n%s\nEOM' % value)
+                        buf.write(' << EOM\n%s\nEOM' % value)
                 if node.child():
                     buf.write(' {\n')
                     hdf_tree_walk(node.child(), prefix + '  ')

[Christian Boos]

We're dropping support for ClearSilver, so there's no real need for that anymore. However, if someone else feels like committing the patch, please do.