You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Jiri Daněk (Jira)" <ji...@apache.org> on 2021/11/14 11:32:00 UTC

[jira] [Updated] (DISPATCH-2285) Hide the API defined in address_lookup_utils.c by moving that file into router_core/modules/address_lookup_server directory

     [ https://issues.apache.org/jira/browse/DISPATCH-2285?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jiri Daněk updated DISPATCH-2285:
---------------------------------
    Priority: Minor  (was: Major)

> Hide the API defined in address_lookup_utils.c by moving that file into router_core/modules/address_lookup_server directory
> ---------------------------------------------------------------------------------------------------------------------------
>
>                 Key: DISPATCH-2285
>                 URL: https://issues.apache.org/jira/browse/DISPATCH-2285
>             Project: Qpid Dispatch
>          Issue Type: Improvement
>    Affects Versions: 1.17.0
>            Reporter: Jiri Daněk
>            Priority: Minor
>
> Comment on https://github.com/apache/qpid-dispatch/pull/1211#discussion_r631820969
> The address lookup stuff is only used by router-core module. It's not used by I/O threads, for example. This means that we should probably "hide" the API defined in address_lookup_utils.c by moving that file into router_core/modules/address_lookup_server directory.
> Then I would move the address_lookup_server.h file out of the general include/qpid/dispatch directory and put it in src/router_core/ directory.
> These changes would result in making the address lookup stuff private to the router_core module, which seems better to me since the address lookup functionality is only to be used by the router_core.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org