logo       

glip: This patch cleans the usage of pcity->occupied...: msg#00118

Subject: glip: This patch cleans the usage of pcity->occupied...
This is an automated notification of a change to freeciv cvs, 
on Fri Jul 25 07:48:32 PDT 2003 = Fri Jul 25 14:48:32 2003 (GMT)
by Gregory Berkolaiko <Gregory.Berkolaiko@xxxxxxxxxxxx>

---- Files affected:

freeciv/client packhand.c tilespec.c
freeciv/common city.h unit.c

---- Log message:
      Tag: HEAD

This patch cleans the usage of pcity->occupied up in two ways:

1.  Instead of pcity->occupied it's now pcity->client.occupied.  This
makes it harder to accidentally use it at the server (which was one of
the original bugs with this code), and is a prelude for more
client-side-only fields (see PR#3771).  Of course this means all users
have to be changed.

2.  Lots of comments added about the client-side usage of this field.
It seems that the client side of the code is correct, so some explanations 
are put into the comments.

Patch by Jason.




<Prev in Thread] Current Thread [Next in Thread>
Google Custom Search

Recently Viewed:
linux.arklinux....    user-groups.lin...    kde.usability/2...    ietf.ipp/2002-0...    mail.spam.spamc...    os.netbsd.devel...    audio.cd-record...    text.unicode.de...    php.documentati...    games.fps.halfl...    window-managers...    suse.oracle.gen...    bug-tracking.gn...    video.dvdrip.us...    xfree86.cvs/200...    java.netbeans.m...    network.argus/2...    culture.sf.kill...    debian.ports.al...    freebsd.questio...    qplus.devel/200...    handhelds.palm....   
Home | blog view | USPTO Patent Archive | advertise | OSDir is an inevitable website. super tiny logo

Free Magazines

Cisco News
Receive a free quarterly e-newsletter with exclusive articles on how Cisco IT uses its own products and solutions to enable the business.
subscribe

Systems Management News, the newspaper for IT systems administration and data center managers! Each issue of Systems Management News is chock-full of news and analysis to help you understand what's happening in your field.
subscribe

The Enterprise Newsweekly eWeek is the essential technology information source for builders of e-business.
subscribe

Oracle Magazine Oracle Magazine contains technology strategy articles, sample code, tips, Oracle and partner news, how to articles for developers and DBAs, and more. Oracle (NASDAQ: ORCL) is the world's largest enterprise software company.
subscribe

Total Telecom Total Telecom is "The Economist of the communications industry".
subscribe