Bug 25031

Summary:	[PATCH] LineLayoutManager.getNextBreakPoss fails to add bp to vecInlineBreaks
Product:	Fop - Now in Jira	Reporter:	Simon Pepping <spepping>
Component:	page-master/layout	Assignee:	fop-dev
Status:	CLOSED FIXED
Severity:	normal
Priority:	P3
Version:	trunk
Target Milestone:	---
Hardware:	PC
OS:	Linux

Description Simon Pepping 2003-11-26 20:12:46 UTC

[PATCH] LineLayoutManager.getNextBreakPoss:
If prevBP == null, the bp is accepted after all. It should
now also be added to vecInlineBreaks. The patch does that.

Comment 1 Simon Pepping 2003-11-26 20:36:09 UTC

For some reason which I do not understand my attachment always seems to be
empty. Therefore I submit the patch in this way:

Index: layoutmgr/LineLayoutManager.java
===================================================================
RCS file:
/home/cvspublic/xml-fop/src/java/org/apache/fop/layoutmgr/LineLayoutManager.java,v
retrieving revision 1.5
diff -u -r1.5 LineLayoutManager.java
--- layoutmgr/LineLayoutManager.java	16 Oct 2003 23:54:21 -0000	1.5
+++ layoutmgr/LineLayoutManager.java	26 Nov 2003 19:48:21 -0000
@@ -313,6 +313,7 @@
             return null;
         }
         if (prevBP == null) {
+            vecInlineBreaks.add(bp);
             prevBP = bp;
         }

Comment 2 Glen Mazza 2003-12-01 04:48:16 UTC

Changes made.  Thanks!

Comment 3 Glenn Adams 2012-04-01 06:46:27 UTC

batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed