On Fri, 2017-12-29 at 15:39 +0100, Martin Liška wrote: > On 10/06/2017 06:20 PM, David Malcolm wrote: > > Martin: I noticed that your switch expansion patch added a > > vec::reverse () > > method. Here's a proposed selftest for it, mostly to verify > > that it handles even vs odd lengths (which it does). > > > > Only lightly tested; hope this is useful. > > Dave > > Hi. > > I like it! Do you want to commit it before there will be > a consumer of the function? > > Martin
Looks like vec::reverse isn't in trunk yet. I don't have a strong opinion on whether vec::reverse should be added without a "real" consumer, or added when the consumer is added, but feel free to add my selftest patch into whichever patch adds vec::reverse.
