tree 45ff87c065ef77f9126f08bf53d097ce0f371efc
parent b7f82779272a6a8c7e92fabb544233b0ec49c968
author Tom Churchman <thomas@churchman.nl> 1745487916 +0200
committer GitHub <noreply@github.com> 1745487916 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJoCgg7CRC1aQ7uu5UhlAAA03MQAH9vDVM2/aYBWCp17jmt+Vgs
 OdQMVBsr1AKmQvFHkj91/PA/7o3as8O+SQRNb86uzaSZ1Qix7OXxaOlBtb5sBV7e
 Xl4zdUQ5+5pkzO4ZqY8LEhFRrd3didFre1phL49hwdEeCaGXoaZeLNswjNyJS+Lr
 j+zGH6kb86ofUtQItL3yxP5QI8jHQOZQGdfKdhzLFIxolLWSfnUke4Y/58g+Py7F
 4KqzK3v0jKRQTtlkuIf8rBH5jebpPXJpUJBt9nV/3Zg2L0AJztzaZPEpn1yCSgBP
 6lPBt78FmQOaT50ACP84uk0ocsL8jE7N63VFL2ISJfkp3sCsqLOybVLT4nP+SmyQ
 zN5dT8C2VyFLxfGPkzInA6ehtutMeufbCW/0jQFbmN9Cc93zr0CNKJBzsuolC91D
 aqIod1mYiacfTPpLsGjVrVZKuL6yZYgHmftCD/4RYYQWYlq0RCPtqV2gtaZwrcpp
 shgNdb5/QvJidXtom9Xk2QYHcA+yT8SXhQDVhhzOSOY57gAYjH635EkIMibogENS
 ryOaKqlfJGiylaBfdTMgPmIPiJCcw5wtVWWceQb8tWeN7MDblzucvfvhekINdViD
 PWRbKn1IO34+XQbhTpPmmubbpiKFj5BR1YcyY7S5sngbQzcFg+zDsWMmyj/f+Xah
 vrMZ3amEDSLFmDrHdpa1
 =j+qH
 -----END PGP SIGNATURE-----
 

Reduce clipping memory usage and allocation (#920)

The scratch strip buffer may be quite large when the render context has
been used for rendering complex paths. If the clip geometry is simple,
storing that buffer in the clip state has two downsides:
- the buffer will have unused capacity, inflating the memory footprint
of the renderer,
- as the buffer is moved into the clip state, a new buffer has to be
allocated, and for subsequent rendering of complex paths will have to
grow large again.

This PR proposes keeping the original allocation for the scratch strip
buffer and memcpy'ing the clip's strips to a new allocation of the right
size. (As a side-effect the size of the `Clip` struct is reduced from 40
to 32 bytes.)

There are alternatives: the first downside could be alleviated by a
`shrink_to_fit`, and allocator thrashing of the second downside could be
reduced by allocating the new buffer with the same capacity as the old
buffer. It's probably better to just keep the original allocation for
the scratch buffer, though.