[asterisk-dev] [Code Review] Set MWI on Register and restore lastmsgssent function

[David Vossel]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1272/#review3735
-----------------------------------------------------------



/team/irroot/distrotech-customers-1.8/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/1272/#comment7482>

    What happens if this peer doesn't have a mailbox?



/team/irroot/distrotech-customers-1.8/channels/chan_sip.c
<https://reviewboard.asterisk.org/r/1272/#comment7481>

    What does lastmsgssent even mean to us?  It doesn't even look like it is ever used.  Unless we find we need to use it for something, I'd say remove it. Its probably worth looking in 1.4 to see if it is used there in order to understand the history of the variable.


- David


On 2011-06-16 14:49:16, irroot wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1272/
> -----------------------------------------------------------
> 
> (Updated 2011-06-16 14:49:16)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> peer->lastmsgssent is never set ...
> 
> send MWI to peer on register reverting to previous behavior 
> 
> 
> Diffs
> -----
> 
>   /team/irroot/distrotech-customers-1.8/channels/chan_sip.c 323469 
> 
> Diff: https://reviewboard.asterisk.org/r/1272/diff
> 
> 
> Testing
> -------
> 
> This works ans has been tested .... the problem with realtime is lastmsgssent is reset to -1 when
> it is reloaded from db.
> 
> perhaps remove lastmsgssent completely ??
> 
> 
> Thanks,
> 
> irroot
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110616/d8349604/attachment-0001.htm>