-
Notifications
You must be signed in to change notification settings - Fork 11
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
Refactor exceptions #23
Comments
This is a plain copy from Text::CSV_XS. If there is an easier way, I don't mind simplification. At least I think it should be easy to keep the two in sync as much as possible. |
That might become eventually impossible... But if we can automatically
convert it, I guess we could do it again...
El mar., 31 dic. 2019 a las 16:53, H.Merijn Brand (<[email protected]>)
escribió:
… This is a plain copy from Text::CSV_XS. If there is an easier way, I don't
mind simplification. At least I think it should be easy to keep the two in
sync as much as possible.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#23?email_source=notifications&email_token=AAAAD5B7FQYVVOZPYIDXX6DQ3NTGBA5CNFSM4KBUYKZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH4LYNI#issuecomment-569949237>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAAD5D533ET3GOXOOBZ4J3Q3NTGBANCNFSM4KBUYKZA>
.
--
JJ
|
Also, usually exceptions use the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Right now it looks like...
We should probably make some attempt to automatize it, maybe generate
X::Text::CSV::Ini
names and then generate the message automatically. Refactoring the code, which looks like self!fail(%error<1000>) shouldn't be too difficult either.The text was updated successfully, but these errors were encountered: