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

Add support for Free Format parameter declaration #43

Open
patrickhrastnik opened this issue Aug 7, 2019 · 3 comments
Open

Add support for Free Format parameter declaration #43

patrickhrastnik opened this issue Aug 7, 2019 · 3 comments

Comments

@patrickhrastnik
Copy link

Hi!

In IBM/nodejs-itoolkit#75 I suggested to add support for free format parameter syntax ("char(10)" instead of "10A").
@ThePrez suggested to open an issue here.

This may be something related to #24 - but as far as I've seen, #24 is more about changing this packages' source to total free syntax.

@jimoibm
Copy link
Contributor

jimoibm commented Dec 8, 2020

Agreed with what @kadler said in itoolkit#75. Current xmlservice is more than a low level component that is fledged with lots of features. Not sure how many of them are still used.

Anyway supporting new format 'type' can be done in

P bigValType B export

This procedure might have a if leg ahead to serve the new format to have below vars set up correctly ...

xmlservice/src/plugile.rpgle

Lines 2412 to 2414 in e9cfced

D myAttr 1A
D myDigits 10i 0
D myFrac 10i 0

@kadler
Copy link
Member

kadler commented Dec 8, 2020

I don't think it makes much sense to invest in the format of the underlying transport format. How many users use XMLSERVICE directly vs using a toolkit which hides all this from the user?

@jimoibm
Copy link
Contributor

jimoibm commented Dec 9, 2020

Agree with @kadler again :-)
FWIW from customer issues, majorities are still in fixed format or mixed format. I will hold this and consider free format in future if this is really necessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants