The following review has been posted through the commitfest application:
make installcheck-world:  not tested
Implements feature:       not tested
Spec compliant:           not tested
Documentation:            tested, passed

a) I'm wondering if we should apply one more change to this page. The 
line-by-line listing of operators occupies much space. What do you think about 
an inline list (in a separate line) of the operators?

old source:
    <simplelist>
<member><literal>&lt;</literal></member>
<member><literal>&lt;=</literal></member>
<member><literal>=</literal></member>
<member><literal>&gt;=</literal></member>
<member><literal>&gt;</literal></member>
    </simplelist>

new source:
   <synopsis>&lt; &nbsp; &lt;= &nbsp; = &nbsp; &gt;= &nbsp; &gt;</synopsis>
It looks nice in HTML as well as in PDF.

b) I'm in favor of the indentation of all affected lines as it is done in the 
patch.

The new status of this patch is: Waiting on Author

Reply via email to