public inbox for [email protected]  

help / color / mirror / Atom feed

From: Alexey Borzov <[email protected]>
To: [email protected]
Cc: [email protected]
Cc: [email protected]
Subject: Re: development questions
Date: Sat, 14 Feb 2004 22:15:17 +0300
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>
References: <[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>

Hi!

Andreas Grabmüller wrote:
> When I saw Alexeys posts, I decided to not answer them - it's difficult for me to have an objective opinion as I simply don't like the way Alexey came in and said "everything you do sucks, what I do is the only right way you can do that". That makes me think "what the hell does that guy want from us?". However, let me try it.

You are unfair to yourself. While the code I inherited could use some 
optimisations (and I did optimise some parts of it) I mostly had just to 
refactor the existing code, it already worked OK.

> I have reviewed the code and it looks good - most of it seems to work, much of my code has been optimized.
> 
> I can't say if this code is good or this code is bad. He did the changes he stated in his mail, and they seem to work and the code techincally looks good (at least better than mine). The question is, do we want to have this changes? I like the template usage and the new path format page.html.{ln}. I personally don't like the usage of gettext as for me this seems to be overhead - before we had a php file containing all variables, now we have several .po etc. files which have to be edited by special software (I think?). All this is just a question of the personal point of view.

Yes, it is the matter of taste. The benefit of having .po files instead of .php 
files is (like I see it):
1) .po files are generated automatically and thus it is more difficult to have 
the translations that are out of sync due to programmer error.
2) It is not difficult to edit the .po files. There is some special software, 
but I just used the same PHPEdit to prepare the Russian translation, as if I was 
editing the plain text files.

view thread (23+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: development questions
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox