Bug 38954 - patch: new functionality which allows working with shapes in HSLF
Summary: patch: new functionality which allows working with shapes in HSLF
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: HSLF (show other bugs)
Version: unspecified
Hardware: All All
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-03-13 17:08 UTC by Yegor Kozlov
Modified: 2006-03-19 09:56 UTC (History)
0 users


Attachments
commit notes (1.01 KB, text/plain)
2006-03-13 17:09 UTC, Yegor Kozlov 		Details
short summary of changes (1.91 KB, text/plain)
2006-03-13 17:09 UTC, Yegor Kozlov 		Details
modified files (54.78 KB, application/zip)
2006-03-13 17:10 UTC, Yegor Kozlov 		Details
patch with changes (20.29 KB, patch)
2006-03-13 17:10 UTC, Yegor Kozlov 		Details | Diff
View All 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 Yegor Kozlov 2006-03-13 17:08:38 UTC 
New functionality which allows working with shapes in HSLF.
 Summary:
 - creating new slides on the fly
 - reading shapes from a ppt slide
 - addning a new shape
 - initial support of Graphics2D in HSLF
Comment 1 Yegor Kozlov 2006-03-13 17:09:16 UTC 
Created attachment 17884 [details]
commit notes
Comment 2 Yegor Kozlov 2006-03-13 17:09:43 UTC 
Created attachment 17885 [details]
short summary of changes
Comment 3 Yegor Kozlov 2006-03-13 17:10:10 UTC 
Created attachment 17886 [details]
modified files
Comment 4 Yegor Kozlov 2006-03-13 17:10:50 UTC 
Created attachment 17887 [details]
patch with changes
Comment 5 Nick Burch 2006-03-14 10:39:30 UTC 
Thanks for this, it looks great! I'll try to give it a full review+commit on the
weekend.
Comment 6 Nick Burch 2006-03-19 17:56:53 UTC 
Thanks for the code Yegor, it looks great :) I've committed it.

I ended up also adding some new records, and methods to RecordContainer. This
allowed me to refactor a few bits of your code, which should hopefully make it
easier to follow. The tests all still pass, so I hope I didn't break anything
with these changes :)

I'll add your example useage stuff to the website in a little bit.