The patch seems appropriate.
However, I think there should be more patches, IIUC. There are comments in this
file referring to being automatically generated. While this may have been the
original genesis of the file, since it's in input/regression it's not
automatically generated from out/*. I think these comments should go away.
But maybe I'm mistaken.
Thanks,
Carl
On 2011/04/16 14:33:10, Carl wrote:
> I think these comments should go away.
Agreed. I looked over the new midi regression tests again; these comments are
leftovers from copy/paste, and could only confuse us, so they are gone.
This patch affects only a regression test, so I'll push now and close this
rietveld issue.
Issue 4433044: MIDI: partcombine regtest needs skips (issue 1609)
(Closed)
Created 13 years ago by Keith
Modified 13 years ago
Reviewers: carl.d.sorensen_gmail.com
Base URL:
Comments: 0