Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Config_error does not support adding additional context #480

Open
jbigot opened this issue Oct 14, 2024 · 1 comment
Open

Config_error does not support adding additional context #480

jbigot opened this issue Oct 14, 2024 · 1 comment
Labels
level:: newcomer_job An issue whose resolution is well suited for newcomers module:: pdi core request:: feature

Comments

@jbigot
Copy link
Member

jbigot commented Oct 14, 2024

Very often, the context for a configuration error can only be provided higher in the call stack. In this case, one would like to add information to the Config_error exception.
For example:

try {
} catch (Config_error& err) {
    throw Config_error{/**/, "Some more info"};
}

Unfortunately, the current implementation doesn't make it possible to extract the line & column of the error from the exception so this is not possible.

@benoitmartin88
Copy link
Member

update deisa plugin when this issue is fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
level:: newcomer_job An issue whose resolution is well suited for newcomers module:: pdi core request:: feature
Projects
None yet
Development

No branches or pull requests

2 participants