Garrett D'Amore wrote: > This looks like a potentially handy feature, and the architecture seems > fairly straight-forward. However, I'm not sure it qualifies for > self-review. (Although the bar for self review seems to be moving > somewhat...) If only because it introduces new interfaces and behavior > for consumers. > > I'd suggest that this maybe should be a fast-track instead of self > review, unless there is some particular rush to get this case through? > Please consider promoting it to a fast track. In any case, I'll happily > give it a +1. > > Btw, you failed to specify a release binding. Is this patch or minor > (or something else)? (It looks like this could be Patch binding fairly > easily...) > > - Garrett
I'm not opposed to making it a fast track. However, I'd like to ask you to reconsider. It's a trivial option being implemented purely for Linux compatibility. Software we care about (Xorg) uses it, which is the sole reason for introducing it. It seems well below the line to me... I did specify the release binding, in the middle: >> ----- >> >> Release Binding: Patch/Micro >> >> New ld options: >> -z wrap Committed >> -wrap Committed >> --wrap Committed >> >> ----- Thanks... - Ali