Re: Changes in hyphenation

Johannes Hofmann <[email protected]>
Newsgroups gmane.comp.web.dillo.devel
Message-ID <[email protected]>
Hi Sebastian,

On Mon, Oct 29, 2012 at 08:08:16PM +0100, Sebastian Geerken wrote:
> Hi!
> 
> I just pulled some change to make <pre> work. Please test it again,
> even things which have worked recently, so that we can be sure that
> nothing is broken again.

Works very nice for me! With the following change:

diff -r 5e7a06855f37 dw/textblock.hh
--- a/dw/textblock.hhMon Oct 29 20:05:20 2012 +0100
+++ b/dw/textblock.hhMon Oct 29 22:23:15 2012 +0100
@@ -552,8 +552,7 @@
    {
       if (style->whiteSpace != core::style::WHITE_SPACE_NOWRAP &&
           style->whiteSpace != core::style::WHITE_SPACE_PRE)
-         if (word->badnessAndPenalty.lineMustBeBroken())
-            word->badnessAndPenalty.setPenalty (0);
+         word->badnessAndPenalty.setPenalty (0);
    }
 
    void addHyphen();



ideographic characters also seem to work again, but I'm not sure
about all the side-effects.
furaisanjin, could you please test this?

Cheers,
Johannes
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.