Transport Tycoon Forums

The place to talk about Transport Tycoon
It is currently Thu Dec 13, 2018 9:54 am

All times are UTC




Post new topic  Reply to topic  [ 229 posts ]  Go to page Previous 16 7 8 9 1012 Next
Author Message
PostPosted: Wed Jun 26, 2013 5:12 pm 
Offline
Tycoon
Tycoon

Joined: Wed Aug 26, 2009 6:33 am
Posts: 2493
Location: Poland
"used the current trunk" - try with version that was used to create the patch (OpenTTD 1.3 assuming that you used patch from the previous post).

"But I have an error." - what kind of error? (related guide: http://www.chiark.greenend.org.uk/~sgtatham/bugs.html - especially "It doesn't work." section)

_________________
Correct me If I am wrong - PM me if my English is bad
AIAI - AI for OpenTTD


Top
   
PostPosted: Wed Jun 26, 2013 7:46 pm 
Offline
Tycoon
Tycoon

Joined: Wed Jan 17, 2007 12:14 am
Posts: 7228
you cannot simply apply a patch to current trunk version, you have to go back to the revision that the patch was written for.

_________________
You might not exactly be interested in Ferion, but if you are, have fun :)


Top
   
PostPosted: Wed Jun 26, 2013 8:04 pm 
Offline
Engineer
Engineer

Joined: Fri Jun 01, 2012 11:49 am
Posts: 10
ok thank you for the informations


Top
   
PostPosted: Thu Jul 11, 2013 10:36 am 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
Version 0.2.2 released - now the patch is up to trunk! Some minor bugs were fixed also. No binaries available, just source code (you can still download 0.2.1 binaries for OpenTTD 1.3.0).

_________________
:] don't worry, be happy and checkout my patches


Top
   
PostPosted: Fri Jul 12, 2013 10:17 pm 
Offline
Engineer
Engineer

Joined: Tue Nov 18, 2008 12:10 am
Posts: 19
I found a bug in clipboard version 0.2.1 (for openttd 1.3.0). In multiplayer (and not in singleplayer), when you paste within the copied region, the pasted content also becomes part of the clipboard. Depending on the direction of the offset, the new pasted content can then be pasted again.

So if you copy a 5x5 region with only a depot in the upper corner and paste it in the tile below, you are actually pasting 5 depots instead of the expected 1 depot.

Also, I frequently encounter failing 'not interrupted' assertions. I believe that these can be changed into warnings?

_________________
https://github.com/bcmpinc/openttd


Top
   
PostPosted: Sat Jul 13, 2013 9:44 am 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
bcmpinc wrote:
In multiplayer (and not in singleplayer), when you paste within the copied region, the pasted content also becomes part of the clipboard. Depending on the direction of the offset, the new pasted content can then be pasted again.
Actually this is the way it works. It's a TODO to expose this difference by providing different graphics and text labels. I even drew some icons but they were ugly :P and I didn't include them, yet.

In multiplayer, the tool works like you said:
copy -> select area to copy/paste
paste -> copy current content of area selected previously with copy tool and paste it instantly at given location

There is no need to synchronize anything over the network. Otherwise each player would have to poses it's own copy of all other player clipboards. Another solution would be to send clipboard over the network while pasting which is also problematic. Current approach is really bug proof and functional. And enough for players IMO. Do you think otherwise?

_________________
:] don't worry, be happy and checkout my patches


Last edited by adf88 on Sat Jul 13, 2013 10:30 am, edited 2 times in total.

Top
   
PostPosted: Sat Jul 13, 2013 10:28 am 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
bcmpinc wrote:
Also, I frequently encounter failing 'not interrupted' assertions. I believe that these can be changed into warnings?
Please, upload these files: crash.dmp, crash.sav, crash.log, crash.png. They can be found next to your openttd.cfg.

_________________
:] don't worry, be happy and checkout my patches


Top
   
PostPosted: Thu Jul 25, 2013 1:56 pm 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
New version available (0.2.3), minor bug fixes and refinements.

1.0.0 is coming, very soon!

