@linuxmaniac requested changes on this pull request.
On src/modules/misc_radius/radius.h:
On ``radcli/radcli.h``:
```
#define RADCLI_VENDOR_MASK 0xffffffff
#define VENDOR_BIT_SIZE 32
#define RADCLI_VENDOR_ATTR_SET(attr, vendor)
((attr)|((uint64_t)((vendor)&RADCLI_VENDOR_MASK)) << VENDOR_BIT_SIZE)
#define VENDOR(x) (((x) >> VENDOR_BIT_SIZE) & 0xffffffff)
#define ATTRID(x) ((x) & 0xffffffff)
```
And on the old ``radiusclient-ng.h``:
```
#define VENDOR(x) (((x) >> 16) & 0xffff)
#define ATTRID(x) ((x) & 0xffff)
```
So, 55-60 lines are not necessary, since the definitions are only missing on
``freeradius-client.h``
--
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/4762#pullrequestreview-4407740030
You are receiving this because you are subscribed to this thread.
Message ID: <kamailio/kamailio/pull/4762/review/[email protected]>
_______________________________________________
Kamailio - Development Mailing List -- [email protected]
To unsubscribe send an email to [email protected]
Important: keep the mailing list in the recipients, do not reply only to the
sender!
