Bump up 0.7.4

Change-Id: I6a39af13ee6a87994f697830e0f18d1bf2e8cc7f
Signed-off-by: jykeon <jykeon@samsung.com>

svg_loader: memleak prevention

If image href set more than once, the memory was not freed.

Change-Id: Iff8edacb94d4da93dfbd81c7505c535ea675d28c

svg_loader: preventing memcpy from a nullptr

In a case when a polygon/polyline had no points, there is nothing
to be copied.

Change-Id: I6644c3984668d1a2f566bc755a0b47e2804d076e

svg_loader: fixing cyclic cloning of nodes

The reference node doesn't have to be placed inside the defs block,
so when refering to it, it is necessary to search the entire node tree.
In case of invalid svg file this can lead to circular references.
Added prevention against such a sutuation.

Change-Id: Ia572dfbe344939d25611d130b636c84a73ba213d

svg_loader: fixing memleak

Memory was not freed before reallocation (grad->ref can be overwritten
without freeing memory first) - fixed.

Change-Id: I0ab6c931fb29c680b96bbce27f4a6edfdf244104

svg_loader: postponed nodes properly cloned

In case when no defs tag was used, no child node was found.
Also the node attributes should not override those set in the ancestor.

Change-Id: Iff14ce38a17a264bcb71b6a2a2be897e8837ca1f

updated copyright date.

Change-Id: I00af27fbb4170f365bece9e5d2bc218c9a3de9cb

Bump up 0.7.3

Change-Id: I2cdb51b9c327911667d202986f1c36bef5d67e6b
Signed-off-by: jykeon <jykeon@samsung.com>

Merge "svg_loader: changed the order of if conditions" into tizen

Merge "svg_loader: the color attribute properly inherited" into tizen

Merge "svg_loader: removing repeated lines" into tizen

Merge "svg_loader: fix grad update" into tizen

svg_loader: changed the order of if conditions

The weekest condition should be checked as the last one.
Otherwise CDATA marker was never found.

Change-Id: I42cbbe867484cb88b156e182c0452a8cb59cc53d

svg_loader: the color attribute properly inherited

The color attribute hat to be inherited regardles whether it is used
in the child node or not - it may be used i.e. in the grandchild node.

Change-Id: If2895ec64c9ed7f7e176cb32cb4adc8a61f63551

svg_loader: removing repeated lines

The clip-path and the mask attributes are both parsed in _attrParseGNode
function, which is called in th _attrParseUseNode.

Change-Id: Ibef6256e25d63d52571949ead21241e3e4346bef

svg_loader: fix grad update

The grad update should be handled after the postponed nodes are cloned.

Change-Id: I7391d416a81a9cb465ea9a01f5360c126abc53a2

Bump up 0.7.2

Change-Id: I0d11c0712aa2e1c85ff8e0cb8a10feaf0fb2b64b
Signed-off-by: jykeon <jykeon@samsung.com>

Merge "svg_loader: preventing memory leak" into tizen

Merge "svg_loader: LumaMask used only for masks other than white" into tizen

Merge "svg_loader: typo fixed" into tizen

Merge "svg_loader: mask-type attribute introduced" into tizen

Merge "svg_loader: ++robustness" into tizen

svg_loader: preventing memory leak

A memory leak occured when the 'id' attribute was given multiple times
for a given gradient element. Fixed.

Change-Id: I386565d016c23172dc6b2628ef0f0c158a461417

svg_loader: LumaMask used only for masks other than white

For the performance reasons, the LumaMask is used only when
it's necessary - when the mask color is other than white.
Otherwise the AlphaMask is enough.

Change-Id: Ia97fb753854435e2e58eeb1ead40dbb3a5ead7cc

svg_loader: typo fixed

_svgLoaderParerXmlClose -> _svgLoaderParserXmlClose

Change-Id: I24a3fc61e52c74507dc6dd62ddda63f97ed3d05f

svg_loader: mask-type attribute introduced

In an svg file the mask-type attribute can be specified.
It takes one of the two values: Luminosity or Alpha.
After the LumaMask is introduced into TVG, this attribute can be
properly read by the svg loader.

Change-Id: I8de1c94aed6bc0e7b31c1026b57f3336f29c2e30

svg_loader: ++robustness

prevent a crash with an exceptional handling.

@Issue: https://github.com/Samsung/thorvg/issues/1131

Change-Id: If45a5ae201165ba97f087cdeb8c6e35d97adca72

Bump up 0.7.1

Change-Id: I3410b3abbbebcb42a71a8c24bf392001da65290f
Signed-off-by: jykeon <jykeon@samsung.com>

sw_engine raster: fix a regression bug.

Picture example were broken by 90fa26b7bb75cb938290170882e65da8d9fc5d9e

the correct condition must be like this change.

