Commit c55e0637 authored by Maciej Suminski's avatar Maciej Suminski

Removed double freeing of some of Cairo paths.

parent 1b1216e5
...@@ -686,8 +686,7 @@ void CAIRO_GAL::DeleteGroup( int aGroupNumber ) ...@@ -686,8 +686,7 @@ void CAIRO_GAL::DeleteGroup( int aGroupNumber )
{ {
if( it->command == CMD_FILL_PATH || it->command == CMD_STROKE_PATH ) if( it->command == CMD_FILL_PATH || it->command == CMD_STROKE_PATH )
{ {
if( it->cairoPath->status == CAIRO_STATUS_SUCCESS ) cairo_path_destroy( it->cairoPath );
cairo_path_destroy( it->cairoPath );
} }
} }
...@@ -839,13 +838,12 @@ void CAIRO_GAL::storePath() ...@@ -839,13 +838,12 @@ void CAIRO_GAL::storePath()
// then check, if the path needs to be stroked/filled and // then check, if the path needs to be stroked/filled and
// add this command to the group list; // add this command to the group list;
cairo_path_t* path = cairo_copy_path( cairoImage );
// pathList.push_back( path ); // FIXME: it's not used anywhere else? // pathList.push_back( path ); // FIXME: it's not used anywhere else?
if( isStrokeEnabled ) if( isStrokeEnabled )
{ {
GroupElement groupElement; GroupElement groupElement;
groupElement.cairoPath = path; groupElement.cairoPath = cairo_copy_path( cairoImage );
groupElement.command = CMD_STROKE_PATH; groupElement.command = CMD_STROKE_PATH;
groups.back().push_back( groupElement ); groups.back().push_back( groupElement );
} }
...@@ -853,7 +851,7 @@ void CAIRO_GAL::storePath() ...@@ -853,7 +851,7 @@ void CAIRO_GAL::storePath()
if( isFillEnabled ) if( isFillEnabled )
{ {
GroupElement groupElement; GroupElement groupElement;
groupElement.cairoPath = path; groupElement.cairoPath = cairo_copy_path( cairoImage );
groupElement.command = CMD_FILL_PATH; groupElement.command = CMD_FILL_PATH;
groups.back().push_back( groupElement ); groups.back().push_back( groupElement );
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment