Merged revisions 67492 via svnmerge from
authorRussell Bryant <russell@russellbryant.com>
Tue, 5 Jun 2007 20:55:59 +0000 (20:55 +0000)
committerRussell Bryant <russell@russellbryant.com>
Tue, 5 Jun 2007 20:55:59 +0000 (20:55 +0000)
commit7903eac38fdd4beca9f2b584a57b616a684b25ee
tree80b8510cbe58489d2aba57c210060601214b11ce
parent6a8be027d11350872465913cc42a6cce25e46c31
Merged revisions 67492 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r67492 | russell | 2007-06-05 15:53:28 -0500 (Tue, 05 Jun 2007) | 16 lines

This bug has been hanging over my head ever since I wrote this SLA code.
Every time I tried to go debug it by adding some debug output, the behavior
would change.  It turns out I wasn't crazy.  I had the following piece of code:

   if (remove)
      AST_LIST_REMOVE_CURRENT(...);

Well, AST_LIST_REMOVE_CURRENT was not wrapped in braces, so my conditional
statement didn't do much good at all.  It always ran at least all of the
macro minus the first statement, so I was seeing list entries magically
disappear when they weren't supposed to.

After many hours of debugging, I have come to this extremely irritating fix. :)

(issues #9581, #9497)

........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@67493 65c4cc65-6c06-0410-ace0-fbb531ad65f3
include/asterisk/linkedlists.h