Complete.Org: Mailing Lists: Archives: freeciv-dev: November 2001:
[Freeciv-Dev] Coding Guideline Updated RFC
Home

[Freeciv-Dev] Coding Guideline Updated RFC

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: freeciv development list <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Coding Guideline Updated RFC
From: Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx>
Date: Mon, 26 Nov 2001 15:43:28 +0000 (GMT)

Following suggestion of Raimar below is enlarged questionnaire.

============ Cut here.  Fill in blue ink only ==================

/*************** 
 * 1: init vars 
 ***************/

/* A */
int foo1a(struct city *punit)
{
  int x = punit->x;
}

/* B */
int foo1b(struct city *punit)
{
  int x;

  x = punit->x;
}


/**************************
 * 2: empty line after vars 
 **************************/

/* A */
int foo2a(void)
{
  int x;
  x = 3;
}

/* B */
int foo2b(void)
{
  int x;

  x = 3;
}


/****************
 * 3: comments 
 ****************/

/* A */
int foo3a(int x)
{
  x = 3;                        /* assign 3 to x */
}

/* B */
int foo3b(int x)
{
  /* assign 3 to x */
  x = 3;
}


/**************************
 * 4: extra {} on iterates 
 **************************/
int foo4(struct city *pcity)
{

  /* A */
  unit_list_iterate(pcity->units_supported, punit) {
    kill(punit);
  } unit_list_iterate_end;

  /* B */
  unit_list_iterate(pcity->units_supported, punit)
      kill(punit);
  unit_list_iterate_end;
}


/****************************************** 
 * 5: unnecessary braces after conditionals 
 ******************************************/
int foo5(int x)
{

  /* A */
  if (x == 3) {
    return;
  }

  /* B */
  if (x == 4)
    return;
}


/**************************************************** 
 * 6: unnecessary braces after conditionals with else 
 ****************************************************/
int foo6(int x)
{

  /* A */
  if (x == 3) {
    return 1;
  } else {
    return 0;
  }

  /* B */
  if (x == 4)
    return 1;
  else
    return 0;
}


/****************************
 * 7: merge declaration lines
 ****************************/

/* A */
int foo7a(struct city *pcity)
{
  int total, cost;
  int build = pcity->shield_stock;
}

/* B */
int foo7b(struct city *pcity)
{
  int total, cost, build = pcity->shield_stock;
}


/*************************
 * 8: double dereferencing
 *************************/

/* A */
int foo8a(struct city *pcity)
{
  struct player *owner = city_owner(pcity);
  struct nation_type *nation = get_nation_by_plr(owner);
}

/* B */
int foo8b(struct city *pcity)
{
  struct player *owner = city_owner(pcity);
  struct nation_type *nation;

  nation = get_nation_by_plr(owner);
}


__________________________________________________
Do You Yahoo!?
Everything you'll ever need on one web page from News and Sport to Email and 
Music Charts
http://uk.my.yahoo.com


[Prev in Thread] Current Thread [Next in Thread]