Hi oldk1331,

(should I call you oldk1331?)

I tried it and it seems to work OK.

Here is a test case:
)set output algebra off
)set output html on
a=b

Without this condition 'a','=' and 'b' would appear on separate lines but both with and without the patch they are correctly combined onto one line. (although whitespace is not relevant for html it make it easier to work with).

However something looks funny about the patch, do you need both => and return void()$Void ? is the 'return' redundant? Its been a long time and I may have forgotten something.

Martin

On 28/03/18 03:16, oldk1331 wrote:
Sorry about that.  Can you review a small cleanup in html.spad?

--- a/src/algebra/html.spad
+++ b/src/algebra/html.spad
@@ -380,12 +380,8 @@
        if enableGrid then
          if tagName = "table" then enableGrid := false
          if tagName = "tr" then enableGrid := false
-      b : List Boolean := [leaf?(c1) for c1 in c]
        -- if all children are strings then no need to wrap in table
-      allString : Boolean := true
-      for c1 in c repeat if not leaf?(c1) then allString := false
-      if allString then
-        s:String := ""
-        for c1 in c repeat s := concat(s, value(c1))
+      every?(leaf?, c) =>
+        s : String := concat [value c1 for c1 in c]
          sayHt s
          if value(t) ~= "" then sayHt concat["</",tagName,">"]


--
You received this message because you are subscribed to the Google Groups "FriCAS - 
computer algebra system" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/fricas-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to