_________________
:] don't worry, be happy and checkout my patches


Top
   
PostPosted: Thu Aug 01, 2013 9:51 pm 
Offline
Traffic Manager
Traffic Manager
User avatar

Joined: Tue Dec 21, 2004 5:35 am
Posts: 185
Skype: andrey-zaharov
Location: St. Petersburg, Russia
about svn.zip in first post:
got
Code:
src/openttd/src/ai/../script/api/script_window.hpp:26:44: fatal error: ../../widgets/clipboard_widget.h: No such file or directory


clipboard_widget just absent in patch

will be nice to have plain attachments - not zipped. thanks!

update: hg patch gives me
Code:
Message: Assertion failed at line 184 of /home/vaulter/src/openttd-hg/src/hotkeys.cpp: keycode < 128

on exit

Code:
*** OpenTTD Crash Report ***

Crash at: Thu Aug  1 22:24:35 2013
In game date: 1950-01-01 (0)

Crash reason:
 Signal:  Aborted (6)
 Message: Assertion failed at line 184 of /home/vaulter/src/openttd-hg/src/hotkeys.cpp: keycode < 128

OpenTTD version:
 Version:    h7ce060ce-CLIPBOARD (0)
 NewGRF ver: 1400642c
 Bits:       64
 Endian:     little
 Dedicated:  no
 Build date: Aug  2 2013 02:21:03

Stacktrace:
 [00] bin/openttd(_ZNK12CrashLogUnix13LogStacktraceEPcPKc+0x37) [0x6fa4d7]
 [01] bin/openttd(_ZNK8CrashLog12FillCrashLogEPcPKc+0xeb) [0x5d43cb]
 [02] bin/openttd(_ZNK8CrashLog12MakeCrashLogEv+0x7c) [0x5d465c]
 [03] bin/openttd() [0x6fa445]
 [04] /lib/x86_64-linux-gnu/libc.so.6(+0x370b0) [0x7fde32ddc0b0]
 [05] /lib/x86_64-linux-gnu/libc.so.6(gsignal+0x37) [0x7fde32ddc037]
 [06] /lib/x86_64-linux-gnu/libc.so.6(abort+0x148) [0x7fde32ddf698]
 [07] bin/openttd() [0x6e5e5f]
 [08] bin/openttd(_Z12SaveKeycodesPK6Hotkey+0x26c) [0x63c85c]
 [09] bin/openttd(_ZNK10HotkeyList4SaveEP7IniFile+0x4c) [0x63ceac]
 [10] bin/openttd() [0x63cf25]
 [11] bin/openttd(_Z12openttd_mainiPPc+0x187a) [0x6e7aca]
 [12] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf5) [0x7fde32dc6ea5]
 [13] bin/openttd() [0x525a01]

Operating system:
 Name:     Linux
 Release:  3.8.0-27-generic
 Version:  #40-Ubuntu SMP Tue Jul 9 00:17:05 UTC 2013
 Machine:  x86_64
 Compiler: GCC 4.7.3 "4.7.3"

Configuration:
 Blitter:      8bpp-optimized
 Graphics set: OpenGFX (1021)
 Language:     /home/vaulter/src/openttd-hg/bin/lang/russian.lng
 Music driver: extmidi
 Music set:    NoMusic (0)
 Network:      no
 Sound driver: sdl
 Sound set:    NoSound (2)
 Video driver: sdl

AI Configuration (local: 0):
  0: Human

Libraries:
 FontConfig: 2.10.2
 FreeType:   2.4.11
 ICU:        4.8.1.1
 LZMA:       5.1.0alpha
 LZO:        2.06
 PNG:        1.2.49
 SDL:        1.2.15
 Zlib:       1.2.7

---- gamelog start ----
Tick 0: new game started
     Revision text changed to h7ce060ce-CLIP, savegame version 185, not modified, _openttd_newgrf_version = 0x1400642c
     New game mode: 1 landscape: 0
---- gamelog end ----

*** End of OpenTTD Crash Report ***

_________________
OpenTTD Hard Game Pack


