root
/
aseprite
mirror of https://github.com/aseprite/aseprite.git
1
0
Fork 0
Code Issues Actions 6 Packages Projects Releases Wiki Activity

Remove unused parameter

This commit is contained in:
Martín Capello 2025-08-29 15:58:24 -03:00
parent 9fd1214947
commit f6b23dc4e8
4 changed files with 8 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/app/commands/cmd_duplicate_layer.cpp
View File

@ -51,7 +51,7 @@ void DuplicateLayerCommand::onExecute(Context* context)
Tx tx(writer, "Layer Duplication");
LayerImage* sourceLayer = static_cast<LayerImage*>(writer.layer());
DocApi api = document->getApi(tx);
api.duplicateLayerAfter(sourceLayer, sourceLayer->parent(), sourceLayer, " Copy");
api.duplicateLayerAfter(sourceLayer, sourceLayer->parent(), sourceLayer);
tx.commit();
}

12
src/app/doc_api.cpp
View File

@ -726,10 +726,7 @@ Layer* DocApi::copyLayerWithSprite(doc::Layer* layer, doc::Sprite* sprite)
return clone.release();
}
Layer* DocApi::duplicateLayerAfter(Layer* sourceLayer,
LayerGroup* parent,
Layer* afterLayer,
const std::string& nameSuffix)
Layer* DocApi::duplicateLayerAfter(Layer* sourceLayer, LayerGroup* parent, Layer* afterLayer)
{
ASSERT(parent);
Layer* newLayerPtr = copyLayerWithSprite(sourceLayer, parent->sprite());
@ -741,14 +738,11 @@ Layer* DocApi::duplicateLayerAfter(Layer* sourceLayer,
return newLayerPtr;
}
Layer* DocApi::duplicateLayerBefore(Layer* sourceLayer,
LayerGroup* parent,
Layer* beforeLayer,
const std::string& nameSuffix)
Layer* DocApi::duplicateLayerBefore(Layer* sourceLayer, LayerGroup* parent, Layer* beforeLayer)
{
ASSERT(parent);
Layer* afterThis = (beforeLayer ? beforeLayer->getPreviousBrowsable() : nullptr);
Layer* newLayer = duplicateLayerAfter(sourceLayer, parent, afterThis, nameSuffix);
Layer* newLayer = duplicateLayerAfter(sourceLayer, parent, afterThis);
if (newLayer)
restackLayerBefore(newLayer, parent, beforeLayer);
return newLayer;

10
src/app/doc_api.h
View File

@ -119,14 +119,8 @@ public:
void removeLayer(Layer* layer);
void restackLayerAfter(Layer* layer, LayerGroup* parent, Layer* afterThis);
void restackLayerBefore(Layer* layer, LayerGroup* parent, Layer* beforeThis);
Layer* duplicateLayerAfter(Layer* sourceLayer,
LayerGroup* parent,
Layer* afterLayer,
const std::string& nameSuffix = std::string());
Layer* duplicateLayerBefore(Layer* sourceLayer,
LayerGroup* parent,
Layer* beforeLayer,
const std::string& nameSuffix = std::string());
Layer* duplicateLayerAfter(Layer* sourceLayer, LayerGroup* parent, Layer* afterLayer);
Layer* duplicateLayerBefore(Layer* sourceLayer, LayerGroup* parent, Layer* beforeLayer);
// Images API
void replaceImage(Sprite* sprite, const ImageRef& oldImage, const ImageRef& newImage);

5
src/app/doc_range_ops.cpp
View File

@ -405,8 +405,7 @@ static DocRange drop_range_op(Doc* doc,
if (place == kDocRangeBefore) {
Layer* beforeThis = (!dstLayers.empty() ? dstLayers.front() : nullptr);
for (Layer* srcLayer : srcLayers) {
Layer* copiedLayer =
api.duplicateLayerBefore(srcLayer, parent, beforeThis, " Copy");
Layer* copiedLayer = api.duplicateLayerBefore(srcLayer, parent, beforeThis);
resultRange.startRange(copiedLayer, -1, DocRange::kLayers);
resultRange.endRange(copiedLayer, -1);
@ -417,7 +416,7 @@ static DocRange drop_range_op(Doc* doc,
Layer* afterThis = (!dstLayers.empty() ? dstLayers.back() : nullptr);
for (Layer* srcLayer : srcLayers) {
Layer* copiedLayer = api.duplicateLayerAfter(srcLayer, parent, afterThis, " Copy");
Layer* copiedLayer = api.duplicateLayerAfter(srcLayer, parent, afterThis);
resultRange.startRange(copiedLayer, -1, DocRange::kLayers);
resultRange.endRange(copiedLayer, -1);