Change-Id: Ib3119a832118221c115cbc6ba9cf5f96ab7625b4

sw_engine SwRaster: Initialize uninitialized transform value

Change-Id: I12e6a63cebbd765c19e83194587722a19db06fa2

svg_loader: composite node splitted on mask and clip nodes

The SvgCompositeNode is replaced by the SvgMaskNode and SvgClipNode.
This is needed for using Luma/AlphaMask in the svg loader and in the future,
when we introduce other mask's features.

Change-Id: Ieb90615da7dedcabe296950af20b59ac5d81e6ba

example: added LumaMasking.cpp (#621)

Added an example LumaMasking.cpp that tests CompositeMethod::LumaMask

Change-Id: I30d6fde4a209b8a818ad2240025832f44b8d4f84

compositeMethod: introduced LumaMask

Introduced CompositeMethod::LumaMask that converts the source pixels to the
grayscale (luma value) before alpha blending. Thanks to it, mask works more like
typical mask in graphics editor software.
Grayscale is calculated with  weighted method:
(0.0721*B + 0.7154*G + 0.2125*R) * A
Introduced surface->blender.lumaValue function

Change-Id: Ia0258e2f374aa27acaef9b2f4161af5b04400422

Bump up 0.7.0

Change-Id: I94c0b425e13a8fae232d1ba9c63cd012ebda157c
Signed-off-by: jykeon <jykeon@samsung.com>

common: Introduced Accessor for traversing the scene-tree.

Basically, this Accessor is a utility to debug the Scene structure,
by traversing the scene-tree by users.

You can search specific nodes to read the property information,
figure out the structure of the scene tree and its size.

We actually don't recommend you to touch the property unless you really
know the each paint's position and role because it's not visible, difficult to
understand its anatomy.

Also, You must underatnd that modifying the nodes of the scene will be going
well with both the art-design structure and the prorgram logic.

In this first version, Accessor only supports for the Picture class.

@example:

auto picture = tvg::Picture::gen();
picture->load("test.svg");

//The callback function from lambda expression.
//This function will be called for every paint nodes of the tree.
auto f = [](const tvg::Paint* paint) -> bool
{
    if (paint->identifier() == Shape::identifier()) {
        //override properties?
        uint8_t r, g, b, a;
        paint->fillColor(&r, &g, &b, &a);
        paint->fill(r / 2, g / 2, b / 2, a);
    }

    //You can return false, to stop traversing immediately.
    return true;
};

auto accessor = tvg::Accessor::gen();

picture = accessor->access(move(picture), f);

...

@Issue: https://github.com/Samsung/thorvg/issues/693

Change-Id: I4f46754de46dfab9bb5b14f58cc659237344822a

Revert "common: introduce iterator"

This reverts commit e947fef9a4170b601cb92b868ae71dfd675a9173.

Bad... This patch was wrongly applied...

Change-Id: I5af1b3755ff5a0b44028fc664d8fd8c80c3ed3cd

sw_engine texmap: fix trivial compiler warnings.

Change-Id: I3649d84e83c257df6d51429c52a88a1fc9d6fdd1
warning: suggest braces around initialization of subobject [-Wmissing-braces]

sw_engine texmap: ++safety

Prevent range over just in case.

Change-Id: I27ed449d1292bc8e191d0a6fb751ab27ea3ee125

Hide the thorvg LZW implementation.

Change-Id: I9fc4049b0d3dd04f22e6327d44c71d663096e366

sw_engine raster: fix a crash at the texmap clipping.

Handle correctly duplicated spans from the multiple y span data.

Previous logic only expected the one single y span data from the rle.
However rle might have multiple y span data if the anti-aliasing is applied.

This patch also removed the bad design of the common engine
which handles the anti-alising option to ignore the anti-aliasing rle generation.

Just realized, it's difficult to control that condition due to scene-composition.

Change-Id: I0caab0e22b0bb88bf0c7ed25b58229c216579086

sw_engine raster: ++safety of the scaled image rasterization

Prevent out of buffer boundary access.

@Issues: https://github.com/Samsung/thorvg/pull/1119

Change-Id: I9d007b00f4193448fe2fd05194621190144ad7ce

common: introduce iterator

+++

auto picture = tvg::Picture::gen();

auto func = [](const tvg::Paint* paint, const tvg::Paint* parent, bool hasChildren) -> int
{
    if (paint->identifier() == Shape::identifier())
        //TODO: override properties.

    //return true to continue, return false to stop.
    return true;
};

picture = tvg::Iteratorv::iterate(move(picture), func);

Change-Id: I2ea956cb46b20547239686969c2e403e23fc8943

capi: missing arg description added

Change-Id: If9b6a81ac27a0f7f7b52f791e0c6205bf014d9df

sw_engine texmap: implemented fast span-edge anti-aliasing

This Anti-Aliasing mechanism is originated from Hermet Park's idea.
To understand this AA logic, you can refer this page:
www.hermet.pe.kr/122 (hermetpark@gmail.com)

@Issue: https://github.com/Samsung/thorvg/issues/161

Change-Id: Ibf9a3d14057a781e09e4ae3d72393aa522a2c71b

sw_engine raster: improve the transformed rle image rasterizer.

replace the transformed rle rgba image with the texmap raster.

this patch removes the several scattered transformed image rasterizer,
reuse the unified one texmap method instead.

yay, it's much clean and optimal.

Change-Id: Ie5c1a99b28c5242639618e1f4e41f8912c443b4c

Revert "svg_loader: the preserveAspectRatio attrib handled"

This reverts commit 2111930048843b2373907f4e9813f97a3b2d6206.

Change-Id: I608f73c16e8bb9c6caf2a3fb40254e6eca056dcf

svg_loader: the preserveAspectRatio attrib handled

The preserveAspectRatio attribute handled according
to the svg standard.

Change-Id: Ie7292ba47341994cf703b3634016c267292a22d2

bump up 0.6.7

Change-Id: Ib07f04d7c74b5f3be2ec6aaf6b76e3a88f2dbe2f
Signed-off-by: jykeon <jykeon@samsung.com>

sw_engine: fix comparison of integer expressions of different signedness

Some gcc versions and configurations was giving an error:
error: "comparison of integer expressions of different signedness: 'const SwCoord'
{aka 'const long int'} and 'uint32_t' {aka 'unsigned int'} [-Werror=sign-compare]"
This patch should fix the error

Change-Id: Ie683f35220b130b3548d0059ffe130fa4fc53fdc

sw_engine: fixed declaration of 'i' shadows a previous local

Some gcc versions and configurations was giving an error: error:
"declaration of 'i' shadows a previous local"
This patch should fix these errors

Change-Id: Ic33bc0095a7ddfd78dd530a7dd4712c000ddd2eb

bump up 0.6.6

Change-Id: I1106ae0d6d5623ca397232964f06bc99aeaadd42
Signed-off-by: jykeon <jykeon@samsung.com>

sw_engine raster: fix a regression bug.

Picture example were broken by 90fa26b7bb75cb938290170882e65da8d9fc5d9e

the correct condition must be like this change.

Change-Id: Ie3af07bca9d44872073b111874c60575fa4b38f4

sw_engine SwRaster: Initialize uninitialized transform value

Change-Id: I7aac462c6d6e481655c35a10b237970de7dbea76

bump up 0.6.5

Change-Id: I8431f13558ce30db255650ab2f52f794729e7c57
Signed-off-by: jykeon <jykeon@samsung.com>

capi: rename tvg_paint_get_bounds

Apply the changed code of 9db4a49

Change-Id: I101ec1b5ac3876a0cba76f4da97e953d0894e8f0

sw_engine SwShape: Fix coding style

sw_engine SwShape: Prevent null access

strokeOutline returned strokeExportOutline() is the address of mpool->strokeOutline[idx].
Assuming this value is null, mpoolRetStrokeOutline on line 617 will access mpool->strokeOutline[idx].
Logically, the logic of the mpool* functions does not occur in this case.

png_loaders: fix the wrong premultiplying operations.

It should not modify the alpha channel value while premultiplying
that turned out with the wrong visual result.

@Issue: https://github.com/Samsung/thorvg/issues/655

common paint: code refactoring

Grouping the composite data to add source paint necessarily.

this refactoring is a prerequisite job for the texmap anti-aliasing.

common picture: remove unused variable.

common: revise the identifier() implementation

Migrate the id property to the base class internals
so that pimpl classes could access the data easier.

This is a sort of prerequisite change for the coming texmap anti-aliasing.

bump up version 0.6.4

Change-Id: I2f1e645ab71d37430503341c4b16d32e053331f1
Signed-off-by: jykeon <jykeon@samsung.com>

capi: removed tizen only code.

These APIs have been removed in thorvg,
Nowhere used in tizen anymore.

Change-Id: I21940cdc982e226d4f8e2e6fdc1487ec3a434bbd

sw_engine image: support non-premultiplied alpha images.

Previously, translucent png images are not displayed properly
due to alpha channels premultiplication.

This patch implements that missing part to support it properly
by introducing the Surface data between canvas engine & rasterizer

@Issue: https://github.com/Samsung/thorvg/issues/655

Change-Id: I42d2b8d6854517f61cdc0eaf3ae105a426e0c61b

sw_engine raster: remove unnecessary functions.

These alpha/inverse alpha blender table is not useful so far,
we remove them since it just decrease the performance by by-pass addressing.

sw_engine SwCommon: Change spans's x,y value type

The x and y of spans cannot be negative
because they are specified as coordinates inside the buffer.
Change the type to fix warnings and potential problems
that occur in conversion between int16_t and uint32_t.

sw_engine SWRaster: Explicitly declare type cast to uint32_t

sw_engine SWRaster: Initialize uninitialized local variable

bump up version 0.6.3

Change-Id: I7ee8ad468f02ac8b69ce3baf0d7918a5b9e6879a
Signed-off-by: jykeon <jykeon@samsung.com>

sw_engine raster: fix the regression bug of the image simple scaling rendering.

Gave up the optimization this logic.

It brings too many digressed code, not good for maintenance...

Change-Id: I6adafcba58e58b0f6d389159a92b7dfd2ea68444

sw_engine texmap: remove unnecessary conditions.

remove exceptional handling for the performance.

svg_loader: a function to establish shapes boundries without a stroke introduced

To handle the objectBoundingBox units, the shape's boundaries have to be known.
According to the SVG standard, a stroke shouldn't be taken into account. Since
the bounds() api uses the shape's stroke information, a new function is introduced,
that compensates this and returns boundaries without any strokes.

svg_loader: struct used to pass 1 args instead of 4 of them

The Box struct is introduced to replace the four functions
args: vx, vy, vw, vh, so all of them can be passed at once.

sw_engine raster: fix the texmap regression bug.

Wrong inline function with C-preprocessing doesn't work at all...
Correct them with including the separate files instead.

Revert "sw_engine raster: code refactoring."

This reverts commit 860068301a7992f375a74a1f6b95e96e3bb3eb80.

found a regression bug. this macro doesn't work actually.

Change-Id: Id3c734ed8bae10451227ecf172c4e47bc06b6753

sw_engine: code refactoring

unified the two color interpolate methods.

Change-Id: I7ff67af280fa78b11d3b083aec790ac043387b1a

sw_engine texmap: code refactoring.

Unrolled the blending stages by applying macro magics.

I know this is a bit bad for code readibility
but good for the performance and the optimal LOC.

sw_engine texmap: introduced texture mapping polygon drawing.

Introduced the texture-mapping algorithm for the image-scaler.
With this approach, we can support any arbitrary forms of the transformed shapes
while appling the scale factors - x/y separately.

Also this is more efficient when the transformed image is far from the rectangular form
because our current normal image rasterizer based on the given drawing region whereas
our new approach is span-based.

Thus, we applied this mechanism only for the transformed images.

sw_engien renderer: ++safety

+exceptional handling for the drawing region.

sw_engine raster: fix a regression bug.

mistaken by 8ef31f6cd031b1369ec52e18cc0c97b6efaca6ab

sw_engine raster: fix compiler warnings.

popped up unused-functions when simd enabled.

bump up version 0.6.2

Change-Id: I3f732e458c85133e3b5d17bd2ca19b5f5cf17254
Signed-off-by: jykeon <jykeon@samsung.com>

sw_engine raster: fix a mistake by the previous hot refactoring.

sw_engine raster: code refactoring

+neat and clean code

sw_engine image: improvement sequeqnce of the transformed rle rgba image drawing.

implement composition by translucent / opaque.

yet this transformed rle rgba image drawing is on the enhancement stage.

sw_engine image: complete sequeqnce of the scaled rgba rle image drawing.

implement composition by translucent / opaque

sw_engine image: improvement sequeqnce of the direct rgba rle image drawing.

implement composition by translucent / opaque.

+ correct compositor stride reference.

sw_engine image: improvement sequeqnce of the transformed rgba image drawing.

implement composition by translucent / opaque.

yet this transformed rgba image drawing is on the enhancement stage.

sw_engine image: complete sequeqnce of the scaled rgb image drawing.

implement composition by translucent / opaque.

sw_engine image: complete sequeqnce of the direct rgb image drawing.

implement composition by translucent / opaque.

sw_engine image: refactoring before further development.

separate internal functions per context
and keep the coding convention.

Update CONTRIBUTING.md

sw_engine image: support separate logics for scaled rle rgba image

+ scaled rle rgba image
+ scaled masked rle rgba image
+ scaled translucent rle rga image

common: nothing was rendered after an empty masked node came across

The problem was observed for empty masked scenes (except the fastTrack
cases). Solved by imimediate returning from the func without passing 'false'
as its result.

examples: replaced png resource to verify alpha channel.

alpha blending looks a bit awkward.. need to fix it.

sw_engine image: apply rgba blending.

rgba requires the default blending.

infra stylebot: Delete existing comments

In order to avoid duplication of the already created comment,
already created results will be deleted.

sw_engine image: code refactoring

Clarify image sequences by renaming & adding TODO comments.

sw_engine image: introduced scaled image raster logics.

These raster functions are accelerated only for the scaled images.
(no rotation, skrewed)

examples: remove boundary samples.

Removed these tests from the example since it's not so proper showcases
for user scenaro.