Dekel Tsur <[EMAIL PROTECTED]> writes:

| I've already manage to do (1) + (3). A patch is attached.

I am currently looking at it.

+{
+       const max_numer_of_lines = 40;       

is not allowed,

        int const max_number_of_lines = 40;

is.

+       if (!IsDummy()) {
+               if (isRightToLeftPar())
+                       reverse(&s[len],s.end());
+       }       

is dangerous. Alternate approach is

        if (!IsDummy()) {
                if (isRightToLeftPar())
                        reverse(s.begin() + len, s.end());
        }


Can you do those changes and send a new patch?

I would also like to have others views on this.
        
        Lgb

Reply via email to