[Freeciv-Dev] Re: (PR#2521) effects patch
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
<URL: http://rt.freeciv.org/Ticket/Display.html?id=2521 >
On Fri, 27 Aug 2004, Jason Short wrote:
>
> <URL: http://rt.freeciv.org/Ticket/Display.html?id=2521 >
>
> Vasco Alexandre da Silva Costa wrote:
> > <URL: http://rt.freeciv.org/Ticket/Display.html?id=2521 >
> >
> > On Wed, 25 Aug 2004, Jason Short wrote:
> >
> >
> >><URL: http://rt.freeciv.org/Ticket/Display.html?id=2521 >
> >>
> >>What's the difference between an eff, an eft, and an effect? Can't we
> >>just always call it an effect?
> >
> >
> > eff is a struct effect, eft is an enum effect_type.
>
> IMO that's bad. Aside from that they look almost identical eft is also
> a suitable shortening of effect. Can we just call them effect and
> effect_type?
>
> I'm not concerned with local variables but with global or file-wide
> types or fields. I was rather confused by the EFF types in packets.def.
> There is nothing about the REQ type that would make the casual reader
> think it has anything to with effects.
I've already renamed this stuff more than once. I suggest you and Mike
agree on names for whatever you don't think is clear enough and present
me the name list for approval afterwards. You are the native speakers
after all. :-)
I was just trying to avoid Pascal or Hungarian notation like names, which
I loathe.
---
Vasco Alexandre da Silva Costa @ Instituto Superior Tecnico, Lisboa
- [Freeciv-Dev] Re: (PR#2521) effects patch, Jason Short, 2004/08/25
- [Freeciv-Dev] Re: (PR#2521) effects patch, Jason Short, 2004/08/25
- [Freeciv-Dev] Re: (PR#2521) effects patch, Jason Short, 2004/08/25
- [Freeciv-Dev] Re: (PR#2521) effects patch, Jason Short, 2004/08/26
- [Freeciv-Dev] Re: (PR#2521) effects patch, Vasco Alexandre da Silva Costa, 2004/08/27
- [Freeciv-Dev] Re: (PR#2521) effects patch, Vasco Alexandre da Silva Costa, 2004/08/27
- [Freeciv-Dev] Re: (PR#2521) effects patch, Vasco Alexandre da Silva Costa, 2004/08/27
- [Freeciv-Dev] Re: (PR#2521) effects patch, Jason Short, 2004/08/27
- [Freeciv-Dev] Re: (PR#2521) effects patch, Jason Short, 2004/08/27
- [Freeciv-Dev] Re: (PR#2521) effects patch,
Vasco Alexandre da Silva Costa <=
- [Freeciv-Dev] Re: (PR#2521) effects patch, Vasco Alexandre da Silva Costa, 2004/08/27
- [Freeciv-Dev] Re: (PR#2521) effects patch, Jason Short, 2004/08/27
- [Freeciv-Dev] Re: (PR#2521) effects patch, Vasco Alexandre da Silva Costa, 2004/08/27
|
|