Ok thanks for the suggestion, I think I have a more generalized working solution I can submit. Thanks.
>Sounds like while it saved you a lot of work to get your immediate goal accomplished, it was built in a manner considered not very desirable. Specifically, cut and >paste code. >From your description, it sounds like most of the code is exactly as it was originally, but now in more than one location. A suggestion might be to go >back to the original blacklist app and see how you could have refactored the code to be generalised and allow you to pass in an additional value that determines if >you want blacklisting or whitelisting(friends). Maybe keep an eye forward for other options such as grey listing. Maybe you could change lookupblacklist to just >lookuplist(blacklist), lookuplist(friend), lookuplist(questionable). My opinion on this is that you then have made asterisk much more flexable than by adding such a >narrow feature that was mostly the same code. While I don't use the feature, I am sure a patch to implement the above, plus a backwards compatible app >registration that makes a lookupblacklist() point to the lookuplist(blacklist) would be acceptable provided you have sent in your disclaimer to Digium and you post >your patch to bugs.digium.com
>-- Steven Critchfield <[EMAIL PROTECTED]> _______________________________________________ --Bandwidth and Colocation provided by Easynews.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
