[asterisk-dev] Insulting source code comments in main/channel.c

Vlasis Hatzistavrou (KTI) vhatz at kinetix.gr
Mon Nov 23 05:56:06 CST 2009


Well said, Florian.

I would go as far as saying that this particular comment is a perfect 
example of how some people live in their own versions of reality.

Best regards,
Vlasis Hatzistavrou.

Floimair Florian wrote:
> Hi guys!
> 
>  
> 
> While reading through some of the code today I came across this in 
> main/channel.c (line no. 655 in 1.4.27)
> 
>  
> 
> /*! \brief Pick the best audio codec */
> 
> int ast_best_codec(int fmts)
> 
> {
> 
>       /* This just our opinion, expressed in code.  We are asked to choose
> 
>          the best codec to use, given no information */
> 
>       int x;
> 
>       static int prefs[] =
> 
>       {
> 
>             /*! Okay, ulaw is used by all telephony equipment, so start 
> with it */
> 
>             AST_FORMAT_ULAW,
> 
> *            /*! Unless of course, you're a silly European, so then 
> prefer ALAW */*
> 
>             AST_FORMAT_ALAW,
> 
>> 
>  
> 
> Well, what can I say. I’m European but I’m definitely not silly. This is 
> not only insulting but also very narrow-minded. Therefore I kindly 
> request you to remove the “silly” from the comment in the code.
> 
>  
> 
> I have no idea who is responsible for that but for an open source 
> project I suppose that a lot of people have had a look on this and did 
> not offend, which is kind of awkward.
> 
> Also I would like to note that u-law is the suitable codec for North 
> America and Japan, while the majority of the rest of the world uses 
> a-law (so the “comment” would in fact better fit to the 
> Americans/Japanese and u-law).
> 
>  
> 
> But enough of the talk. Please just remove the word “silly” from the 
> comment and everyone should be fine.
> 
>  
> 
>  
> 
>  
> 
> With best regards
> 
> *Florian Floimair*
> Firmware Development
> 
> *COMMEND INTERNATIONAL GMBH*
> A-5020 Salzburg, Saalachstraße 51
> Tel: +43-662-85 62 25
> Sip: f.floimair at commend.com <sip:f.floimair at commend.com>
> Fax: +43-662-85 62 26
> f.floimair at commend.com <mailto:f.floimair at commend.com>
> http://www.commend.com <http://www.commend.com/>
> 
> *Security and Communication by Commend*
> 
> FN 178618z | LG Salzburg
> 
>  
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> --Bandwidth and Colocation Provided by http://www.api-digital.com--
> 
> asterisk-dev mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-dev



More information about the asterisk-dev mailing list