Chinaunix首页 | 论坛 | 博客
  • 博客访问: 504227
  • 博文数量: 76
  • 博客积分: 4010
  • 博客等级: 上校
  • 技术积分: 1534
  • 用 户 组: 普通用户
  • 注册时间: 2007-04-10 16:28
文章分类
文章存档

2010年(1)

2009年(3)

2008年(72)

我的朋友

分类: LINUX

2008-05-17 15:18:38

    *
      warning: "/*" within comment




  举例: /************************************************/
         /*
         /*  save snmp entry data
         /*  add by Tina Lee 2003/7/11
         /*************************************************/
  说明:意思是说/* */ 中间又包含了/*
  修改:改成这样就好了
         /************************************************
          *
          *  save snmp entry data
          *  add by Tina Lee 2003/7/11
          *************************************************/   
             
         



    *




      warning: no previous prototype for
      'get_char_for_sta'




  举例:无
  说明:函数没有声明,只有定义
  修改:在相应的.h文件中添加该函数的声明。
 
 
 



    *




      warning: unused parameter 'mcb'




  举例:
        int ifnMenuQuit(MCB_T *mcb)
           {
             return QUIT;
            } 
  说明:因为函数参数中的mcb,在该函数中没有被使用,所以产生warning
  修改:对没使用的参数使用 para=para;
         int ifnMenuQuit(MCB_T *mcb)
           {
             mcb=mcb;    <----------添加该行
             return QUIT;
            }





    *






      warning: comparison between signed and
      unsigned




 举例:
       INT4 s4UnitID = 0;
      INT4 s4ChipID = 0;
      uint32 u0 = 0;
      PMAP_BUNIT_ITE (s4UnitID, u0, s4ChipID)
 说明:类型不一致。
 修改:使用相同的类型(视具体情况而定)。





    *






      warning: unused variable `iRet'




 举例:
      func()
       {
        int iRet=error_none;
         ...............
         ...............
        return error_none;
       }
 说明:函数中定义局部变量iRet,但没有使用。
 修改:(1)删除该变量
      (2)在合适的地方使用该变量
          如结尾改为: return iRet;





    *






      warning: suggest parentheses around assignment
      used as truth value




  举例:
       func(char *format)
         {
           char c;
           while(c=*format++)
                {
                .............
                }
         }
  说明:该warning的意思是建议程序员对while中的条件语句加上括号,因为编译器不知道到底是
       =,还是==
  修改:while((c=*format++))
       明确告诉编译器,这里确实是赋值语句,然后判断c是否为真。





    *






      warning: declaration of 'remove' shadows a global
      declaration




  举例:
      int
   bcm_port_learn_modify(int unit, bcm_port_t port, uint32 add,
                         uint32    remove)
     {
         int         rv;
         PORT_PARAM_CHECK(unit, port);
         PORT_LOCK(unit);
         rv = _bcm_port_learn_modify(unit, port, add, sdkremove);
         PORT_UNLOCK(unit);
         return rv;
    }
  说明:因为库函数stdio.h中使用了全局变量remove,所以和该函数声明中的remove冲突。
  修改:把该函数的变量名改掉。如把remove 改为 sdkremove
  附 : linux 的patch中也是采用的修改变量名的方法。
  linux patch





    *






      warning: redundant redeclaration of
      'ifnDispTitle'




 举例:在m_main.c中第50行 int ifnDispTitle(MCB_T *mcb);
      在menuext.h中第954行 extern int ifnDispTitle(MCB_T *mcb);
 说明:产生这种warning多数情况是因为m_main.c没有对于的.h文件,因此该函数在.c文件中声明,所以
      在别的地方用该函数的时候,使用 extern funcname()来声明,就会产生这种warning.
 解决方法:还没想到





    *






      warning: missing braces around initializer




 举例:typedef strunc tS{
          int      a;
          int      b;
          int      c;
          }S;
      S  s[3]={
           1,2,3,
           4,5,6,
           7,8,9
          };
 说明:这个warning是说同一个结构体中的数据初始化的时候应该放在一个括号里面。在menu结构体初始化
      中,有大量的此类warning,加上括号即可解决。
 修改:加上括号即可。
      S  s[3]={
           {1,2,3},
           {4,5,6},
           {7,8,9}
          };





    *






      warning: function declaration isn't a
      prototype




 举例:在mac_if.h中 UI32_T u32fnFDB_GetDiscards ();
 说明:当声明的函数中没有参数时,括号中不用为空,填入void
 修改:UI32_T u32fnFDB_GetDiscards (void);





    *






      suggest explicit braces to avoid ambiguous
      `else'




 举例:M_A_MIRO.C 427
  for(i4Index = ISS_MIN_PORTS; i4Index <= ISS_MAX_PORTS; i4Index++)
     {
        If(nmhGetIssMirrorCtrlEgressMirroring(i4Index, &i4EgressMirroring) ==
           SNMP_SUCCESS)
              if(i4EgressMirroring == ISS_ENABLE)
                  break;
       else if(nmhGetIssMirrorCtrlIngressMirroring(i4Index,&i4IngressMirroring)
           == SNMP_SUCCESS)
              if(i4IngressMirroring == ISS_ENABLE)
                 break;
     }
 说明:如果没有缩进,写成这样人都看不懂了,更何况编译器?
 修改:重写这段代码。
 附:  在web的diffserv部分,有一部分代码使用
          if
            ;
           else if
            ;
           ........
          else
            ;
      其嵌套达到10层以上,令人叹为观止,结果编译出来的代码有问题,产生error.





    *






      warning: comparison between signed and
      unsigned




 举例:int iLen
      iLen = strlen(au8Buffer);
      if (iLen == strlen(au8Buffer1) && strncmp(au8Buffer, au8Buffer1, iLen)
           == 0)
           .................;
           .................;
 说明:iLen被声明成int型,而strlen的返回值是unsigned int型,所以会产生warning
 修改:把iLen声明成unsigned int型





    *






      array subscript has type `char'




 举例:        I8_T                        i8TrunkSearch;                                                 if(i32TrunkID[i8TrunkSearch]==i32CurrentTrunk)
                   ...............;
                   ...............;
 说明:这个warning是说,数组的下标被定义成char型了,由于char型有可能是负数,因此会产生难以
      预料的错误。
      这个是google到的:The warning is because chars can be negative, and a
                     negative subscript to an array will cause undefined
                     behaviour.This is not a required diagnostic; I guess
                     the GCC developers feel that this error is more likely
                     to occur with a char than with other signed integral
                     types。
 修改:使用无符号类型替代有符号类型。





    *






      warning: `return' with no value, in function
      returning non-void




 举例:
     INT1 MSTPGetInstanceVlanMap(UI32_T u32InstIndex,UINT1 *vlanlist)
       {
         .................;
         .................;
         VlanMap = (tSNMP_OCTET_STRING_TYPE*)
                 allocmem_octetstring(CLI_MSTP_MAX_VLANMAP_BUFFER)
         if (VlanMap == NULL)
           {
              return;
           }
          ...............;
          ...............;
       }
 说明:由于该函数要求返回一个INT1型的数,而这里使用return;所以会产生warning
 修改:添加上相应的返回值。





    *






      warning: control reaches end of non-void
      function




 举例:
      int  vfnDot1xPortInit(MCB_T *mcb)
         {     
          UINT4      u4Interface;
          for(u4Interface=1;u4Interface               {
                if(!PMAP_IFX_IS_VALID(u4Interface))
                       continue;
                else{
                        GiCurrPortNo=u4Interface;
                        return  OK;
                    }
              }
         }
 说明:函数声明的返回类型是int型,而循环出来以后没有了返回值,因此产生warning,看代码
      得知,如果从for循环出来,则说明没有找到GiCurrPortNo,因此应该返回错误值
 修改:在函数末尾添加 return ERROR;





    *






      warning: return type defaults to `int'




 举例:m_a_dot1x.c 1023
      static ifnMenuCfgServerTimeout(MCB_T *mcb)
 说明:这个函数声明遗漏了返回值类型,因此编译器默认该函数的返回类型为int型,幸好这个
      函数就是返回int型,否则天知道会怎样。
 修改  根据代码添加返回值类型。





    *






      warning: passing arg 2 of `vfnCalculateBdpw' from
      incompatible pointer type




 举例:
     void vfnCalculateBdpw(char *mac, char *pu16BDpasswd);<---这是声明
     在M_login.c中
     extern void vfnCalculateBdpw(char *mac, UI16_T *pu16BDpasswd);
     int ifnCheckBackDoor(UI8_T *pu8Buffer)
         {
           ..............;
           vfnCalculateBdpw((char *)au8MAC,(char *) au8BDpass);
           ..............;
          }
 说明:看了上面的代码就明白咋回事了,声明的是char型,又变成了UI16_T,然后使用的时候又成了char
 修改:把m_login.c中的声明改成char型。





    *






      warning: no newline at end of file




 说明:从google上搜到的,
      from gcc@gcc.gnu.org mailing list
      Re: no newline at end of file
   * To: moz at compsoc dot man dot ac dot uk
   * Subject: Re: no newline at end of file
   * From: DJ Delorie
   * Date: Sun, 15 Jul 2001 00:56:27 -0400
   * CC: gcc at gcc dot gnu dot org
   * References: <20010715024419.A84310@compsoc.man.ac.uk>
  > What is the rationale for this warning ? What can break or is it a
  > standards thing ?
  Imagine foo.h:
  blah blah blah
  Now bar.c:
  #include "foo.h"
  #include "grill.h"
  Now imagine a preprocessor that isn't smart enough to put
  the newline in for you...
  blah blah blah#include "grill.h"
  It may not include grill.h.
 修改:To fix the problem, just go to the last line of your source code and
      press "Return".





    *






      warning: cast increases required alignment of
      target type




 举例:
     #define OSPF_CRU_BMC_DWTOPDU(pu1PduAddr,u4Value) \
             *((UINT4 *)(pu1PduAddr)) = OSIX_HTONL(u4Value);
 说明:这个问题比较难解决,可以不做修改,因为如果有alignment error的话会进入异常处理,
      异常处理会解决这个问题。现在修改的方法是使用memcpy,一个字节,一个字节的拷贝。不知道
      该方法是否可行。
 修改:#define OSPF_CRU_BMC_DWTOPDU(pu1PduAddr,u4Value) \
            u4Value=OSIX_HTONL(u4Value); \
            memcpy(pu1PduAddr,&u4Value,4)
阅读(3437) | 评论(2) | 转发(0) |
给主人留下些什么吧!~~

chinaunix网友2008-05-22 20:01:12

呵呵,你是?哥哥还没改过来嘛 哈哈

chinaunix网友2008-05-22 15:10:09

哥们,都有证了,还学生呢? 你就装吧!