Top
   
PostPosted: Fri Aug 02, 2013 6:33 am 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
Sorry for that broken svn package. I'll release new version soon. Maybe today.

Vaulter wrote:
Assertion failed at line 184 of /home/vaulter/src/openttd-hg/src/hotkeys.cpp: keycode < 128


As a quick fix you can open your hotkeys.cfg and edit/add these entries:

[clipboard]
rotate_l = H
rotate_r = J
reflect_ne_sw = K
reflect_nw_se = L

HJKL keys are just an example, you can use other keys as well.

Vaulter wrote:
will be nice to have plain attachments - not zipped

I'm not sure why you do think so. I think it's better to have all files grouped in a package. Besides this way I can fit into "max 3 attachments" limit on this forum.

_________________
:] don't worry, be happy and checkout my patches


Top
   
PostPosted: Tue Aug 06, 2013 7:30 am 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
Milestone has bean reached - fully featured in-game clipboard for single and multi player vanilla games.
The Clipboard 1.0.0 is can be downloaded from usual place - the first page of this topic.

_________________
:] don't worry, be happy and checkout my patches


Top
   
PostPosted: Tue Aug 06, 2013 11:01 am 
Offline
Engineer
Engineer

Joined: Tue Aug 06, 2013 10:49 am
Posts: 4
Hello,

I try to apply your patch to the given svn version 25647 as you suggested:

Code:
svn update -r 25647
patch -p0 < the_clipboard_1.0.1-ottd_r25647-svn.patch


When compiling I get the following errors:
Code:
/home/lukas/Software/openttd-svn/src/clipboard_gui.cpp: In member function 'void ClipboardToolbarWindow::UpdateButtons()':
/home/lukas/Software/openttd-svn/src/clipboard_gui.cpp:256:40: error: '_clipboard_copy_paste_mode' was not declared in this scope


Since I am using the same version as you supposed and applied the svn patch (omitting the hg version), I do not know, what I did wrong.

To deliver further information, I attached the log files for configure and make. I am using Ubuntu 13.04.

Thank you for your answer in advance.


Attachments:
log_configure.txt [3.37 KiB]
Downloaded 100 times
log_make.txt [1.82 KiB]
Downloaded 106 times
Top
   
PostPosted: Tue Aug 06, 2013 2:06 pm 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
How the hell did it happen :? I built win32 binaries upon this patch and everything went fine.

Fixed in 1.0.2.

_________________
:] don't worry, be happy and checkout my patches


Top
   
PostPosted: Tue Aug 06, 2013 2:39 pm 
Offline
Engineer
Engineer

Joined: Tue Aug 06, 2013 10:49 am
Posts: 4
Thank you.

With your provided patch compiling works against 25650.


Top
   
PostPosted: Sun Aug 11, 2013 4:53 pm 
Offline
Engineer
Engineer

Joined: Mon Oct 24, 2011 1:06 pm
Posts: 3
Hey, I love this patch! I used it a long time ago under Windows. But now I'm using Mac OS. Any chance to bring this patch to the official release or can anyone compile this for Mac?


Top
   
PostPosted: Mon Aug 12, 2013 1:58 pm 
Offline
Engineer
Engineer

Joined: Mon Oct 24, 2011 1:06 pm
Posts: 3
Ok, I could compile it on my own.
But one question: It's not possible to save and load templates anymore, isn't it? Do you plan to bring back that feature? Would be great!


Top
   
PostPosted: Mon Aug 12, 2013 3:26 pm 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
It would be nice if you do that. Others may be happy with a Mac build. If you are not familiar with patching and compiling OpenTTD source code then just install development tools (there are several methods see OpenTTD Wiki::Compiling on Mac OS X#Installing required libraries) and I will provide you exact set of commands that you need to execute in terminal to do the rest.

Lupurus wrote:
It's not possible to save and load templates anymore, isn't it? Do you plan to bring back that feature?
Save/loading is not implemented and I don't plan to "bring it back". "The Clipboard" is mostly a rewrite and save/loading wasn't ever implemented. I decided already at the design stage that there will be no save/loading. Previous patches ("Build Templates (Copy&Paste)" and "Copy & Paste patch, reworked") were strongly criticized - some thought that they are killing creativity. Hence my decision.

_________________
:] don't worry, be happy and checkout my patches


