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

Better PropertyNames #2746

Open
bernardnormier opened this issue Sep 18, 2024 · 0 comments
Open

Better PropertyNames #2746

bernardnormier opened this issue Sep 18, 2024 · 0 comments
Labels
Milestone

Comments

@bernardnormier
Copy link
Member

We use config/PropertyNames.xml and its associated Python script to define all Ice-reserved properties, and to generate supporting code in C++, C#, Java, and JavaScript.

For example:

<section name="Ice">
        <property name="AcceptClassCycles" default="0" />
        <property name="Admin" class="objectadapter" />
        <property name="Admin.DelayCreation" default="0" />

is the beginning of the Ice.* reserved properties.

I propose we make the following improvements:

  • for each property, specify the language(s) to which is applies as cpp, cs, java, js or all
    the current behavior is "all", which does not make sense for many properties, especially in js

  • generate helper code to parse ObjectAdapter properties
    we currently duplicate this logic / list of properties in the various ObjectAdapter implementations, which is error-prone.

@bernardnormier bernardnormier added this to the 3.8.0 milestone Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant