The provider is checked by fetching the server config
Details
Details
Diff Detail
Diff Detail
- Repository
- R235 Attica
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 331 Build 331: arc lint + arc unit
Comment Actions
Autotests are good, but so is documentation - in principle this is good, but new public functions without documentation isn't really acceptable :)
Apart from that, though, looks good! :) (was just about to add a comment about your invalid data string being less than descriptive... but hey, it's invalid data, why not ;) )
| src/provider.h | ||
|---|---|---|
| 275 ↗ | (On Diff #36668) | When adding new functions to anywhere, putting in documentation really needs to happen as well :) You've got it in other places already, though, so i'm sure that's just a minor bit of forgetfulness, these things happen :) |