You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pdfbox.apache.org by "Tilman Hausherr (JIRA)" <ji...@apache.org> on 2014/06/02 14:07:01 UTC
[jira] [Updated] (PDFBOX-2068) Add filter parameter to
PDImageXObject(document, filteredStream) constructor
[ https://issues.apache.org/jira/browse/PDFBOX-2068?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Tilman Hausherr updated PDFBOX-2068:
------------------------------------
Description:
I am adding a third parameter to
{code}
public PDImageXObject(PDDocument document, InputStream filteredStream)
{code}
i.e. changing it to
{code}
public PDImageXObject(PDDocument document, InputStream filteredStream, COSBase cosFilter)
{code}
because in the code, the filter is always set afterwards. My change improves code clarity. The caller _knows_ what filter was used because he used it to prepare the filteredStream content.
WDYT about also adding width, height, bpc and colorspace to that constructor? These four parameters are always used.
This cool guy (enter his name on youtube) has arguments to use constructors parameters instead of setters:
http://misko.hevery.com/2009/02/19/constructor-injection-vs-setter-injection/
IMHO, mixing constructor initialization and setter initialization looks confusing.
was:
I am adding a third parameter to
{code}
public PDImageXObject(PDDocument document, InputStream filteredStream)
{code}
i.e. changing it to
{code}
public PDImageXObject(PDDocument document, InputStream filteredStream, COSBase cosFilter)
{code}
because in the code, the filter is always set afterwards. My change improves code clarity. The caller _knows_ what filter was used because he used it to prepare the filteredStream content.
WDYT about also adding width, height, bpc and colorspace to that constructor? These four paramaters are always used.
This cool guy (enter his name on youtube) has arguments to use constructors parameters instead of setters:
http://misko.hevery.com/2009/02/19/constructor-injection-vs-setter-injection/
IMHO, mixing constructor initialization and setter initialization looks confusing.
> Add filter parameter to PDImageXObject(document, filteredStream) constructor
> -----------------------------------------------------------------------------
>
> Key: PDFBOX-2068
> URL: https://issues.apache.org/jira/browse/PDFBOX-2068
> Project: PDFBox
> Issue Type: Improvement
> Affects Versions: 2.0.0
> Reporter: Tilman Hausherr
> Assignee: Tilman Hausherr
> Priority: Minor
> Fix For: 2.0.0
>
>
> I am adding a third parameter to
> {code}
> public PDImageXObject(PDDocument document, InputStream filteredStream)
> {code}
> i.e. changing it to
> {code}
> public PDImageXObject(PDDocument document, InputStream filteredStream, COSBase cosFilter)
> {code}
> because in the code, the filter is always set afterwards. My change improves code clarity. The caller _knows_ what filter was used because he used it to prepare the filteredStream content.
> WDYT about also adding width, height, bpc and colorspace to that constructor? These four parameters are always used.
> This cool guy (enter his name on youtube) has arguments to use constructors parameters instead of setters:
> http://misko.hevery.com/2009/02/19/constructor-injection-vs-setter-injection/
> IMHO, mixing constructor initialization and setter initialization looks confusing.
--
This message was sent by Atlassian JIRA
(v6.2#6252)