[sr-dev] Dialog module issue on OS/X

[Daniel-Constantin Mierla]

On 30/07/14 15:28, Olle E. Johansson wrote:
> On 30 Jul 2014, at 15:20, Daniel-Constantin Mierla <miconda at gmail.com> wrote:
>
>> Working a bit on dialog and looking at the archive to see what was reported about it, I noticed this message sent a while back.
>>
>> Is the issue still there or a fix was done meanwhile? I haven't seen any follow up to the email...
>>
> I did not dare to commit it, since I have no idea of the effects. I could not compile without this change.
So it is still not working for you on os x? For me it compiles and links 
fine.

Anyhow, removing the inline in this prototype and function definition 
will not change any performance -- it can be done if that solve issues 
with some platforms.

Daniel

>
> /O
>
>> Cheers,
>> Daniel
>>
>> On 04/06/14 18:50, Olle E. Johansson wrote:
>>> I could not load the dialog module, because of a linking problem with the symbol destroy_dlg.
>>>
>>> Found that it's declared "inline" twice which may cause an issue. With this change, that I don't dare to commit myself, I could load dialog and explore it. Can someone take a look at this, someone that understands dialog and stuff?
>>>
>>> This was on OS/X.
>>>
>>> Thanks
>>> /O
>>>
>>> diff --git a/modules/dialog/dlg_hash.h b/modules/dialog/dlg_hash.h
>>> index 6994256..174ef15 100644
>>> --- a/modules/dialog/dlg_hash.h
>>> +++ b/modules/dialog/dlg_hash.h
>>> @@ -214,7 +214,7 @@ static inline void unlink_unsafe_dlg(dlg_entry_t *d_entry, dlg_cell_t *dlg)
>>>    * \brief Destroy a dialog, run callbacks and free memory
>>>    * \param dlg destroyed dialog
>>>    */
>>> -inline void destroy_dlg(dlg_cell_t *dlg);
>>> +void destroy_dlg(dlg_cell_t *dlg);
>>>       /*!
>>>
>>>
>>> _______________________________________________
>>> sr-dev mailing list
>>> sr-dev at lists.sip-router.org
>>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
>> -- 
>> Daniel-Constantin Mierla - http://www.asipto.com
>> http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
>>

-- 
Daniel-Constantin Mierla - http://www.asipto.com
http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda