Support validation options in `lyd_validate_op()`

[mbohal]

Would it be possible to support passing LYD_VALIDATE_* options to lyd_validate_op function?

lyd_validate_all does support it.

Specifically I would need the LYD_VALIDATE_MULTI_ERROR and LYD_VALIDATE_PRESENT.

We need it to determine if a given value is valid for a given leaf in the context of a given data tree. The problem is that we need this to work even if other nodes in the data tree are invalid (unsatisfied must condition, missing required value).

[michalvasko]

I can understand the need for LYD_VALIDATE_MULTI_ERROR but am not sure what you are trying to achieve with LYD_VALIDATE_PRESENT, elaborate please.

[mbohal]

what you are trying to achieve with LYD_VALIDATE_PRESENT

We do not need it, but it allows us to skip validation of nodes that are not present. I assumed the validation would be faster as there would be less nodes to validate.

[michalvasko]

I think you misunderstand what the flag does, please look into the docs.

[mbohal]

You are right, we do not need LYD_VALIDATE_PRESENT.

We pass this flag for similar use case to lyd_validate_all and I did not realize the difference.

Supporting LYD_VALIDATE_MULTI_ERROR would be sufficient.

[michalvasko]

Okay, added.