<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" content="1" /><style type="text/css">p, li { white-space: pre-wrap; }</style></head><body style=" font-family:'DejaVu Sans'; font-size:9pt; font-weight:400; font-style:normal;">On Thursday 05 March 2009, Dmitry Kazakov wrote:<br>
> PS:<br>
> And an issue for the dispute: why do we have such a strange API?<br>
><br>
> The first issue is toXML/toLegacyXML stuff. These functions are implemented<br>
> in _different_ classes although they do the same things. More than that,<br>
> both of them are called from different parts of krita: bookmark manager<br>
> uses toXML, kis_kra_ uses toLegacyXML Why not have the only one? The only<br>
> united interface?<br>
.kra file are expected to use the toLegacyXML because this functions saves to an XML formated for the 1.6 .kra files. While toXML use the OpenRaster style, described in http://create.freedesktop.org/wiki/index.php?title=OpenRaster/Layers_Stack_Specification .<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>I suppose we could have used the OpenRaster style in .kra files as well, but then we would have had to make migration path from the old style to the OR style, which also have the drawback to be not finished, since only the XML is specified, but the values and parameters isn't specified at the moment, even if it will be very inspired by the SVG spec.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>> And the second issue is defaultConfigurations of the filter. The specific<br>
> Kis<smth>FilterConfiguration class for the filter can be (and is) created<br>
> in two different places:<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>I will reorder the question :)<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>> Speaking truly i can't understand the meaning of these factory-classes at<br>
> all =(<br>
If you are not familiar with the factory method pattern, I suggest to read http://en.wikipedia.org/wiki/Factory_method_pattern . The main idea here is that 95% of filters can live happilly with the KisFilterConfiguration class, they just have to read a few integers, and then they can filter a paint device. But for some filters it's not good enough, and they need to have a 'cache' of some information, like the transfert curve (or the KoColorTransformation, but that cache was removed sometime ago since it triggered issue with multithreading, but it might be worth to restore it, sometime), hence the need of the custom KisFilterConfiguration.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>> KisBaseProcessor::defaultConfiguration(const KisPaintDeviceSP pd)<br>
The documentation says all. It's the configuration that will be used in the filter dialog when the user start the dialog.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>You forgot that way as well:<br>
KisFilterConfiguration * KisBaseProcessor::factoryConfiguration(const KisPaintDeviceSP) const<br>
whose name sucks and is a bit confusing (if anyone has a better idea, the name comes from what you can sometime see on some devices: "reset to factory settings", but here it might be confusing in the middle of the factory pattern), it allows filters developers to provide the user of the filter with some sane initial values for the first run.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>> and<br>
> KisFilterConfigurationFactory::createDefault(). Why?! The same thing with<br>
> KisFilterConfigurationFactory::create(...). It's a clone. Why is it so?<br>
Filter configurations are saved in KisBookmarkedConfigurationManager which is a generic class which can be used with all kind of KisSerializableConfiguration to store them, this can be filters/generators configuration, paintop settings, ... The only way KisBookmarkedConfigurationManager can creates the correct subclass of KisSerializableConfiguration is by using the KisFilterConfigurationFactory.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>The difference between createDefault and create, is that create is used to unserialized configuration, and createDefault is used to create the first to use default configuration (now that I have said that, I note that there is a bug in the implementation, since it means KisBaseProcessor::factoryConfiguration is never called, and I think that KisFilterConfigurationFactory::createDefault should call KisBaseProcessor::factoryConfiguration).<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>I hope it makes things a little bit more clear :)<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>-- <br>
Cyrille Berger</p></body></html>