-
Notifications
You must be signed in to change notification settings - Fork 37
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
iPgm parameters: Make declarations for parameter types easier to use #75
Comments
I agree with this proposal completely. I am guessing if XMLSERVICE had the change, the the itoolkit modules for python, Node, PHP, Ruby could all benefit "for free" with no changes (but I haven't looked at the code). I would suggest an issue on XMLSERVICE regardless, at http://github.com/IBM/XMLSERVICE |
As suggested, I opened an issue IBM/xmlservice#43. |
I don't agree with this proposal. If anything, we should be reducing our coupling with RPG, not strengthening it. XMLSERVICE is a low-level service and its implementation should be shielded by the toolkits, though the Node toolkit went rather away from this for whatever reason. I'd rather see wrapper classes for the different types and have them translate it in to what XMLSERVICE wants. |
Seems like a better options |
This should be rather easy to implement. This approach would still be compatible with existing implementations. |
Seems good to me. I'd probably forgo the class and just have a module that exports the functions. Also FYI, going forward the intent is to drop the "i" prefix from things. |
Suggested fix for IBM#75 Signed-off-by: HRASTNIK Patrick Ing <patrick.hrastnik@holter.at>
👋 Hi! This issue has been marked stale due to inactivity. If no further activity occurs, it will automatically be closed. |
Hi!
This is rather a nice-to-have than an actual issue, but it seems to be worth to mention it.
In our company, we currently modernize our ILE RPG software. Hence we go for free format. As far as I‘ve seen, we need to declare parameter types for iPgm with the classic format („10A“ instead of „char(10)“).
May it be possible to support free format syntax in iPgm functions as well?
If I got it correctly, this may also be something that needs to be changed in the XMLSERVICE layer (?)
This is just syntactic sugar. But in long term, it may be better maintainable to have the same syntax in both definitions.
The text was updated successfully, but these errors were encountered: