https://issues.apache.org/bugzilla/show_bug.cgi?id=55873
Nick Burch <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |NEEDINFO --- Comment #1 from Nick Burch <[email protected]> --- Generally looks good, but a couple of minor points: * Unit tests should normally use assertEquals rather than naked asserts, so that the test runner can better capture problems * I can't see anything that registers the function, is something needed? -- You are receiving this mail because: You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
