Attention is currently required from: laforge, pespin, fixeria.
1 comment:
Patchset:
I find it difficult to review this patch because afaict there's several steps merged int one: […]
I don't know what to split here because the patch does only one thing, and that is replacing the dozen args with the cfg struct for the devsice classes and the dev base class,radiodevice, which makes it really easy to track the usage of the config args because they don't get mangled and copied five times or pushed into vectors just to be passed down two layers. Of course open() does not require any args because the full config is already known at construction time. the dev args string is a string and ensuring it always is a string and not a dangling pointer by properly initializing it as a string is required to be able to just use it.
To view, visit change 32762. To unsubscribe, or for help writing mail filters, visit settings.