Project

General

Profile

Actions

Feature #4372

closed

Replace osmo_sccp_timer_val with osmo_tdef

Added by pespin about 4 years ago. Updated 6 months ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
Start date:
01/21/2020
Due date:
% Done:

100%

Spec Reference:

Description

libosmo-sccp internally uses its own set of structs called osmo_sccp_timer_val, which basically duplicates what osmo_tdef already does in libosmocore.

Since that struct is only used internally (and exported only in src/sccp_internal.h), we can easily drop it and replace it with libosmocore's osmo_tdef.

src/sccp_internal.h
25:struct osmo_sccp_timer_val {
30:extern const struct osmo_sccp_timer_val osmo_sccp_timer_defaults[];
57:     struct osmo_sccp_timer_val timers[OSMO_SCCP_TIMERS_COUNT];
127:const struct osmo_sccp_timer_val *osmo_sccp_timer_get(const struct osmo_sccp_instance *inst,

src/sccp_vty.c
151:    struct osmo_sccp_timer_val set_val = { .s = atoi(argv[1]) };
168:static const char *osmo_sccp_timer_val_name(const struct osmo_sccp_timer_val *val)
190:            const struct osmo_sccp_timer_val *def;
202:                                 osmo_sccp_timer_val_name(def));
219:            const struct osmo_sccp_timer_val *val = osmo_sccp_timer_get(inst, i, default_if_unset);
223:                    osmo_sccp_timer_val_name(val), VTY_NEWLINE);

src/sccp_scoc.c
278:const struct osmo_sccp_timer_val osmo_sccp_timer_defaults[] = {
297:const struct osmo_sccp_timer_val *osmo_sccp_timer_get(const struct osmo_sccp_instance *inst,
301:    const struct osmo_sccp_timer_val *val;
302:    const struct osmo_sccp_timer_val *def;
331:    const struct osmo_sccp_timer_val *val = osmo_sccp_timer_get(conn->inst, timer_name, true);

Actions #1

Updated by pespin 9 months ago

  • Status changed from New to Feedback
  • Assignee set to pespin
  • % Done changed from 0 to 90

Can be closed once this patch is merged:
remote: https://gerrit.osmocom.org/c/libosmo-sccp/+/33644 sccp: Use tdef to implement osmo_sccp_timers

Actions #2

Updated by pespin 6 months ago

  • Status changed from Feedback to Resolved
  • % Done changed from 90 to 100

Merged a long time ago, closing.

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)