Attention is currently required from: fixeria.

View Change

2 comments:

• File src/host/layer23/src/common/main.c:

  • Patch Set #1, Line 271: handle_options

    I may be missing something, but I believe option parsing, config parsing, and setting up the VTY sho […]

    It doesn't take VTY config anyway. It's chicken an egg problem we have in all apps.
    The idea anyway is that l23_app_info() mostly initializes memory, but doesn't "open" stuff which may be prone to errors, etc. that will be done in new l23_app_start() API, which is called after cmdline and VTY parsing.

  • Patch Set #1, Line 331: talloc_free(config_file

    This looks wrong to me. […]

    Ack

To view, visit change 30950. To unsubscribe, or for help writing mail filters, visit settings.