[svn-commits] kpfleming: trunk r245680 - /trunk/apps/app_fax.c

[SVN commits to the Digium repositories]

Author: kpfleming
Date: Tue Feb  9 10:24:52 2010
New Revision: 245680

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=245680
Log:
Don't offer MMR or JBIG transcoding during T.38 negotiation.

After further discussion with Steve Underwood, we should not (yet) be offering
to receive MMR or JBIG transcoded streams from T.38 endpoints. A future spandsp
release will support those features, and then they can be enabled during
negotiation


Modified:
    trunk/apps/app_fax.c

Modified: trunk/apps/app_fax.c
URL: http://svnview.digium.com/svn/asterisk/trunk/apps/app_fax.c?view=diff&rev=245680&r1=245679&r2=245680
==============================================================================
--- trunk/apps/app_fax.c (original)
+++ trunk/apps/app_fax.c Tue Feb  9 10:24:52 2010
@@ -373,8 +373,12 @@
 							     .rate = AST_T38_RATE_14400,
 							     .rate_management = AST_T38_RATE_MANAGEMENT_TRANSFERRED_TCF,
 							     .fill_bit_removal = 1,
-							     .transcoding_mmr = 1,
-							     .transcoding_jbig = 1,
+/*
+ * spandsp has API calls to support MMR and JBIG transcoding, but they aren't
+ * implemented quite yet... so don't offer them to the remote endpoint
+ *							     .transcoding_mmr = 1,
+ *							     .transcoding_jbig = 1,
+*/
 	};
 
 	/* if in called party mode, try to use T.38 */