Log for #openttd on 12th March 2017:
Times are UTC Toggle Colours
00:01:32  *** Wormnest has quit IRC
00:05:29  *** FLHerne has quit IRC
00:49:12  *** maciozo has quit IRC
00:58:46  *** Maarten has quit IRC
01:00:06  *** Maarten has joined #openttd
01:00:55  *** Progman has quit IRC
01:01:37  *** Gja has quit IRC
01:17:22  *** FLHerne has joined #openttd
01:20:15  *** iSoSyS has quit IRC
02:02:25  *** glx has quit IRC
02:05:28  *** FLHerne has quit IRC
02:45:34  *** HerzogDeXtEr has quit IRC
02:49:38  *** Snail has joined #openttd
02:58:31  *** Snail has quit IRC
02:59:02  *** Snail has joined #openttd
02:59:30  *** Snail has joined #openttd
03:00:35  *** Snail has joined #openttd
03:07:05  *** Snail has joined #openttd
03:28:05  *** chomwitt has joined #openttd
03:32:58  *** Snail has quit IRC
03:58:54  *** tokai has joined #openttd
03:58:54  *** ChanServ sets mode: +v tokai
04:06:00  *** tokai|noir has quit IRC
06:24:13  *** Alberth has joined #openttd
06:24:13  *** ChanServ sets mode: +o Alberth
06:24:23  <Alberth> min
06:24:26  <Alberth> +o
06:57:56  *** sim-al2 is now known as Guest132
06:57:57  *** sim-al2 has joined #openttd
07:00:12  *** supermop has quit IRC
07:00:26  *** supermop has joined #openttd
07:01:17  *** Guest132 has quit IRC
07:18:24  *** cHawk has quit IRC
07:30:39  *** Montana has joined #openttd
07:31:57  *** andythenorth has joined #openttd
07:33:21  <andythenorth> o/
07:39:04  *** zeknurn has quit IRC
07:40:16  <Alberth> o/
07:41:09  *** zeknurn has joined #openttd
07:49:01  *** sim-al2 has quit IRC
07:50:16  *** Guest75 has quit IRC
07:53:33  <andythenorth> Alberth: playing any games at the moment? o_O
08:01:25  <Alberth> not sure :p
08:01:50  <Alberth> I think I still have one or two steel-town games, but didn't do that for a while
08:02:17  <Alberth> this morning I played with NSW trains for half an hour, but too realistic
08:03:07  <Alberth> trains do seem to have nice colours though, maybe I should try a little longer
08:03:38  <Alberth> I was trying to get iron horse make-stuff move, mostly :p
08:03:44  <Alberth> s/was/am/
08:06:51  <Alberth> then aircraft type display patch, state machine code generator for corsix, fixing tar loading as step in terkhen scenario stuff
08:07:12  <Alberth> surely I may have missed a few items :p
08:07:57  *** Hiddenfunstuff has joined #openttd
08:14:31  *** sla_ro|master has joined #openttd
08:17:05  *** Progman has joined #openttd
08:29:27  * andythenorth considers buying Tropico
08:29:49  <andythenorth> think I need some new game to wake my brain up
08:30:03  <Alberth> factorio seems popular too :p
08:30:39  <Alberth> but Tropico looks nice too
08:30:48  <andythenorth> I tried factorio :)
08:31:04  <andythenorth> “not for me"
08:32:48  <Supercheese> Really?
08:32:57  <Supercheese> Factorio is like crack
08:33:00  <Supercheese> Cracktorio
08:33:07  <Alberth> too much low-level stuff?
08:34:26  <Alberth> you could try corsixth, but you need the original data (cheap to get), the game isn't very deep though, easy to figure out
08:35:40  <andythenorth> factorio is minecraft, but you don’t build your own redstone
08:35:42  <andythenorth> afaict
08:35:51  <andythenorth> ‘x is y’ is such a silly thing to say :)
08:35:51  <Alberth> I heard about Software Inc a few days ago, about managing a software company; not sure if it's fun
08:35:59  <andythenorth> sounds like RL :P
08:36:10  <andythenorth> I play that game 12 hours a day weekdays
08:36:13  <Alberth> yeah, that's my problem too, somewhat
08:37:10  <Alberth> newer simcities look very nice, I watched a few steel sky(???) videos, seem quite tunable
08:38:03  <Alberth> maybe checkout a few "builder sims 2017" videos ?
08:38:05  <andythenorth> I need something that doesn’t require much hardware
08:38:15  <Alberth> ah, no idea there
08:38:55  <Alberth> corsix would work, I think, try the demo level, that's free
08:40:28  * andythenorth reading about it
08:47:26  *** zeknurn has quit IRC
08:47:38  *** zeknurn has joined #openttd
08:47:51  * Supercheese loads up Cracktorio
09:16:05  <andythenorth> I have no idea how to refactor this :)
09:16:19  <andythenorth> the CPP macro needs to be replaced by python templating
09:18:29  <Alberth> you can't expand a template with 2 parameters for a list of such pairs?
09:19:45  <andythenorth> I can yes
09:19:57  <andythenorth> my problem is this....
09:20:08  <andythenorth> keeping data in templates is considered a no-no-no
09:20:20  <andythenorth> but some of the checks don’t fit a standard format
09:20:48  <andythenorth> the list of incompatible grfs should be a dict / objects in a python file
09:21:00  <andythenorth> but the non-standard cases make a mess of that
09:21:13  <Eddi|zuHause> why does that need refactoring at all?
09:21:25  <andythenorth> I am removing CPP
09:21:38  <andythenorth> the destination is no make, no GCC
09:21:40  <Alberth> yep, so either move them out to some NML source?
09:21:45  <Alberth> s/either//
09:22:11  <andythenorth> I think I just have to write a complicated template probably )
09:22:15  <andythenorth> :)
09:22:38  <andythenorth> the limiting factor is then that I don’t know what the non-standard cases are for
09:22:39  <Alberth> such a nice trap :p
09:22:48  <andythenorth> so I’m writing code with no idea of purpose
09:22:52  <andythenorth> nor how to test it works
09:23:06  <Eddi|zuHause> i don't know your python templates
09:23:09  <andythenorth> in which case, can I just delete it?  (As I’ll probably break it)
09:23:24  <Eddi|zuHause> but there's probably an almost 1:1 translation of that file into your templates
09:23:33  <Eddi|zuHause> requiring no refactoring
09:23:56  <Alberth> simplest is to copy the entire string to your python code
09:24:10  <Alberth> which almost trivially will not break anything
09:24:46  <Alberth> simplest way to check afterwards is compare output with original file, possibly after CPP processing
09:25:41  <Alberth> Grfs tend to stick around, so I'd say removal is not a useful option
09:26:44  <Alberth> tbh, I don't see the problem in keeping the non-standard cases as is, GRFs are stable, code isn't going to be changed
09:26:53  <Alberth> ie it's constant text
09:27:29  * andythenorth is being purist about separating data and templates
09:27:34  <Alberth> quite
09:27:35  <andythenorth> that distinction might have to fail
09:27:53  <Alberth> you also move <h1> to python code, don't you?
09:27:56  <andythenorth> no list of incompatible grfs in the docs for you lot then :P
09:28:06  <Alberth> I mean, the HTML standard just might change
09:28:17  <andythenorth> it does :)
09:28:25  <andythenorth> then we have to rewrite templates
09:28:34  <Alberth> for <h1> ?  :p
09:28:56  <andythenorth> yes, the html 5 document outlining algorithm changed the semantics
09:29:02  <andythenorth> although no browser implements it :P
09:29:13  <Alberth> :)
09:29:28  <Eddi|zuHause> andythenorth: i still don't know your templates, but the way i see it you have these parameters (some of which may be None): 1) grf-id, error message, error message parameter, condition (outer), condition (parameter), other grf-id
09:29:39  <andythenorth> Eddi|zuHause: templating it is the least of the problems :)
09:29:55  <andythenorth> it’s a question about data
09:30:03  <andythenorth> is the list of incompatible grfs data?
09:30:09  <Eddi|zuHause> yes
09:30:30  <andythenorth> ok, so it shouldn’t be stored in the presentation layer
09:30:37  <andythenorth> non-controversial
09:31:11  <Eddi|zuHause> but still, what if your "data" is just make a table with the parameters i said above
09:31:41  <andythenorth> yes, your parameters look correct
09:32:00  <andythenorth> know I just need to be sure I implement correctly
09:32:14  <andythenorth> for which I just need to diff old/new?
09:32:15  <Eddi|zuHause> and your template does things like "if (condition_outer is not None): write code"
09:32:25  <andythenorth> I don’t actually need to know what the code does, just don’t change the output?
09:32:44  <Eddi|zuHause> yes, pretty much
09:32:54  <andythenorth> ok problem solved thanks
09:33:02  *** ATS65 has joined #openttd
09:34:27  *** Supercheese has quit IRC
09:37:12  <Alberth> how does this horse look?
09:37:45  <Alberth> properly coloured version looks much better though
09:38:14  * andythenorth reading
09:39:21  <andythenorth> Alberth: looks about 15 times easier to work with :)
09:39:34  <andythenorth> I haven’t tested it yet
09:40:06  *** ATS64 has quit IRC
09:41:12  <Alberth> oh, and only a factor 13 in Makefile line reductions :)
09:41:44  <andythenorth> ha
09:42:29  <andythenorth> is it ready to push?
09:43:31  *** supermop has quit IRC
09:44:04  <Alberth> could be
09:44:31  <Alberth> I can also give you a tar with the goodies
09:48:20  <andythenorth> just push it :)
09:48:24  <andythenorth> see what bundles does :P
09:48:31  <Alberth> break :p
09:48:45  <Alberth> at least it's missing a few targets
09:49:32  <Alberth> but I'll copy the stuff properly into the repo
10:05:58  *** HerzogDeXtEr has joined #openttd
10:11:54  <Alberth> let's see how hard it fails :p
10:13:51  <Alberth> likely, the bin/* stuff should be in its own repo
10:15:33  *** Montana has quit IRC
10:16:41  * peter1138 ponders installing Debian 6 or so just to compile old openttd...
10:20:31  <andythenorth> :)
10:21:21  <Alberth> can't you simply download an old binary?
10:21:35  <Eddi|zuHause> i don't think we store old binaries
10:25:24  <LordAro>
10:25:38  <LordAro> not linux, insterestingly
10:25:59  <LordAro> something about that version
10:26:36  <Eddi|zuHause> that was before "my time"
10:27:13  <Eddi|zuHause> hm, i have a checkout at r12066, but it fails compiling with "missing daylength.h"
10:27:57  <LordAro> me too, but it's the first old version that came to mind :)
10:28:08  <Alberth> :o it had daylength, but we removed it?
10:28:18  <Eddi|zuHause> no, it has a patch applied
10:28:19  <peter1138> old versions exist
10:28:34  <peter1138> if you can remember what versions existed
10:28:55  <Eddi|zuHause> yeah, old releases. but not old nightlies
10:29:11  <Eddi|zuHause> which makes bisecting a bit... unwieldy
10:29:29  <LordAro> in theory it should all compile, i guess
10:29:36  <LordAro> just a matter of getting the right libraries
10:29:40  <peter1138> hahahaha
10:29:43  <Eddi|zuHause> LordAro: no, compilers changed
10:29:44  <peter1138> need the right compiler too
10:30:10  <LordAro> i'd like to think that different compilers would only change the set of warnings produced
10:30:41  <Alberth> then you need to change the configuration settings, at least
10:31:25  <peter1138> In file included from /home/petern/src/openttd/src/ai/../script/api/ai/ai_station.hpp.sq:12:0,
10:31:28  <peter1138>                  from /home/petern/src/openttd/src/ai/ai_instance.cpp:70:
10:31:28  <peter1138> yeah... what
10:31:31  <peter1138> /home/petern/src/openttd/src/ai/../script/api/ai/../script_station.hpp:52:96: error: call to non-constexpr function ‘StationFacility operator|(StationFacility, StationFacility)’
10:31:35  <peter1138>    STATION_ANY        = STATION_TRAIN | STATION_TRUCK_STOP | STATION_BUS_STOP | STATION_AIRPORT | STATION_DOCK, ///< All station types
10:31:58  <Alberth> wow :)
10:32:55  *** maciozo has joined #openttd
10:38:57  <Eddi|zuHause> uhm what? "make" -> "source.lst: file not found"...
10:39:05  *** Wormnest has joined #openttd
10:39:11  <Eddi|zuHause> err, source.list
10:39:36  <Eddi|zuHause> but it's there?
10:40:02  <Alberth> spelling correct? directory correct?
10:40:30  <Eddi|zuHause> it's a thing that definitely compiled once...
10:41:01  <Eddi|zuHause> version from 2008, binary from 2013
10:41:31  *** gelignite has joined #openttd
10:45:41  <Eddi|zuHause> so, manual reconfigure seems to work
10:46:49  <Eddi|zuHause> has a million warnings, but seems to compile so far
10:47:30  <Eddi|zuHause> has norev000 for some reason?
10:50:01  * LordAro tries compiling r20001
10:50:09  * LordAro gets immediate failure from squirrel
10:51:03  <peter1138> :)
10:51:21  <Eddi|zuHause> i don't know if r12xxx has squirrel yet
10:52:53  <LordAro> surprised how many changes have been made to squirrel over the years
10:53:45  <Alberth> everything moves :)
10:53:52  *** frosch123 has joined #openttd
10:53:58  <Alberth> quak
10:53:59  <Eddi|zuHause> hm, i have some cargodist version from 2013 which seems to make fine
10:55:00  <Eddi|zuHause> that seems to be r25xxx based
10:56:18  <frosch123> moi
10:56:35  <Eddi|zuHause> you dropped an n
10:58:27  <Eddi|zuHause> someone remind me that next time i compile something i make -jX
10:58:45  <andythenorth> next time you compile, make -jX
10:58:49  <andythenorth> probably 13 or so
10:59:08  <Eddi|zuHause> last time i checked, beyond -j6 made hardly a difference
10:59:08  <Alberth> Eddi|zuHause:   export MAKEFLAGS=-j4
10:59:09  <LordAro> export MAKEFLAGS
10:59:16  <LordAro> ^^
10:59:55  <Alberth> I once spend the effort to configure it :)
11:00:16  <Eddi|zuHause> yeah, but i always forget in which file i should put that to make it permanent
11:01:06  <Alberth> login script if you have it, else the normal start script; for bash, that would be .bash_profile and .bshrc
11:01:35  <Alberth> not that some systems have weird ideas whether or not you log in
11:01:42  <Alberth> *note
11:02:23  <Alberth> as this overrides any previous setting (instead of extending it), it will work in .bashrc
11:09:29  <LordAro> well that wasn't *so* hard
11:10:02  <LordAro> (diff is partial r27079)
11:17:45  <andythenorth> Alberth: IH makefile works
11:17:58  <Alberth> tell Jenkins that :p
11:18:08  <Alberth> working on it
11:18:22  <Eddi|zuHause> LordAro: sure, but a year from now, nobody remembers that, and we go looking again :p
11:18:50  <LordAro> heh, true
11:18:56  <LordAro> could wiki it, somehow
11:19:12  <Alberth> like one will remember it exists in a year :p
11:20:07  <frosch123> Eddi|zuHause: just do not delete checkouts
11:20:19  <frosch123> i have checkouts of all branches, with various modifies in them
11:20:56  <Alberth> even that fails as libraries you dynamically link against break
11:21:05  <Eddi|zuHause> frosch123: sure, but i tried looking through those, but i can't distinguish between compile fixes and local uncommited changes
11:21:23  <Alberth> mq patches?
11:21:27  <Eddi|zuHause> most of them are svn checkouts
11:23:06  <Alberth> make hg repo with unmodified svn checkout version, then put that under your current checkout
11:23:07  <andythenorth> 34 #defines left to delete in FIRS
11:23:31  <Alberth> you're not interested to switch to other revisions anyway
11:24:36  * LordAro has a git repo with various remotes
11:25:56  *** maciozo has quit IRC
11:26:12  <Alberth> try having a set of checked out versions with uncommitted random changes in them :)
11:26:43  <LordAro> eh, that's just a load of different branches with "tmp" commits :p
11:27:25  <Alberth> I have "random_stuff" commits :)
11:27:46  <Eddi|zuHause> i have lots of tmp.diff files :p
11:28:22  <Alberth> :)
11:39:34  *** Samu has joined #openttd
11:39:44  <Eddi|zuHause> uhm...
11:39:46  <Samu> test
11:39:47  <Eddi|zuHause> In file included from /home/johannes/spiele/OpenTTD/chipp/src/window.cpp:20:0:
11:39:49  <Eddi|zuHause> /home/johannes/spiele/OpenTTD/chipp/src/blitter/factory.hpp: In destructor ‘virtual BlitterFactoryBase::~BlitterFactoryBase()’:
11:39:50  <Eddi|zuHause> /home/johannes/spiele/OpenTTD/chipp/src/blitter/factory.hpp:73:22: warning: cast from type ‘const char*’ to type ‘void*’ casts away qualifiers [-Wcast-qual]
11:39:52  <Eddi|zuHause>    free((void *)this->name);
11:40:21  <frosch123> it's a warning
11:40:41  <Eddi|zuHause> "a warning is something that you can safely ignore" :pü
11:41:00  *** Montana_ has joined #openttd
11:41:20  <Samu> isn't working, testing
11:42:23  <Eddi|zuHause> but much funnier are these:
11:42:26  <Eddi|zuHause> In file included from /home/johannes/spiele/OpenTTD/chipp/src/openttd.cpp:69:0:
11:42:27  <Eddi|zuHause> /home/johannes/spiele/OpenTTD/chipp/src/town.h: In member function ‘uint16 Town::GrowthRatePercent() const’:
11:42:29  <Eddi|zuHause> /home/johannes/spiele/OpenTTD/chipp/src/town.h:242:40: warning: statement has no effect [-Wunused-value]
11:42:30  <Eddi|zuHause>    if (larger_town) default_growth_rate / 2;
11:42:56  <Eddi|zuHause> i've not looked at that file yet to determine what was acutally meant :p
11:43:19  <frosch123> "/=" instead of "/" probably
11:43:23  <Eddi|zuHause> but looks like it actually built
11:43:45  <Eddi|zuHause> yeah, most likely
11:44:58  <frosch123> anyway, if you make functional changes to old revisions to fix warnings, there is no purpose of bisecting
11:45:16  <frosch123> bisecting is about comparing behaviour, which is pointless if you patch the behaviour
11:45:37  <Eddi|zuHause> i was not trying to bisect this :p
11:46:18  <Eddi|zuHause> now if you would tell me which game i played with chipp...
11:46:28  <frosch123> how about you update the release history on the wiki? i think lordaro's younger sibling is not going to do it
11:47:19  <peter1138> ./openttd: error while loading shared libraries: cannot open shared object file: No such file or directory
11:47:24  <peter1138> of course, even old binaries may not run :p
11:47:37  <Eddi|zuHause> ah, i think i found it
11:48:11  <Eddi|zuHause> peter1138: you didn't reconfigure properly?
11:48:24  <peter1138> ?
11:48:40  <Eddi|zuHause> ah, no. right.
11:48:56  <Eddi|zuHause> you downloaded an old binary, but not its dependencies
11:49:11  <peter1138> should've made static binaries...
11:50:03  <Eddi|zuHause> peter1138: LordAro's link seems to work for compiling r2xxxx binaries
11:50:28  <LordAro> frosch123: still got no replacement, have i? :p
11:50:56  <frosch123> LordAro: should i file a feature request for aspell?
11:52:34  <LordAro> haha
12:03:15  *** FLHerne has joined #openttd
12:03:36  <peter1138> hmm, the void area is broken with 8bpp-simple
12:09:42  <andythenorth> ooh an NRT bug report
12:09:49  <andythenorth>
12:11:07  <frosch123> andythenorth: improve error messages
12:11:18  <frosch123> it means "too many roadtypes defined"
12:11:34  <andythenorth> @seen wolf01
12:11:34  <DorpsGek> andythenorth: wolf01 was last seen in #openttd 2 days, 12 hours, 8 minutes, and 28 seconds ago: <Wolf01> 'night
12:12:03  <andythenorth> I should add parameter to disable Road Hog types
12:12:47  <frosch123> andythenorth: wolf is at some lego fair till tomorrow
12:12:52  <andythenorth> ha
12:23:34  <FLHerne> No roadtypes for 1.7.0? Aw
12:26:20  <peter1138> what was the debug key to get the blitter to highlight drawn rects?
12:26:29  <Alberth> ^B
12:26:39  <Alberth> or just b
12:26:58  <frosch123> it's in the ? menu
12:27:07  <frosch123> but you need to enable debug tools or something
12:32:49  <Eddi|zuHause> ^B is bounding boxes
12:32:53  <andythenorth> FLHerne: it’s in an official binary, which is kept more or less in sync with trunk
12:33:17  <andythenorth> I suppose that’s not so useful for servers
12:33:45  <FLHerne> Sure, it just seems a shame to miss it when 1.7.0 has so few other interesting features
12:34:35  <Eddi|zuHause> FLHerne: but at the same time, it's too untested/unfinished to put it into a release
12:34:56  <FLHerne> If you don't want to freeze the grf API yet, does 1.7.0 *have* to be April 1st?
12:35:19  <FLHerne> Well, now would be a bad time to complain
12:35:42  <Alberth> if source code is broken, there is no point to put it in stable
12:36:02  <andythenorth> I am pretty happy with the situation where the fork is officially built as binaries
12:36:03  <Eddi|zuHause> FLHerne: no, but you would have to delay 1.7 for like 3 months to stabilize it
12:36:16  <Alberth> ie NRT would be in stable, and still not usasble
12:36:36  <FLHerne> Eddi|zuHause: Ok, I got the impression it was slightly closer than that
12:36:49  <andythenorth> I rthink it’s ~done
12:37:00  <andythenorth> but running a fork let’s people test it, especially the newgrf sped
12:37:02  <andythenorth> spec *
12:37:03  <Alberth> for some value of ~  :)
12:37:17  <andythenorth> adjusting a spec after it hits trunk…..too much whining :)
12:37:47  <frosch123> i thought we would have to start over :p
12:37:53  <frosch123> 16 roadtypes is not enough
12:37:59  <Alberth> :D
12:38:15  <Eddi|zuHause> frosch123: no amount is ever enough
12:38:22  <andythenorth> bad frosch123 :P
12:44:07  * andythenorth -> shops, bbl
12:44:10  *** andythenorth has quit IRC
12:45:33  <Eddi|zuHause> you have shops on sundays?
12:46:13  <Alberth> they have
12:46:15  <__ln__> we do, too
12:47:31  *** andythenorth has joined #openttd
12:49:11  <__ln__> does it vary by bundesland, or is virtually everything closed on sundays all over the country?
12:49:38  <frosch123> by default it is everywhere
12:49:54  <frosch123> every town can allow exceptions every now and then
12:50:06  <frosch123> but not all the time
12:50:38  <frosch123> there are "sunday open" events sometimes
12:51:12  <frosch123> opening times during the week vary between federal states though
12:51:25  <Eddi|zuHause> it used to be a lot stricter
12:51:28  <frosch123> bavaria is most strict, with closing time at 20:00
12:52:01  <__ln__> how was it in the east?
12:52:23  <Eddi|zuHause> there was not much difference between west and east, as far as i can tell
12:52:59  <frosch123>
12:54:01  <Eddi|zuHause> __ln__: the rules for shop openings started to get relaxed in the mid-90s, after unification
12:54:32  <__ln__> i see
12:55:22  <__ln__> for east the fundamental reason for closed-on-sundays was not religious, or was it?
12:55:35  <Eddi|zuHause> i have no clue
12:55:36  *** andythenorth has quit IRC
12:56:05  <Eddi|zuHause> it's not like religion didn't exist
12:56:38  <frosch123> i would think the origins are rather in labor unions than in religion
12:57:39  <Eddi|zuHause> it was probably originally a church thing, that was then picked up by labour unions
12:58:33  <frosch123> <- apparently the current law is from 1919
13:00:45  <Eddi|zuHause> that was a pretty labour-union-friendly time
13:00:51  <frosch123> "In den Niederlanden verbietet ein Gesetz von 1954 das Fußballspielen vor 13 Uhr." <- sounds over-specific
13:02:16  <__ln__> over here, until about 2010, only very small (kiosk-size) shops were permitted to be open beyond {7-21, sat 7-18, (sometimes sun 9-18)}. that is, even during the christmas season all the shops needed to close at 18 on saturdays.
13:02:32  <__ln__> no exceptions to that
13:03:34  <__ln__> the rules were since relaxed a bit, and since 2016 all shops of any size are permitted to be open 24/7 on any day of the year.
13:04:36  *** Montana_ has quit IRC
13:05:03  <__ln__> there are very very few 24h shops though in practice, most close either at 21, 22 or 23.
13:06:21  <__ln__> but basically every grocery type shop is open on each sunday.
13:07:47  *** sim-al2 has joined #openttd
13:15:19  *** Gja has joined #openttd
13:32:31  <peter1138> hmm... if i use a non-compositing window manager, the cursor is fine
13:33:31  <frosch123> does it have an effect if you disable the multithreaded drawing?
13:33:45  <frosch123> -v sdl:no_threads
13:33:59  <peter1138> nope
13:34:35  <peter1138> still disappears
13:39:27  *** supermop has joined #openttd
13:47:11  *** iSoSyS has joined #openttd
14:01:12  <peter1138> feels like sdl_updaterects is doing the first rect, then waiting for vsync, and then doing the rese.t
14:01:15  <peter1138> *rest
14:07:22  *** FLHerne has quit IRC
14:13:45  *** FLHerne has joined #openttd
14:14:42  *** andythenorth has joined #openttd
14:15:40  <Alberth> o/ andy
14:16:55  <Alberth> I mostly finished the Makefile of iron horse, "bundle_src" still fails, due to the spaces in some of your source files. It's a feature of 'make', which will not be fixed
14:18:02  <Alberth> freaking backward compatibility reasons :(
14:19:08  <peter1138> yeah, also works ok in fullscreen mode.
14:19:09  <Alberth> The Makefile_old* files should be deleted, and spaces in filenames killed
14:19:12  <peter1138> sigh, progress.
14:19:48  <Alberth> but some names seem generated
14:22:53  <peter1138> mind you, that doesn't explain why it's the same in windows
14:22:59  <peter1138> although maybe it's similar
14:23:09  <peter1138> who needs 2d graphics anyway...
14:23:49  *** sim-al2 has quit IRC
14:29:06  <andythenorth> Alberth: is bundle_src a required target? o_O
14:29:09  <andythenorth> or just legacy?
14:29:24  <Alberth> required
14:29:49  <Alberth> gpl requirements etc :)
14:30:04  <Alberth> just pushed removing the old makefiles
14:30:06  * andythenorth runs it
14:30:16  <andythenorth> any spaces can likely be removed
14:30:18  <Alberth> have a look at jenkins output
14:31:02  <andythenorth> oh it ships the psds? :o
14:31:04  <andythenorth> interesting
14:31:16  <Alberth> 'source' eh :)
14:31:41  <Alberth> it mirrors all repo files
14:31:59  <Alberth> which makes sense as source tarball, I think
14:32:13  * andythenorth remembers the argument about whether files for rendered sprites constitute source
14:32:55  *** FLHerne_ has joined #openttd
14:33:02  <Eddi|zuHause> "source" is anything you would make manual changes to
14:33:03  <andythenorth> a lot of psds isn’t meaningfully source, it’s more like sketching
14:33:08  <Alberth> you distribute them anyway in the hg clone
14:33:10  <andythenorth> I could move some stuff out of it
14:33:48  <andythenorth> GPL requirement is met by the repo
14:33:58  <Alberth> updated to h1548
14:33:59  <Eddi|zuHause> you can leave drafts out if they don't end up being processed for the final program
14:34:05  <Alberth> ?
14:34:08  <andythenorth> seems bundling non-usable, non-useful source files is over-stepping the principle
14:34:22  <andythenorth> it fails some kind of convenience rule I think
14:34:41  *** FLHerne has quit IRC
14:34:46  <Eddi|zuHause> in general, if the files are in the repo, there's no reason to leave them out of the bundle
14:34:55  <andythenorth> who consumes bundles?
14:34:59  * andythenorth is curious
14:35:13  <andythenorth> I can just delete the space, ‘solving’ the problme
14:35:17  <Eddi|zuHause> who knows?
14:35:23  <andythenorth> it’s only 2.8Mb
14:35:23  <Alberth> the GarryGs of this world
14:35:40  <andythenorth> there’s no particular argument against distributing the psds then
14:35:42  <Eddi|zuHause> just replace space by _
14:36:49  <andythenorth> as long as there’s no expectation of a reliable psd -> png -> grf workflow :)
14:37:00  <andythenorth> the psds aren’t very usable unless you’re me
14:37:23  *** FLHerne_ has quit IRC
14:37:24  <Alberth> don't think the gpl goes that far :p
14:37:29  <andythenorth> lots of the files aren’t even psds :P
14:37:31  <andythenorth> nvm
14:37:37  <Alberth> haha :p
14:37:54  <andythenorth> the dir was called ‘graphics_sources’, but I kept conflating that with ‘src’ and ‘graphics’ dirs
14:37:55  *** FLHerne_ has joined #openttd
14:38:11  <andythenorth> also they’re not reliable sources :)
14:38:43  <Alberth> you need a "various_other_junk" directory  :)
14:39:34  * andythenorth has replaced the spaces and is compiling
14:40:26  <Alberth> you likely do want to verify contents of the generated files, in particular the .tar files
14:41:18  <Alberth> ie is everything that you want in it?
14:41:20  <andythenorth> would the hash match?
14:41:52  <Alberth> no, I added a unique directory prefix in the tar, and likely the order of files has chnaged too
14:41:54  <andythenorth> ok
14:42:02  <andythenorth> bundle_src failed
14:42:03  <andythenorth> grfid -m iron-horse.grf > iron-horse.check.md5
14:42:16  <andythenorth> : /bin/sh: grfid: command not found
14:42:30  <Alberth> yeah, that would fail :[
14:42:37  <Alberth> s/:[/:p/
14:43:41  <Alberth> previous version added a file with the grfid output, and a Makefile.dist with HG information of the build
14:44:12  <Alberth> grfid output is still generated, the HG information gets inserted into the bundle makefile, so you can rebuild the grf
14:44:34  <Eddi|zuHause> so, what if i use the md5-collision-attack to make two grfs with different content but same grfid?
14:44:57  <Alberth> you can keep both grfs
14:44:59  <Samu> can u fix some bugs, now that you're approaching 1.7.1?
14:45:31  <Eddi|zuHause> i mean different content that creates the same md5sum
14:45:58  <Alberth> I know Eddi, you can keep both, and be happy with them, or so
14:46:47  <Alberth> not sure why you'd want to have same but not identical grfs
14:47:58  <Alberth> the files provides the means to check your build is correct, nothing else
14:48:02  <Samu> FS#6508 - Draw buoy when placed
14:48:20  <peter1138> Then you can have a desyncing network game D:
14:48:22  <Samu> FS#6506 - Rail isn't converted if there is a ship in the tile
14:48:52  * andythenorth can’t remember what’s distributed in the tar
14:48:57  <Samu> FS#6505 - Allow rail when ships are on lower halftile
14:49:31  <andythenorth> Alberth: everything seems to be correct in the tar, just grf, license, readmet
14:49:33  <andythenorth> readme *
14:49:49  <Alberth> \o/
14:49:59  <andythenorth> actually, it’s changelog, not readme :P
14:50:02  <peter1138> .
14:50:07  * andythenorth should maybe improve that, but later
14:50:18  <Samu> FS#6479 - Dropdown Setting is changing value of non-visible Parameter
14:51:17  <Samu> FS#6460 - Can't access Random AI Parameters in a game
14:51:38  <Alberth> bin scripts don't really belong in that repo, I don't have a better place yet, for them
14:51:47  <andythenorth> they do no harm
14:52:00  <andythenorth> I don’t really want remotes and so on
14:52:08  <andythenorth> we could give them a repo, and vendor them in
14:52:09  <Samu> FS#6366 - Windows 10 main viewport and mouse issues in fullscreen mode
14:52:19  <andythenorth> vendor = copy-paste :P
14:52:20  <Samu> these bugs are relatively easy fixes, plz take a look
14:52:34  <Alberth> it's fine that you have your own copy, but it makes using them elsewhere tricky
14:52:46  <Alberth> or updating them, for that matter
14:52:47  <andythenorth> devzone repo?
14:53:17  <Alberth> sounds like the simplest solution, doesn't it?
14:53:22  <andythenorth> yup
14:53:28  <andythenorth> got a name? o_O
14:54:04  <Alberth> alternatively, they could move into the general project thingie
14:54:11  <andythenorth> they could
14:56:08  * andythenorth wondered if src/ could be eliminated
14:56:19  <andythenorth> and the makefile handle each section
14:56:27  <andythenorth> it’s a bit pseudo-make-in-python
14:56:49  <Alberth> tricking non-suspecting devs :p
14:56:55  <andythenorth> dates from me not understanding how to call the python scripts correctly from make
15:13:35  <Samu> will you fix these easy to fix bugs for 1.7.0 release?
15:14:47  <Alberth> highly unlikely
15:15:48  <Samu> they're really easy fixable :(
15:17:52  <peter1138> moo
15:18:15  <LordAro> Samu: how do you know they're really easily fixable?
15:18:19  <LordAro> have you fixed them?
15:18:36  <Samu> because, yeah exactly
15:19:14  <LordAro> did you put the patches in the relevant issue?
15:19:24  <Samu> i provided them, yes
15:21:01  <LordAro> do the devs know that the patches are there?
15:21:17  <peter1138> your fix for 5606 breaks when upgrading a rail tile with a road vehicle on it
15:21:48  <Samu> oh no
15:22:28  <peter1138> (or rather, with a level crossing on it)
15:22:41  <peter1138> happened to be occupied :p
15:24:43  <Samu> got to test, brb
15:24:49  <Samu> see what happens
15:26:39  <DorpsGek> Commit by peter1138 :: r27784 trunk/src/rail_cmd.cpp (2017-03-12 16:26:31 +0100 )
15:26:40  <DorpsGek> -Fix [FS#6505]: Allow rail conversion even if ship is on tile (Samu).
15:27:22  <peter1138> Date:   Tue Nov 11 11:50:04 2014 +0000
15:27:28  <peter1138> That's quite a while.
15:31:27  <LordAro> oh hey, 6507 is mine
15:32:46  <DorpsGek> Commit by peter1138 :: r27785 trunk/src/waypoint_cmd.cpp (2017-03-12 16:32:40 +0100 )
15:32:47  <DorpsGek> -Fix [FS#6508]: Buoys not always drawn after being placed depending on zoom level (Samu).
15:34:52  <LordAro> peter1138: that your last commit date?
15:35:16  <peter1138> Yeah it was
15:35:29  <LordAro> nice
15:36:45  <LordAro> Date:   Tue Feb 25 18:25:04 2014 +0000
15:36:53  <LordAro> was mine until 6 months ago :>
15:37:04  <LordAro> although, my commits don't count quite as much as yours :p
15:43:55  <peter1138> Eh, mine stopped counting long ago.
15:43:55  <Samu> oh, an assertion
15:44:01  <Samu> nice eye peter1138
15:44:04  <peter1138> Samu, already sorted it :p
15:47:28  <peter1138> I actually think I stopped playing openttd after I stopped using CRT monitors.
15:47:59  <peter1138> Before that I used to play it in 800x600 on a 17" screen. That was a nice size, with lovely colours and crisp pixels.
15:48:25  <peter1138> I kinda wish I hadn't binned my CRTs, but really they were never going to get used again.
15:50:49  <andythenorth> stuff moves on eh?
15:50:59  * andythenorth had a CRT telly until last year
15:51:12  <andythenorth> it had really nice non-square pixels, looks better
15:54:22  <Eddi|zuHause> i still have a CRT TV here, but it can't receive any programs because they shut down analogue TV and there's no point in investing in a digital receiver
15:54:47  <andythenorth> you can watch VHS cassettes :)
15:54:58  <Eddi|zuHause> yeah. but nobody does that :p
15:57:55  <peter1138> Back then I was using SDL's DGA driver as well. It was super efficient at 8bpp palette animation.
15:59:22  *** gelignite has quit IRC
15:59:45  <peter1138> Hmm, I pressed ^S to save a screenshot, and it said it did, but I can't find it.
16:00:46  <peter1138> Oh. .local/share...
16:00:54  <Samu> peter1138: - plz do this but for the compiler farm thing
16:01:10  <Samu> dunno which vs it uses
16:02:38  <peter1138> What? smiley face?
16:03:03  <Samu> crap
16:03:13  <Samu>
16:03:18  <Samu> wrong copy
16:03:49  <Samu> i did it for vs140, but compiler farm uses another version :(
16:03:58  <peter1138> I...
16:04:08  <peter1138> Neither do I. And I'm on Linux, so...
16:06:30  <Samu> t.t problem has been since 2015 :(
16:09:39  <LordAro> needs a windows person
16:09:49  <LordAro> is that glx or michi?
16:14:14  *** Samu has quit IRC
16:18:52  *** Samu has joined #openttd
16:19:20  <Samu> having trouble staying connected
16:21:57  <LordAro> looks like you quit
16:22:00  <LordAro> from the message
16:22:08  <LordAro> (rather than being disconnected)
16:22:24  <andythenorth> IncompatibleGRF("SZD", "Oil well decrease neutralizer")
16:22:46  <andythenorth> the grfid is being rendered as "SZD"
16:23:03  <andythenorth> I thought python 3 solved unicode? o_O
16:23:34  <Alberth> that's unicode :)
16:23:47  <Alberth> you just don't have a glyph for 
16:24:20  <andythenorth> ach
16:25:17  <andythenorth> I just want it used literally
16:26:01  <andythenorth> how is unicode used in a grfid?
16:26:08  <andythenorth> I thought they were bytes :P
16:26:20  <Alberth> oh, right
16:26:38  <Alberth> not sure nml understands unicode at all
16:26:47  <andythenorth> this is the danger of refactoring code I don’t understand :)
16:26:57  <andythenorth> I have literals in CPP defines
16:27:04  <andythenorth> and I want the same result via python
16:27:06  <andythenorth> but eh :)
16:27:14  <Alberth> escape the \
16:27:27  <michi_cc> LordAro: It needs someone who still has Visual Studio 2010 installed. Syntax is likely unchanged to VS2015.
16:27:56  <andythenorth> Alberth: escape seems to work, horrible interface though
16:28:01  <michi_cc> Samu: You need to change the file and and run the generate script to update the real project files.
16:28:08  <andythenorth> what are these grfids, and where do I find them?
16:28:26  <Alberth> raw literal works too, probably  r"..."
16:28:53  <Alberth> you're fighting python string interpretation :)
16:28:59  <andythenorth> I never maintain the incompatible grf checks because the values used in FIRS don’t match the grfids here
16:29:39  <Alberth> bananas has a large subset at best
16:30:11  <andythenorth> e.g. Experts Hard Industries has grfid 45480101
16:30:21  <andythenorth> but FIRS uses "EH"
16:30:35  * andythenorth knows that the bytes can be translated :P
16:31:09  <andythenorth> but my hex calculator doesn’t have ‘H'
16:31:31  <Alberth> python knows about ord("H")
16:31:50  <Alberth> or find a random ascii table at the interwebs
16:31:51  <andythenorth> I am about to learn some more computer science, aren’t I :|
16:32:06  * andythenorth wonders why nml doesn’t just use the grfid, instead of fucking around :)
16:32:27  <andythenorth> this translation is a game for people smarter than me
16:32:35  <andythenorth> neither openttd nor bananas use it
16:32:43  <Alberth>
16:33:22  <Alberth> people want to write a string to get a number, for some weird reason
16:33:47  <DorpsGek> Commit by peter1138 :: r27786 trunk/src/ai/ai_gui.cpp (2017-03-12 17:33:40 +0100 )
16:33:48  <DorpsGek> -Fix [FS#6479]: AI configuration changed incorrect parameter when some parameters are hidden.
16:34:08  <peter1138> Samu, your patch for ^ was wrong thoughg
16:34:49  * andythenorth trying to find out if nml understands the grfids
16:35:24  <peter1138> Samu, your fixes are so easy they're often wrong ;)
16:37:39  <michi_cc> Somebody with VS2010 should test
16:39:28  <Samu> what was wrong with it t.t.
16:39:44  <Samu> i'll check it out
16:43:20  <Samu> i don't have VS2010 :(
16:43:20  <Samu> using 2015
16:45:49  <Eddi|zuHause> i always wondered: how does python's ord() function handle unicode characters?
16:46:04  <Eddi|zuHause> but never bothered to try it
16:46:08  <Samu> hum... i thought SCRIPTCONFIG_DEVELOPER were the hidden parameters
16:46:59  <andythenorth> hmm raw strings must have a type
16:47:18  * andythenorth might as well support grfids, and this literal string shenanigans
16:49:00  <Samu> ai_developer_tools
16:49:22  <Samu> well, your code is clearer
16:49:33  <Samu> mine was going straigth into the parameter
16:50:07  <andythenorth> ha stack overflow says no way to detect a raw string
16:50:52  <andythenorth> ‘r’ is just a cue to the interpreter
16:51:26  <Alberth> yes, it changes how escaping works, for that string
16:51:28  <Samu> ah I see the difference, yours iterates over the list of visible settings
16:51:54  <Samu> mine iterates over all settings then jumps past the invisible ones
16:52:10  <Samu> i guess mine wasn't exactly "wrong", just different
16:52:52  <Samu> peter1138: but thx
16:55:12  <peter1138> it was wrong
16:55:57  <Alberth> Eddi|zuHause: you get the codepoint, as I expected
16:56:01  <andythenorth> ach, no way to just declare a grfid then :)
16:56:07  * andythenorth bamboozled
16:56:32  <Alberth> can't just specify 4 bytes?
16:56:35  <peter1138> it would not work correctly if the first parameter was hidden, or if multiple parameters in a row were hidden, because it's not iterating on the list, it's just looping on the row counter.
16:56:40  <andythenorth> Alberth: nope
16:56:49  <andythenorth> nml needs it converted to a literal
16:57:01  <andythenorth> I was going to convert it to a literal conditionally as needed
16:57:08  <andythenorth> but there’s no condition I can check :P
16:57:19  <Eddi|zuHause> andythenorth: define a class GRFID, and __init__ takes a string?
16:57:44  <Eddi|zuHause> then __str__ could output the string, and __repr__ the 4 hex bytes?
16:57:58  <andythenorth> that’s what I hoped :P
16:58:05  * andythenorth pushing
16:58:40  *** glx has joined #openttd
16:58:40  *** ChanServ sets mode: +v glx
16:59:43  <andythenorth>
17:00:54  <peter1138> Samu, that's 1 out of 3 i've not had to modify :)
17:01:38  <Samu> keks :(
17:02:10  <Samu> AI first row is always the starting date
17:02:17  <Samu> always visibl
17:02:26  <andythenorth> Eddi|zuHause: ^^
17:02:33  <Samu> but i dunno about GS
17:02:48  <peter1138> well even so
17:03:05  <peter1138> maybe it's hidden :p
17:03:09  <peter1138> (who knows why!)
17:03:26  <peter1138> but anyway it's a gui operation there so makes sense to use the gui list that's already therel
17:04:04  <Samu> i didn't know how to use it
17:04:10  <peter1138> neither did i!
17:04:29  <peter1138> :D
17:05:14  <Eddi|zuHause> andythenorth: magic to convert string to numeric grfid
17:06:07  <andythenorth> Eddi|zuHause: can you reverse it? o_O
17:06:11  <andythenorth> nml demands the literal
17:06:16  *** gelignite has joined #openttd
17:06:16  <andythenorth> but all grfs in the wild use the grfid
17:06:49  <Eddi|zuHause> it's technically a bit wrong, because it's internally treated as little endian, but this code does big-endian-stuff
17:08:02  <Alberth> >>> "".join("{:02X}".format(ord(a)) for a in "EH")
17:08:33  <Eddi|zuHause> andythenorth: the reverse would use the chr function
17:09:04  <Alberth> except for the non-printable character
17:09:17  <Eddi|zuHause> python already handles that
17:09:32  <Eddi|zuHause> >>> chr(0x48)
17:09:33  <Eddi|zuHause> 'H'
17:09:34  <Eddi|zuHause> >>> chr(0x01)
17:09:36  <Eddi|zuHause> '\x01'
17:09:39  <Alberth> ever tried print  chr(240) to an ascii terminal?
17:10:09  <Eddi|zuHause> >>> chr(240)
17:10:10  <Eddi|zuHause> '\xf0'
17:10:25  <Eddi|zuHause> i don't remember what 240 is
17:10:27  <Alberth> sure you're not seeing the repr?
17:10:42  <Alberth> me neither
17:10:42  <Eddi|zuHause> oh, right
17:11:05  <Eddi|zuHause> just a thing to remember to use repr() in the code then
17:11:20  <Eddi|zuHause> just printing a ? here
17:11:23  <andythenorth> so…what do I need to? o_O
17:12:03  <andythenorth> but actually, I can’t think of any conditional that I could use, so it’s maybe moot?
17:12:31  <Eddi|zuHause> andythenorth: what problem are you trying to solve?
17:12:45  <andythenorth> I want to declare grfids as 4 bytes, not literals
17:12:52  <andythenorth> as that’s how they are encountered
17:12:57  <andythenorth> but FIRS declares them as literals
17:13:15  <andythenorth> and I don’t want to go hunting for all those grfs already declared
17:13:31  <Eddi|zuHause> so you want to input [0x45,0x48,0x01,0x01]?
17:13:42  <Alberth> "".join("\{:02X}".format(ord(a)) for a in "EH")
17:13:45  <Eddi|zuHause> or "45480101"?
17:13:49  <andythenorth> the latter Eddi|zuHause
17:14:24  <andythenorth> o/t nml’s use of literals for grfids is utterly stupid
17:14:34  <andythenorth> and perpetuates the cute convention that grfid is meaningful
17:14:40  <andythenorth> and we know what damage that incurs :P
17:14:47  <Eddi|zuHause> it's not that stupid, if you come at it from a C++ mindset
17:15:02  <andythenorth> “but my canadian grfs"
17:15:04  <Eddi|zuHause> because in C++, a 4-letter-string is the same as a 32bit number
17:15:43  <andythenorth> I understand that :)
17:15:51  <Eddi|zuHause> the problem with "45480101" that prevents me from immediately coming up with a neat solution is that you need to consume 2 characters per iteration
17:16:08  <Eddi|zuHause> so a simple iterator over the string fails
17:16:13  <andythenorth> well I can just split every 2 chars and rejoin
17:16:22  <andythenorth> but only if I can be sure the string requires that
17:17:05  <andythenorth> maybe I just have to give my class two parameters for grfid
17:17:12  <andythenorth> and the correct type must be used
17:18:28  <Alberth> I am quite surprised you can't give it 4 byte numbers, tbh
17:18:51  <andythenorth> nml chokes on that
17:19:13  <Eddi|zuHause> it's a bit of an oversight
17:19:16  <andythenorth> I’ll paste the error in a minute
17:20:05  <andythenorth> nmlc ERROR: "generated/firs.pnml", line 193: Expected a string literal of length 4
17:20:29  <Eddi|zuHause> it should be fairly easy to extend nml to accept a number
17:21:10  <Eddi|zuHause> anyway:
17:21:13  <andythenorth> incidentally, I can’t find anything in nml docs about generating a valid grfid
17:21:20  <andythenorth> it links to the nfo docs
17:21:27  <andythenorth> surprised more people don’t fail on this
17:21:43  <andythenorth> I always ask someone else to do it when I start a new grf
17:21:46  <Alberth> everybody copies it from a working grf or from the example
17:22:00  <Samu> peter1138: still working on my patches? :o
17:22:18  <andythenorth> lots of grfs with “AB"
17:22:32  <Eddi|zuHause> andythenorth: i leave splitting the string into 2-byte-pairs as a simple exercise for the reader
17:22:55  <andythenorth> what if it’s a literal of length 8 though? o_O
17:23:01  <andythenorth> that is what puzzles me
17:25:27  *** Snail has joined #openttd
17:25:34  <andythenorth> Snail: o/
17:25:47  <Snail> hey Andythenorth
17:26:42  <andythenorth> Snail: so can you compile on Sierra?
17:26:56  *** iSoSyS has quit IRC
17:28:20  <Eddi|zuHause> andythenorth: how could it be a literal of length 8?
17:28:38  <andythenorth> I don’t know, I’m not a good programmer :)
17:28:49  <Eddi|zuHause> it must be exactly 4
17:28:58  <Snail> anythenorth: yes
17:29:06  <Snail> thanks to the patch submitted on the forums time ago
17:29:17  <Eddi|zuHause> that's why nmlc guards against that
17:29:19  <andythenorth> Eddi|zuHause: so it’s totally safe to check for len > 4?
17:29:28  <andythenorth> nah can’t be
17:29:42  <andythenorth> FIRS is declared as a string of length 12
17:29:47  <Eddi|zuHause> andythenorth: what's that check meant to do?
17:30:10  <Eddi|zuHause> andythenorth: 12 characters on the screen doesn't make the string length 12
17:30:14  <andythenorth> I need to conditionally detect an existing literal string, or a string of bytes that needs converting to literal
17:30:22  <Samu> peter1138: here's all my bugs/requests/patches as of recently FS#6505 FS#6504 FS#6500 FS#6498 FS#6478 FS#6460 FS#6453 FS#6452 FS#6445 FS#6438, if you care to take a look
17:30:28  <Samu> thx, it's much appreciated
17:30:49  <andythenorth> hmm
17:31:15  <andythenorth> I’ll just use different parameters for literal or non-literal strings
17:31:16  <Eddi|zuHause> andythenorth: if in this context you already know it must be a grfid, then len(s)=4 means it's a literal, and len(s)=8 means it's a hex representation
17:31:37  <andythenorth> and if it’s len 12?
17:31:44  <Samu> and FS #6512, forgot that one
17:32:18  <Eddi|zuHause> andythenorth: len("\xx\xx\xx\xx")=4 (where x are valid digits)
17:32:23  <andythenorth> there are literals with length 8
17:32:31  * andythenorth checks that
17:33:16  <andythenorth> Eddi|zuHause: what’s the length of r"AL"
17:33:16  <andythenorth> ?
17:33:20  <Eddi|zuHause> 4
17:33:26  <andythenorth> python disagrees
17:33:34  <Eddi|zuHause> "" is one character
17:33:38  <andythenorth> len(r"AL”) returns 8
17:34:25  <Eddi|zuHause> andythenorth: why are you using r""?
17:34:50  <andythenorth> because it’s neater than escaping the slashes?
17:35:18  <Eddi|zuHause> andythenorth: but the slashes are meant to be treated as escapes
17:35:25  <andythenorth> but then nml barfs
17:35:46  <Eddi|zuHause> andythenorth: you need to feed repr(s) to nml
17:36:06  <Eddi|zuHause> s="AL"
17:36:10  <Eddi|zuHause> len(s)=4
17:36:39  <Eddi|zuHause> repr() puts the slashes back in
17:36:49  <andythenorth> so it doe
17:36:51  <andythenorth> does
17:37:08  <Alberth> allowed = set("0123456789ABCDEFabcdef") ;   all(c in allowed for c in s)
17:38:28  <LordAro> string.hexdigits ?
17:38:50  <Alberth> ^ better  :)   thanks
17:39:08  <andythenorth> using repr(), nmlc ERROR: "generated/firs.pnml", line 137: Expected a string literal of length 4
17:39:21  <andythenorth> it’s escaped (\)
17:39:29  <andythenorth> nml doesn’t like that
17:39:32  <Eddi|zuHause> andythenorth: you need to remove all the r""
17:39:36  <andythenorth> I did
17:40:05  <andythenorth> this is way too hard, I’m just going to subclass IncompatibleGRF
17:40:06  <Eddi|zuHause> then i don't know what you did
17:40:33  <andythenorth> passed the literal, called repr on it
17:41:03  <andythenorth> python has to have the escapes, but nml does not want them
17:41:07  <Alberth> python literal representation != nml literal representation
17:41:20  <andythenorth> this is silly :)
17:41:28  <andythenorth> grfid should have been the grfid, and not a game
17:42:18  <Eddi|zuHause> andythenorth: really, nmlc should be taught to handle numeric IDs
17:42:27  <andythenorth> good luck :)
17:42:46  <Eddi|zuHause> it's probably 2 lines :p
17:43:00  <Eddi|zuHause> one of them checks whether the ID is in range :p
17:43:59  <andythenorth> let’s see
17:47:14  <andythenorth> maybe parse_string_to_dword()
17:47:17  * andythenorth digging
17:48:08  <andythenorth>
17:50:31  <Eddi|zuHause> uhm, yes. you just need to teach all places that expect a grfid to skip that function if it encounters a number
17:51:23  <andythenorth> there are about 800
17:51:54  <Eddi|zuHause> alternatively you check for int in that function and just return that value
17:52:06  <Eddi|zuHause> before that string check there
17:53:40  <Eddi|zuHause> something like isinstance(var, Integral) or so
17:56:05  *** Samu has quit IRC
17:57:07  <andythenorth> I won’t patch nml in ways I don’t understand :)
17:57:10  <andythenorth> so eh
18:19:09  <DorpsGek> Commit by peter1138 :: r27787 trunk/src/ai/ai_gui.cpp (2017-03-12 19:19:01 +0100 )
18:19:10  <DorpsGek> -Fix: Update scrollbar when rebuilding AI/GS setting list.
18:35:22  <peter1138> Now those linger dropdown lists are bugging me...
18:35:26  <peter1138> *lingering
18:37:08  <frosch123> everything you fix for ai/gs settings does likely also apply to newgrf settings :)
18:37:53  <frosch123> newgrf parameters are mostly the same code as ai/gs parameters
18:42:12  <peter1138> dropdown lists aren't closed by clicking elsewhere
18:42:18  <peter1138> so it's possible they have no parent!
18:42:26  <peter1138> and then it'll carash
18:42:29  <peter1138> and also crash
18:43:13  <peter1138> yeah, my weird disappearing mouse cursor also happens to complete windows when i drag them around...
18:43:33  <peter1138> it's like it's waiting for vsync, but doing it at the wrong moment
18:51:29  *** iSoSyS has joined #openttd
19:02:02  <peter1138> jesus the window goes mad if i try to resize it
19:07:01  <andythenorth> smear?
19:07:15  <andythenorth> technicolour-vomit smear?
19:10:09  <andythenorth> GRFIDDword is a horrible classname
19:10:31  *** Stimrol has joined #openttd
19:10:42  <andythenorth> PEP 8 requires ClassName convention
19:11:09  <peter1138> no, resize spaz
19:11:47  <peter1138> trying windowmaker... it goes spazzy with that too, but in a totally different way
19:16:05  *** bwn has quit IRC
19:19:35  <Eddi|zuHause> andythenorth: the class would be GrfID, and depending on the context you'd call GrfID.hex() or GrfID.literal()
19:20:00  <Eddi|zuHause> or something like that
19:21:52  <andythenorth> neat
19:31:13  *** Alberth has left #openttd
19:35:52  <peter1138> Cool, now the game is intermittently crashing inside pulseaudio when I resize...
19:35:55  <peter1138> Pulseaudio.
19:35:58  <peter1138> Pulse*Audio*.
19:40:30  *** iSoSyS has quit IRC
19:45:53  <andythenorth> Eddi|zuHause: this was python 3?
19:46:19  <Eddi|zuHause> no, but i don't see a reason why it shouldn't work in 3
19:47:36  *** bwn has joined #openttd
19:55:43  <andythenorth> in python 3 interpreter, print(chr(int(’01’,16))) returns \x01
19:56:14  <andythenorth> ach no
19:56:25  * andythenorth sees the problem
19:59:05  <Eddi|zuHause> again, repr()
19:59:08  <andythenorth> yup
19:59:14  <andythenorth> compiling now
20:03:23  <andythenorth> ah
20:03:27  <andythenorth>     if (grf_future_status('EH\x01\x01')) {
20:03:41  *** sla_ro|master2 has joined #openttd
20:03:53  <andythenorth> ‘ is not legal apparently, must be “
20:04:02  <andythenorth> stupid smart quote irc client :P
20:04:06  * andythenorth fixes
20:06:17  <andythenorth> nml barfs on the \x01 as well
20:07:46  *** sla_ro|master has quit IRC
20:10:40  <andythenorth> Eddi|zuHause: I gave up trying to make it nice :P
20:10:41  <andythenorth>
20:10:44  <andythenorth> it apparently works
20:13:31  <Eddi|zuHause> actually, there might be a quicker and dirtier workaround, you can just turn the "45480101" string into ""
20:16:45  <andythenorth> I wondered
20:16:52  <andythenorth> but it seemed not proper
20:17:04  <andythenorth> but split and join would do it
20:17:12  <andythenorth> nml literal is not quite a literal, eh?
20:21:09  <peter1138> Using svn feels weird :s
20:22:00  *** sim-al2 has joined #openttd
20:25:52  <frosch123> hg/git are older than cvs was when svn started
20:29:20  <peter1138> all these open issues...
20:37:34  * andythenorth lost in music
20:38:52  <peter1138> sister sledge?
20:43:01  <andythenorth> yarp
20:43:34  <andythenorth> seemed appropriate
20:58:41  <andythenorth> supermop:
20:59:43  *** Supercheese has joined #openttd
21:06:31  *** frosch123 has quit IRC
21:08:13  *** eekee has joined #openttd
21:14:42  *** andythenorth has left #openttd
21:34:24  *** Montana_ has joined #openttd
21:39:05  *** fiatjaf has quit IRC
21:40:45  *** Stimrol has quit IRC
22:02:15  *** gelignite has quit IRC
22:24:00  *** Progman has quit IRC
22:41:52  *** JezK_ has joined #openttd
22:44:47  *** sla_ro|master2 has quit IRC
22:46:03  *** Montana_ has quit IRC
22:56:55  *** Gja has quit IRC
23:09:01  *** Hiddenfunstuff has quit IRC
23:24:24  *** Wormnest has quit IRC
23:25:28  *** HerzogDeXtEr1 has joined #openttd
23:31:17  *** HerzogDeXtEr has quit IRC
23:35:51  *** FLHerne_ has quit IRC

Powered by YARRSTE version: svn-trunk