[asterisk-commits] kpfleming: branch 1.6.2 r245683 - in /branches/1.6.2: ./ apps/app_fax.c

[SVN commits to the Asterisk project]

Author: kpfleming
Date: Tue Feb  9 10:26:35 2010
New Revision: 245683

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=245683
Log:
Merged revisions 245680 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
  r245680 | kpfleming | 2010-02-09 10:24:52 -0600 (Tue, 09 Feb 2010) | 8 lines
  
  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:
    branches/1.6.2/   (props changed)
    branches/1.6.2/apps/app_fax.c

Propchange: branches/1.6.2/
------------------------------------------------------------------------------
Binary property 'trunk-merged' - no diff available.

Modified: branches/1.6.2/apps/app_fax.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/apps/app_fax.c?view=diff&rev=245683&r1=245682&r2=245683
==============================================================================
--- branches/1.6.2/apps/app_fax.c (original)
+++ branches/1.6.2/apps/app_fax.c Tue Feb  9 10:26:35 2010
@@ -375,8 +375,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 */