Transport Tycoon Forums

The place to talk about Transport Tycoon
It is currently Thu Sep 02, 2010 5:53 pm

All times are UTC




Post new topic This topic is locked, you cannot edit posts or make further replies.  [ 50 posts ]  Go to page Previous  1, 2, 3
Author Message
 Post subject:
PostPosted: Sun Jul 02, 2006 3:35 pm 
Director
Director
User avatar
Offline

Joined: Thu Aug 11, 2005 8:22 am
Posts: 627
Location: Localhost, 127.0.0.1, [The Netherlands: South Holland-> Westland]
Strange then, i compiled a clean SVN r5477 from /trunk/ and without your patch it does compile ( rebuilt solution in MSVS2005 ). With your patch it gives the warnings in my previous post.

_________________
Quote:
# Programming is like sex, one mistake and you have to support it for the rest of your life. (Michael Sinz)


Top
 Profile  
 
 Post subject:
PostPosted: Sun Jul 02, 2006 6:13 pm 
Tycoon
Tycoon
Offline

Joined: Mon Oct 25, 2004 3:39 pm
Posts: 4349
Location: Mississauga
Try this one. Guys told me on IRC that StringID wasn't defined (for some files strings.h was called before openttd.h, which was bad)


Attachments:
measurement_6.patch [17.74 KiB]
Downloaded 221 times
Top
 Profile E-mail  
 
 Post subject:
PostPosted: Sun Jul 02, 2006 9:09 pm 
Director
Director
User avatar
Offline

Joined: Thu Aug 11, 2005 8:22 am
Posts: 627
Location: Localhost, 127.0.0.1, [The Netherlands: South Holland-> Westland]
Compiles perfectly! Thanks!

Have fun on your holiday ;)

_________________
Quote:
# Programming is like sex, one mistake and you have to support it for the rest of your life. (Michael Sinz)


Top
 Profile  
 
 Post subject:
PostPosted: Sun Jul 02, 2006 9:29 pm 
Tycoon
Tycoon
Offline

Joined: Mon Oct 25, 2004 3:39 pm
Posts: 4349
Location: Mississauga
Thank you so much :) Bye!


Top
 Profile E-mail  
 
 Post subject:
PostPosted: Fri Aug 11, 2006 5:30 pm 
Director
Director
User avatar
Offline

Joined: Thu Aug 11, 2005 8:22 am
Posts: 627
Location: Localhost, 127.0.0.1, [The Netherlands: South Holland-> Westland]
Darn. The patch doesn't work anymore after this revision-changeset: http://svn.openttd.org/cgi-bin/trac.cgi/changeset/5806

Here Tron deletes the function FindLandscapeHeight() which is used about 7 times in measurement_6.patch. Maybe you could update your patch? That would be great.

_________________
Quote:
# Programming is like sex, one mistake and you have to support it for the rest of your life. (Michael Sinz)


Top
 Profile  
 
 Post subject:
PostPosted: Sat Aug 12, 2006 8:16 pm 
Tycoon
Tycoon
Offline

Joined: Mon Oct 25, 2004 3:39 pm
Posts: 4349
Location: Mississauga
Allright, I'll take a look at this and fix it.


Top
 Profile E-mail  
 
 Post subject:
PostPosted: Tue Aug 15, 2006 11:36 am 
Tycoon
Tycoon
Offline

Joined: Mon Oct 25, 2004 3:39 pm
Posts: 4349
Location: Mississauga
I'm trying to finalize this patch.
Thanks to Tron, who helped me optimizing the code. Also he helped me realize I used FindLandscapeHeight bad way, or rather I didn't even had to use it.

There are two things, however, that aren't as good as I wanted:

2x1 autorail:
what's wrong:
Dragging in a "diagonal" direction shows length = 2 instead of 1
why it isn't that bad:
It is very rare (or it doesn't happen, ever) to measure length on such a small distance

Level land tool:
what's wrong:
I couldn't do what one user suggested; the measurement isn't reduced by 1 in width and height
why I couldn't do it:
The function I'm measuring in doesn't contain selection method data, but I'll work on it


Please voice your comments, also about the code style. Thank you in advance


Attachments:
measurement_8.patch [15.23 KiB]
Downloaded 249 times
Top
 Profile E-mail  
 
 Post subject:
PostPosted: Thu Oct 12, 2006 8:19 pm 
Tycoon
Tycoon
Offline

Joined: Mon Oct 25, 2004 3:39 pm
Posts: 4349
Location: Mississauga
r6758 is our happy revision.
Thanks to great help of Darkvater and other developers, and of course your tips and ideas, the patch is ready :wink:


Top
 Profile E-mail  
 
 Post subject:
PostPosted: Thu Oct 12, 2006 8:29 pm 
Director
Director
User avatar
Offline

Joined: Thu Aug 11, 2005 8:22 am
Posts: 627
Location: Localhost, 127.0.0.1, [The Netherlands: South Holland-> Westland]
MeusH wrote:
r6758 is our happy revision.
Thanks to great help of Darkvater and other developers, and of course your tips and ideas, the patch is ready :wink:


Finally 8)

My idea, and your programming (meush). Great! 8) 8) 8)

_________________
Quote:
# Programming is like sex, one mistake and you have to support it for the rest of your life. (Michael Sinz)


Top
 Profile  
 
 Post subject:
PostPosted: Thu Oct 12, 2006 9:37 pm 
Tycoon
Tycoon
User avatar
Offline

Joined: Mon Feb 24, 2003 6:45 pm
Posts: 3032
Location: Netherlands
/me sneaks in

/me looks around

I'll just close this thread then...

/me runs away

_________________
TrueLight: "Did you bother to read any of the replies, or you just pressed 'Reply' and started typing?"
<@[R-Dk]FoRbiDDeN> "HELP, this litte arrow thing keeps following my mouse, and I can't make it go away."


Top
 Profile  
 
Display posts from previous:  Sort by  
Post new topic This topic is locked, you cannot edit posts or make further replies.  [ 50 posts ]  Go to page Previous  1, 2, 3

All times are UTC


Who is online

Users browsing this forum: Klaatu and 2 guests


You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot post attachments in this forum

Search for:
Jump to:  
Ipod Shuffle | Cheap Plane Tickets | Proactol | House Insurance | Breast Enlargement
Powered by phpBB © 2000-2009 phpBB Group

Copyright © Owen Rudge/The Transport Tycoon Forums 2001-2010.
Hosted by Zernebok.