linter

lib/asciidoctor/converter/semantic_html5.rb

@@ -101,16 +101,11 @@ def convert_ulist node
     ret = [%(<ul#{attributes}>)]
     ret << %(<strong class="title">#{node.title}</strong>) if node.title?
     node.items.each do |item|
-    sub_content = if item.blocks?
-      %(
+      ret << %(<li>#{item.text}#{item.blocks? ? %(
 #{item.content}
-)
-    else
-      ""
-    end
-    ret << %(<li>#{item.text}#{sub_content}</li>)
+) : ''}</li>)
     end
-    ret << %{</ul>}
+    ret << %(</ul>)
     ret.join LF
   end
 
@@ -126,48 +121,38 @@ def convert_olist node
     if node.option? 'reversed'
       olist_attributes << %( reversed="true")
     end
-    ret = [%(<ol#{attributes}#{olist_attributes.join('')}>)]
+    ret = [%(<ol#{attributes}#{olist_attributes.join}>)]
     ret << %(<strong class="title">#{node.title}</strong>) if node.title?
     node.items.each do |item|
-    sub_content = if item.blocks?
-      %(
+      ret << %(<li>#{item.text}#{item.blocks? ? %(
 #{item.content}
-)
-    else
-      ""
-    end
-    ret << %(<li>#{item.text}#{sub_content}</li>)
+) : ''}</li>)
     end
-    ret << %{</ol>}
+    ret << %(</ol>)
     ret.join LF
   end
 
   def convert_dlist node
     roles = []
     roles << node.style if node.style
     roles << node.role if node.role
-    role = roles.join " "
+    role = roles.join ' '
     attributes = common_html_attributes node.id, role.empty? ? nil : role
     dlist_attributes = []
     if node.list_marker_keyword
       dlist_attributes << %( type="#{node.list_marker_keyword}")
     end
-    ret = [%(<dl#{attributes}#{dlist_attributes.join('')}>)]
+    ret = [%(<dl#{attributes}#{dlist_attributes.join}>)]
     ret << %(<strong class="title">#{node.title}</strong>) if node.title?
     node.items.each do |terms, desc|
       terms.each do |term|
-          ret << %(<dt>#{term.text}</dt>)
+        ret << %(<dt>#{term.text}</dt>)
       end
-      sub_content = if desc.blocks?
-        %(
+      ret << %(<dd>#{desc.text}#{ desc.blocks? ? %(
 #{desc.content}
-)
-      else
-        ""
-      end
-      ret << %(<dd>#{desc.text}#{sub_content}</dd>)
+) : ''}</dd>)
     end
-    ret << %{</dl>}
+    ret << %(</dl>)
     ret.join LF
   end