Closed Bug 1246106 Opened 4 years ago Closed 4 years ago
[Static Analysis][Big parameter passed by value] In Function Compute
The Static Analysis tool Coverity added that parameter aStyleBorder is passed as value. This is performance inefficient since it's around 312 bytes depending on platform data alignment. We can pass this as a pointer.
Comment on attachment 8716229 [details] [diff] [review] pass aStyleBorder as pointer to ComputeBorderOverflow I think "const nsStyleBorder& aStyleBorder" is preferred. r=mats with that change.
Attachment #8716229 - Flags: review?(roc) → review+
You need to log in before you can comment on or make changes to this bug.