On Tue, 16 Jan 2018 15:18:28 +0000 Daniel Stone <dan...@fooishbar.org> wrote:
> Hi Pekka, > > On 15 January 2018 at 15:05, Pekka Paalanen <ppaala...@gmail.com> wrote: > > this patch is both trivial and lacking any rationale on why it was > > written, with obviously no impact on its own. Would be nice for > > posterity to have some explanation on the why. I assume it was > > something subtle or annoying. > > > > If not squashing into the first patch needing this. > > As you saw later, this is used when have special drm_planes in the > pre-universal path, where we need divergent destroy paths. This is new > as of v13 so we can do that; I put it at the start since it was so > trivial. Happy to include the expanded rationale here or fold it into > the special-planes (cursor-plane) patch; whichever is better. Hi, I don't really mind which way, whichever seems better to you. Either way, IMHO it would probably warrant a small paragraph on the why in a commit message. Thanks, pq
pgpdKdxo0Lg9q.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel
