On 30/07/14 15:28, Olle E. Johansson wrote:
On 30 Jul 2014, at 15:20, Daniel-Constantin Mierla
<miconda(a)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(a)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