On Mon, Nov 08, 2021 at 09:33:25PM +0530, Sameer Pujar wrote:
With separate callbacks, the string checks can be removed. However for most of the controls, the common part is minimal. So there would be multiple independent small functions depending on the number of controls and the local variables are duplicated that many times. Would there be any concern on the space these local variables take? One pair of callbacks for a control may look like this.
...
Looks like having separate callbacks make it look more cleaner. If this appears fine, I can send next revision.
Looks fine. It'll result in more code but hopefully they should be smaller, especially if you're using substrings rather than the full control name to identify the control and we need to store all them separately to the copy used to identify the control to userspace (I didn't go check).