Bug 25031 - [PATCH] LineLayoutManager.getNextBreakPoss fails to add bp to vecInlineBreaks
Summary: [PATCH] LineLayoutManager.getNextBreakPoss fails to add bp to vecInlineBreaks
Status: CLOSED FIXED
Alias: None
Product: Fop - Now in Jira
Classification: Unclassified
Component: page-master/layout (show other bugs)
Version: trunk
Hardware: PC Linux
: P3 normal
Target Milestone: ---
Assignee: fop-dev
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-11-26 20:12 UTC by Simon Pepping
Modified: 2012-04-01 06:46 UTC (History)
0 users


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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