IDL schema: server_parameter cpp_class should always be an object, not sometimes string

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Server Programmability
    • None
    • 3
    • TBD
    • None
    • None
    • None
    • None
    • None
    • None

      PR context

      -        cpp_class: SSLModeServerParameter
      +        cpp_class:
      +            name: SSLModeServerParameter
      +            override_warn_if_deprecated: true
      

      It was confusing for me to make this change because the schema has two possibilities for cpp_class. Either it's a simple string OR it's a yaml object with properties. To add a property, I have to change the kind of element held by cpp_class, rewriting the simple string to a name: key before adding the new property. I think that's too hard.

      I think it would be logically more straightforward if we removed this flexibility and forced all cpp_class to follow a simple uniform schema, where it's an object and that's that.

            Assignee:
            Unassigned
            Reporter:
            Billy Donahue
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: