From: Jiří Č. <ji...@ci...> - 2017-07-18 10:20:09
|
> Option 2 - ctor with target version sounds the best. You could name pre3 > version enum "Version1" and the new format "Version2" so it's not linked > to > FB version. That sounds good. Maybe confusing for new developers, but what you can do, right? > A nice to have would be to have default enum value be "DetectVersion" > which > would do the roundtrip server check. I'm bit worried about having it as default. Convenient, yes, but hidden perf problem. Though not sure it matters on FbTrace much. -- Mgr. Jiří Činčura https://www.tabsoverspaces.com/ |