[Kde-extra-gear] cvsExtract patch

Klas Kalass kde-extra-gear@mail.kde.org
Sun, 9 Feb 2003 18:49:07 +0100


--Boundary-02=_ZSpR+0Qw3AS789B
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Content-Description: signed data
Content-Disposition: inline

Am Sonntag, 9. Februar 2003 18:42 schrieb Sebastian Trueg:
> On Sunday 09 February 2003 17:03, Klas Kalass wrote:
> > Am Sonntag, 9. Februar 2003 13:28 schrieb Sebastian Trueg:
> > +       echo "available languages: $LANGUAGES"
> > +       rm -r kde-i18n
> > +    fi
> > +fi
> >
> > Please do not put actual code like this above the "MAIN" section, but
> > rather inside. It does not matter code-wise but I think it is easier to
> > maintain if the distinction is kept.
>
> so should I create a function and call it?
That would be nicest, but I don't care so much for that. It is only that th=
e=20
structure of the file used to be (roughly):
=2D option handling
=2D functions
=2D "main" code
and it would be nice if you moved that code around so it fits in that=20
structure.=20


--Boundary-02=_ZSpR+0Qw3AS789B
Content-Type: application/pgp-signature
Content-Description: signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQA+RpSZyPp3mvWRmvgRAptZAJ0bqRoJPEP6KFSpxLsSNLgjQ5BY8ACfSayu
cmoRs3p9/TTpFBGnsQWqfKo=
=YfWd
-----END PGP SIGNATURE-----

--Boundary-02=_ZSpR+0Qw3AS789B--