[sr-dev] git:master:1f7a887a: ims_isc: fix compiler warnings

Victor Seva linuxmaniac at torreviejawireless.org
Tue Jan 17 10:22:11 CET 2017


Module: kamailio
Branch: master
Commit: 1f7a887ab461b643aa2254f5f6748628234f151e
URL: https://github.com/kamailio/kamailio/commit/1f7a887ab461b643aa2254f5f6748628234f151e

Author: Victor Seva <linuxmaniac at torreviejawireless.org>
Committer: Victor Seva <linuxmaniac at torreviejawireless.org>
Date: 2017-01-17T10:22:05+01:00

ims_isc: fix compiler warnings

* inline functions were not defined at mark.h
  so the compiler was ignoring the inline directive

> In file included from mod.c:49:0:
> mark.h:78:12: warning: inline function 'isc_mark_write_route' declared but never defined
>  inline int isc_mark_write_route(struct sip_msg *msg,str *as,str *iscmark);
>             ^~~~~~~~~~~~~~~~~~~~
> mark.h:76:12: warning: inline function 'isc_mark_drop_route' declared but never defined
>  inline int isc_mark_drop_route(struct sip_msg *msg);
>             ^~~~~~~~~~~~~~~~~~~
> CC (gcc) [M ims_isc.so]		third_party_reg.o
> In file included from third_party_reg.h:52:0,
>                  from third_party_reg.c:46:
> mark.h:78:12: warning: inline function 'isc_mark_write_route' declared but never defined
>  inline int isc_mark_write_route(struct sip_msg *msg,str *as,str *iscmark);
>             ^~~~~~~~~~~~~~~~~~~~
> mark.h:76:12: warning: inline function 'isc_mark_drop_route' declared but never defined
>  inline int isc_mark_drop_route(struct sip_msg *msg);
>             ^~~~~~~~~~~~~~~~~~~
> CC (gcc) [M ims_isc.so]		isc.o
> In file included from isc.h:54:0,
>                  from isc.c:47:
> mark.h:78:12: warning: inline function 'isc_mark_write_route' declared but never defined
>  inline int isc_mark_write_route(struct sip_msg *msg,str *as,str *iscmark);
>             ^~~~~~~~~~~~~~~~~~~~
> mark.h:76:12: warning: inline function 'isc_mark_drop_route' declared but never defined
>  inline int isc_mark_drop_route(struct sip_msg *msg);
>             ^~~~~~~~~~~~~~~~~~~

---

Modified: src/modules/ims_isc/mark.c
Modified: src/modules/ims_isc/mark.h

---

Diff:  https://github.com/kamailio/kamailio/commit/1f7a887ab461b643aa2254f5f6748628234f151e.diff
Patch: https://github.com/kamailio/kamailio/commit/1f7a887ab461b643aa2254f5f6748628234f151e.patch

---

diff --git a/src/modules/ims_isc/mark.c b/src/modules/ims_isc/mark.c
index 5baa83b..8d22fff 100644
--- a/src/modules/ims_isc/mark.c
+++ b/src/modules/ims_isc/mark.c
@@ -197,7 +197,7 @@ int base16_to_bin(char *from, int len, char *to) {
  *	@param msg - SIP mesage to mark
  *	@returns 1 on success
  */
-inline int isc_mark_drop_route(struct sip_msg *msg) {
+int isc_mark_drop_route(struct sip_msg *msg) {
 	struct lump* lmp, *tmp;
 
 	parse_headers(msg, HDR_EOH_F, 0);
@@ -273,7 +273,7 @@ int isc_mark_set(struct sip_msg *msg, isc_match *match, isc_mark *mark) {
  *	@param iscmark - the mark to write
  *	@returns 1 on success, else 0
  */
-inline int isc_mark_write_route(struct sip_msg *msg, str *as, str *iscmark) {
+int isc_mark_write_route(struct sip_msg *msg, str *as, str *iscmark) {
 	struct hdr_field *first;
 	struct lump* anchor;
 	str route;
diff --git a/src/modules/ims_isc/mark.h b/src/modules/ims_isc/mark.h
index 1031f6e..4af94a1 100644
--- a/src/modules/ims_isc/mark.h
+++ b/src/modules/ims_isc/mark.h
@@ -73,9 +73,9 @@ typedef struct _isc_mark{
 int isc_mark_get_from_msg(struct sip_msg *msg,isc_mark *mark);
 void isc_mark_get(str x,isc_mark *mark);
 int base16_to_bin(char *from,int len, char *to);
-inline int isc_mark_drop_route(struct sip_msg *msg);
+int isc_mark_drop_route(struct sip_msg *msg);
 int isc_mark_set(struct sip_msg *msg, isc_match *match, isc_mark *mark);
-inline int isc_mark_write_route(struct sip_msg *msg,str *as,str *iscmark);
+int isc_mark_write_route(struct sip_msg *msg,str *as,str *iscmark);
 int isc_mark_write_psu(struct sip_msg *msg, isc_mark *mark);
 int bin_to_base16(char *from,int len, char *to);
 




More information about the sr-dev mailing list