Code review comment for lp:~vkolesnikov/pbxt/pbxt-type-enum

Revision history for this message
Vladimir Kolesnikov (vkolesnikov) wrote :

Tests fixed. However once I was able to see the old style behavior with this test - an error on table creation. I was not able to reproduce it anymore. I propose this for merging because I think this is the correct behavior. IMO it would be illogical to create "exhausting" enum that involves all characters from latin1 just to check that it really fails because of back slashes mode.

« Back to merge proposal