Top
   
PostPosted: Mon Aug 12, 2013 8:03 pm 
Offline
Engineer
Engineer

Joined: Tue Nov 18, 2008 12:10 am
Posts: 19
adf88 wrote:
bcmpinc wrote:
In multiplayer (and not in singleplayer), when you paste within the copied region, the pasted content also becomes part of the clipboard. Depending on the direction of the offset, the new pasted content can then be pasted again.
Actually this is the way it works. It's a TODO to expose this difference by providing different graphics and text labels. I even drew some icons but they were ugly :P and I didn't include them, yet.

In multiplayer, the tool works like you said:
copy -> select area to copy/paste
paste -> copy current content of area selected previously with copy tool and paste it instantly at given location

There is no need to synchronize anything over the network. Otherwise each player would have to poses it's own copy of all other player clipboards. Another solution would be to send clipboard over the network while pasting which is also problematic. Current approach is really bug proof and functional. And enough for players IMO. Do you think otherwise?

My preferred solution would be to send the clipboard over the network while pasting, as this is required to implement save & load functionality for multiplayer games.

Furthermore, the current solution has at least two bugs. One is your TODO. The other is that pasting once inside the copied region (in a multiplayer game) can cause the contents of the region to be pasted multiple times.

For example, if you copy [1] ('.'=grass, 'D'=depot) and then paste it one tile to the right, you get [2] instead of [3], which is what I would expect. Furthermore, if you pasted it one tile to the left, you would get [4].
Code:
1] .....D.....
2] .....DDDDDD
3] .....DD....
4] ....DD.....

_________________
https://github.com/bcmpinc/openttd


Top
   
PostPosted: Tue Aug 13, 2013 7:15 am 
Offline
Chief Executive
Chief Executive
User avatar

Joined: Mon Jan 14, 2008 3:51 pm
Posts: 644
Location: PL
bcmpinc wrote:
My preferred solution would be to send the clipboard over the network while pasting, as this is required to implement save & load functionality for multiplayer games.
What do you mean by that? What "save & load functionality"? You mean save/loading clipboard templates? See my previous post.

bcmpinc wrote:
One is your TODO.
It's no more a TODO since v1.0.0.

bcmpinc wrote:
The other is that pasting once inside the copied region (in a multiplayer game) can cause the contents of the region to be pasted multiple times...
. You are simply wrong and you didn't even bother to check how does it work. The tool works as intended. There is a "select" tool, you can pick a tile area with it. And There is a "copy & paste" tool which copies selected area into clipboard buffer and then pastes it instantly at a given location.

_________________
:] don't worry, be happy and checkout my patches


Top
   
PostPosted: Tue Aug 13, 2013 1:28 pm 
Offline
Engineer
Engineer

Joined: Mon Oct 24, 2011 1:06 pm
Posts: 3
adf88 wrote:
Lupurus wrote:
It's not possible to save and load templates anymore, isn't it? Do you plan to bring back that feature?
Save/loading is not implemented and I don't plan to "bring it back". "The Clipboard" is mostly a rewrite and save/loading wasn't ever implemented. I decided already at the design stage that there will be no save/loading. Previous patches ("Build Templates (Copy&Paste)" and "Copy & Paste patch, reworked") were strongly criticized - some thought that they are killing creativity. Hence my decision.


Isn't it the decision of the user, how much he wants to creative or not? I, for one, don't have the patience to build each cross with every signal from the start on. But am I really the only one, wo thinks like this?


Top
   
Display posts from previous:  Sort by  
Post new topic  Reply to topic  [ 229 posts ]  Go to page Previous 16 7 8 9 1012 Next

All times are UTC


Who is online

Users browsing this forum: No registered users and 6 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:  
Powered by phpBB © 2000-2018 phpBB Limited

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