Doug MacEachern wrote:
> On Mon, 11 Mar 2002, Stas Bekman wrote:
> 
> 
>>in any case C allows you to say 'type   *' why WrapXS shouldn't.
>>
> 
> yeah but we don't do that.  'type *' is the code style, why invent a 
> different style for the .map files?  stick with the enforced style please.

I agree with enforcing a single space inside the type, but not with the 
trailing spaces. I thought to make the the map files more readable,
because we have many columns which aren't aligned.

You don't like allowing this too?:

char *    | foo | ...

here we preserve 'type *' but allow spaces afterwards for a better 
alignment. Here is an example from the code I'm working on. This reads 
much easier:

  unsigned long:DEFINE_conn_bytes      | | modperl_worker_score_t *:self
  unsigned long:DEFINE_conn_count      | | modperl_worker_score_t *:self
  char *       :DEFINE_client          | | modperl_worker_score_t *:self
  char *       :DEFINE_request         | | modperl_worker_score_t *:self
than:

  unsigned long:DEFINE_conn_bytes      | | modperl_worker_score_t *:self
  unsigned long:DEFINE_conn_count      | | modperl_worker_score_t *:self
  char *:DEFINE_client                 | | modperl_worker_score_t *:self
  char *:DEFINE_request                | | modperl_worker_score_t *:self

now add many 3 times more similar lines.

_____________________________________________________________________
Stas Bekman             JAm_pH      --   Just Another mod_perl Hacker
http://stason.org/      mod_perl Guide   http://perl.apache.org/guide
mailto:[EMAIL PROTECTED]  http://ticketmaster.com http://apacheweek.com
http://singlesheaven.com http://perl.apache.org http://perlmonth.com/


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to