Skip to content

[Impeller] Clamp then min should just be clamp #179105

New issue
New issue
Closed
#179110
Closed
[Impeller] Clamp then min should just be clamp#179105
#179110
Assignees
piedcipher
Labels
P1High-priority issues at the top of the work liste: impellerImpeller rendering backend issues and features requestsengineflutter/engine related. See also e: labels.refactorImproving readability/efficiency without behavioral changesteam-engineOwned by Engine teamtriaged-engineTriaged by Engine team
@flar

Description

@flar
flar
opened on Nov 26, 2025

The following 2 lines can be combined into one operation.

flutter/engine/src/flutter/impeller/entity/entity.cc

Line 93 in 9d96df2

Scalar result = std::clamp(clip_depth * kDepthEpsilon, 0.0f, 1.0f);

Metadata

Metadata

Assignees

Labels

P1High-priority issues at the top of the work liste: impellerImpeller rendering backend issues and features requestsengineflutter/engine related. See also e: labels.refactorImproving readability/efficiency without behavioral changesteam-engineOwned by Engine teamtriaged-engineTriaged by Engine team

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions