Vassilii Khachaturov <[EMAIL PROTECTED]> writes:
> -string ConvertRwyNumToSpokenString(string s) {
> +string ConvertRwyNumToSpokenString(const string s) {
this should be string ConvertRwyNumToSpokenString(const string& s)
so we don't make unnecessary copies.
> -double dclGetHorizontalSeparation(Point3D pos1, Point3D pos2) {
> +double dclGetHorizontalSeparation(const Point3D pos1, const Point3D pos2) {
same: const Point3D& pos1, const Point3D& pos2
and so on.
all the functions are making unnecessary copies instead of passing by
reference. i've changed a lot of these for my local copy, so i can
submit a patch if there is interest.
--alex--
--
| I believe the moment is at hand when, by a paranoiac and active |
| advance of the mind, it will be possible (simultaneously with |
| automatism and other passive states) to systematize confusion |
| and thus to help to discredit completely the world of reality. |
_______________________________________________
Flightgear-devel mailing list
Flightgear-devel@flightgear.org
http://mail.flightgear.org/mailman/listinfo/flightgear-devel
2f585eeea02e2c79d7b1d8c4963bae2d
