| Saturday, July 27th, 2013, 00:18 UTC | ||
| [00:18:21] | jya: | still surprised output.cpp works still |
| [00:37:38] | SteveGoodey (SteveGoodey!~steve@host86-160-206-25.range86-160.btcentralplus.com) has joined #mythtv | |
| [00:39:02] | robink (robink!~quassel@unaffilated/robink) has quit (Ping timeout: 240 seconds) | |
| [00:57:42] | lux_2 (lux_2!~lux@p54ABE1CC.dip0.t-ipconnect.de) has quit (Ping timeout: 264 seconds) | |
| [01:28:44] | SteveGoodey (SteveGoodey!~steve@host86-160-206-25.range86-160.btcentralplus.com) has quit (Quit: Konversation terminated!) | |
| [01:31:24] | NightMonkey (NightMonkey!~NightrMon@pdpc/supporter/professional/nightmonkey) has quit (Quit: Body blow! Body blow!) | |
| [02:25:46] | peper03 (peper03!~peper03@mythtv/developer/peper03) has quit (Ping timeout: 276 seconds) | |
| [02:25:56] | peper03_ (peper03_!~peper03@mythtv/developer/peper03) has joined #mythtv | |
| [02:26:09] | peper03_ is now known as peper03 | |
| [02:29:27] | nyloc (nyloc!~quassel@pC19F5EB8.dip0.t-ipconnect.de) has joined #mythtv | |
| [02:33:41] | _nyloc_ (_nyloc_!~quassel@p4FE4CB59.dip0.t-ipconnect.de) has quit (Ping timeout: 248 seconds) | |
| [03:11:25] | stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has joined #mythtv | |
| [03:15:05] | robink (robink!~quassel@unaffilated/robink) has joined #mythtv | |
| [03:47:16] | fetzerch (fetzerch!~quassel@unaffiliated/fetzerch) has quit (Ping timeout: 264 seconds) | |
| [03:48:21] | fetzerch (fetzerch!~quassel@unaffiliated/fetzerch) has joined #mythtv | |
| [05:00:38] | robink (robink!~quassel@unaffilated/robink) has quit (Ping timeout: 240 seconds) | |
| [05:03:29] | robink (robink!~quassel@unaffilated/robink) has joined #mythtv | |
| [05:37:11] | dblain (dblain!~dblain@mythtv/developer/dblain) has quit () | |
| [05:40:21] | dblain (dblain!~dblain@mythtv/developer/dblain) has joined #mythtv | |
| [05:59:56] | joki (joki!~joki@p548628AA.dip0.t-ipconnect.de) has quit (Ping timeout: 256 seconds) | |
| [06:06:05] | joki (joki!~joki@p54864022.dip0.t-ipconnect.de) has joined #mythtv | |
| [06:19:51] | jya: | dblain: ok, i've removed the operator= header |
| [06:21:01] | dblain: | Did you find an issue? I thought it was common in this code base to do (haven't had time to reseach yet) |
| [06:21:47] | jya: | no I found no issue... |
| [06:22:14] | jya: | i know for a fact that no operator= is ever called on a mythnotification, so having it or not make no difference |
| [06:23:00] | jya: | and if someone want to copy one, they can use the copy constructor instead |
| [06:23:06] | dblain: | interesting. It showed a lot of unresolved link errors when it was present (10+ other methods supposedly used it). |
| [06:23:30] | dblain: | either way. Thanks. |
| [06:23:35] | jya: | I guess it depends on how some compiler compile MythNotification foo = MythNotification(args) |
| [06:24:05] | jya: | gcc/clang will make that MythNotification foo(args); |
| [06:24:29] | dblain: | I'm trying to setup a msvc build slave so issues like these can be fixed right away. |
| [06:24:53] | jya: | I had this issue in the past with VS; it complained that I had no operator= , so I had to change plenty of definition to the format ClassName blah(args); |
| [06:25:18] | jya: | too bad, because to me MyClass blah = MythClass(arg) is easier to read |
| [06:25:20] | dblain: | It was happy when I removed the header entry... that is the odd part. |
| [06:26:02] | dblain: | I believe it handles creating a default implementation as long you don't tell it to expect one of your own?! |
| [06:26:12] | jya: | dblain: you probably find that if you added void blah(void); you would get the error that blah isn't defined anywhere |
| [06:26:38] | dblain: | most likely. |
| [06:31:17] | len (len!~quassel@75-168-36-94.mpls.qwest.net) has quit (Remote host closed the connection) | |
| [06:33:22] | dblain: | jya: looking at you commit... strange thing is, I only had to remove the MythNotification operator... the others you removed were okay to keep in. (doesn't make sense to me unless the other were considered "not used" by the code). |
| [06:33:48] | jya: | oh well, makes no difference with or without… so better without |
| [06:56:36] | stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has quit (Ping timeout: 260 seconds) | |
| [06:59:44] | stichnot (stichnot!~stichnot@adsl-68-127-28-24.dsl.pltn13.pacbell.net) has joined #mythtv | |
| [06:59:45] | stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has joined #mythtv | |
| [06:59:45] | stichnot (stichnot!~stichnot@adsl-68-127-28-24.dsl.pltn13.pacbell.net) has quit (Changing host) | |
| [07:42:32] | len (len!~quassel@75-168-36-94.mpls.qwest.net) has joined #mythtv | |
| [07:56:58] | SteveGoodey (SteveGoodey!~steve@host86-160-206-25.range86-160.btcentralplus.com) has joined #mythtv | |
| [08:02:14] | peper03: | jya: I haven't looked at the assembler it's producing but I would have thought that comparing a UInt64 against ~0x0u would cause the literal to be 64-bit too. In any case, having the signed value in GetBookmark is silly when the method returns an unsigned value. I'll change that there to make it consistent. |
| [08:20:37] | Dorward (Dorward!~Dorward@70.75.112.87.dyn.plus.net) has joined #mythtv | |
| [08:57:09] | toeb (toeb!~toeb@HSI-KBW-109-193-196-029.hsi7.kabel-badenwuerttemberg.de) has quit (Read error: Operation timed out) | |
| [08:59:51] | toeb (toeb!~toeb@HSI-KBW-109-193-196-029.hsi7.kabel-badenwuerttemberg.de) has joined #mythtv | |
| [09:09:30] | len (len!~quassel@75-168-36-94.mpls.qwest.net) has quit (Read error: Connection reset by peer) | |
| [09:13:20] | dekarl: | jya, is http://pastebin.com/dxFGXUBA something that might trigger a bug? http://software.intel.com/en-us/articles/cdiag1125 |
| [09:24:49] | Dorward (Dorward!~Dorward@70.75.112.87.dyn.plus.net) has quit (Quit: rebooting) | |
| [09:26:11] | SteveGoodey (SteveGoodey!~steve@host86-160-206-25.range86-160.btcentralplus.com) has quit (Quit: Konversation terminated!) | |
| [09:32:17] | dgeary2 (dgeary2!~debian@49.176.98.199) has joined #mythtv | |
| [09:39:08] | Dorward (Dorward!~Dorward@70.75.112.87.dyn.plus.net) has joined #mythtv | |
| [09:40:49] | jya: | if you are comparing a 64 bits vs 32 bits, it's the 32 bits value that is extended to 64 bits.. Same when adding a 32 bits to 64 bits, the result is 64 bits |
| [09:52:02] | jya: | dekarl:I don't see how that warning could be applicable… The issue of a method hidden by another is when the argument of one can be cast into another, and as such one is never called. But here the functions are virtual, and take the exact same arguments… so there's no ambiguity as to which one will be called |
| [09:53:47] | jya: | how could a PopScreen(MythScreenType*, bool, bool) hide a PopScreen(bool, bool) it just can't there's never any ambiguity |
| [10:05:32] | jya: | unless it thinks that doing popScreen(0) could be an issue, is 0 NULL or false |
| [10:06:56] | jya: | but then there's an issue in the original header |
| [10:16:31] | dekarl: | jya, the builders also found http://pastebin.com/BVQrhfkF |
| [10:17:16] | amessina (amessina!~amessina@50-196-241-78-static.hfc.comcastbusiness.net) has quit (Quit: Konversation terminated!) | |
| [10:20:11] | jya: | thanks… that's one we can get rid of yes |
| [10:22:01] | jya: | dekarl: yeah, I'm now 100% sure that this warning is of no consequence. only if I was using PopScreen without dynamic dispatch would it be an issue. but the screen stack is always used via a pointer to it.. so a non issue |
| [10:22:09] | jya: | will remove the warning though |
| [10:27:13] | amessina (amessina!~amessina@2001:470:c1dc:7779:d6be:d9ff:fe8d:7c1e) has joined #mythtv | |
| [10:33:43] | dekarl: | I'm slowly working through the buildbots' logs. |
| [10:50:12] | IReboot (IReboot!~doug@cpe10bf48e67915-cm78cd8e7e342d.cpe.net.cable.rogers.com) has quit (Quit: Ex-Chat) | |
| [10:53:29] | dekarl: | maybe a simple "using void MythScreenStack::PopScreen(bool, bool);" in MythNotificationScreenStack is all it takes (that's my understanding from the intel link) |
| [10:53:34] | IReboot (IReboot!~doug@CPE10bf48e67915-CM78cd8e7e342d.cpe.net.cable.rogers.com) has joined #mythtv | |
| [10:55:44] | jya: | dekarl: there's another issue at hand regardless… when you do PopScreen(0,0) which functions are you calling? |
| [10:56:01] | jya: | but that issue has been there for a long time |
| [10:56:18] | jya: | but the issue in the intel link is different |
| [10:58:41] | amessina (amessina!~amessina@2001:470:c1dc:7779:d6be:d9ff:fe8d:7c1e) has quit (Quit: Konversation terminated!) | |
| [11:05:06] | dekarl: | yes, the issue with the ambiguous implicit casts is unrelated |
| [11:05:50] | dekarl: | I have adding more types on "the list". (e.g. to catch assignments of network_id to original_network_id and similar) |
| [11:07:16] | jya: | dekarl: well, I got to learn something today... |
| [11:07:50] | jya: | if I had done MythNotificationStack n; n.PopScreen(true, true); it wouldn't have compiled |
| [11:08:14] | jya: | but MythNotificationStack *n = new MythNotificationStack(); n->PopScreen(true, true) then that's okay |
| [11:08:24] | jya: | because the vtable isn't used in the first case |
| [11:17:38] | dekarl1 (dekarl1!~dekarl@p4FCEF084.dip0.t-ipconnect.de) has joined #mythtv | |
| [11:19:20] | dekarl (dekarl!~dekarl@p4FE857B3.dip0.t-ipconnect.de) has quit (Ping timeout: 260 seconds) | |
| [11:32:29] | peper03: | jya: I'm no expert on compiler intricacies but why would the literal '0' automatically be a 32-bit value? That said, I've never understood why you can get a signed/unsigned mismatch when comparing against zero. Or, for that matter, why a positive literal has to be explicitly marked as unsigned to avoid that warning. |
| [11:32:47] | jya: | because you put 0x0u |
| [11:32:53] | jya: | u is for unsigned int |
| [11:33:09] | peper03: | Ah, there you go then :) |
| [11:33:19] | jya: | your want ULL here |
| [11:35:38] | jya: | though here you're not comparing 0 |
| [11:35:45] | jya: | you're comparing ~0 |
| [11:36:33] | peper03: | I've always just thought of 'u' as meaning 'unsigned' without size. Although isn't an int generally 64-bit if compiling for a 64-bit system? (I know there are differences, but in general) |
| [11:36:35] | jya: | which for a int is -1, and a uint is 4294967296 |
| [11:36:57] | jya: | no, a int is a 32 bits integer on most platforms |
| [11:37:03] | jya: | certainly all the one myth runs on |
| [11:37:24] | jya: | for all the platform myth runs on, a int is 32 bits |
| [11:37:38] | jya: | a long is a 32 bits on all platforms but windows 64 bits, where it's 64 bits |
| [11:38:02] | jya: | that reminds me, I need to fix some bugs if we ever want to port myth to windows 64.. right now, it can't work |
| [11:38:07] | jya: | that's for dblain |
| [11:38:16] | peper03: | Ok. It's been a long time since I looked into the details behind this. So I'm certainly a bit rusty. |
| [11:44:03] | jya: | I'm aware of this because the project I just finished working on, turned out the code failed to compile on win64 (and that was code I took from myth) :) |
| [11:55:47] | jya: | hum…. changing the PopScreen I've broken things :( |
| [11:56:18] | jya: | looking at the warnings in the buildbot… funny that's exactly why i wanted to remove the ambiguity in the first place |
| [12:51:04] | danielk22 (danielk22!~danielk22@96.57.9.142) has left #mythtv () | |
| [13:21:15] | htpc_ (htpc_!~htpc@c-24-34-23-78.hsd1.ma.comcast.net) has left #mythtv ("Leaving") | |
| [13:43:20] | peper03 (peper03!~peper03@mythtv/developer/peper03) has quit (Quit: Quit) | |
| [13:48:03] | peper03 (peper03!~peper03@mythtv/developer/peper03) has joined #mythtv | |
| [14:18:16] | lux_2 (lux_2!~lux@p54ABE2BE.dip0.t-ipconnect.de) has joined #mythtv | |
| [14:18:26] | dgeary2_ (dgeary2_!~debian@pa49-187-93-5.pa.nsw.optusnet.com.au) has joined #mythtv | |
| [14:20:38] | dgeary2 (dgeary2!~debian@49.176.98.199) has quit (Ping timeout: 240 seconds) | |
| [14:20:45] | dgeary2_ is now known as dgeary | |
| [14:20:57] | dgeary is now known as dgeary2 | |
| [14:48:14] | jya: | jpabq: you may want to copy the notification-ui.xml for your Steppes theme. The default was originally based on your theme, so using it as-is should be fine… it's missing all the new goodies from it |
| [14:54:37] | stuartm: | iirc int is only 64 bits with ILP64 – which is so extremely obscure that it's not even worth a second thought |
| [14:55:23] | wagnerrp: | that some itanium architecture? |
| [14:55:49] | stuartm: | apparently HAL systems Solaris port to SPARC64 |
| [14:56:18] | stuartm: | jya: don't you have that backwards, long is 64bits on all platforms except windows 64 where it's 32bit? |
| [14:58:03] | stuartm: | fwiw, although I've phrased that as a question, it's more of a statement ... |
| [14:59:28] | stuartm: | Windows uses the LLP64 model, which probably due to some laziness on their part, is identical to the 32bit model except for pointers |
| [15:00:37] | wagnerrp: | so moral of the story is, any time we actually care about the size of our integers, we should be using [u]int64_t? |
| [15:02:03] | stuartm: | yes, pretty much |
| [15:03:38] | stuartm: | well that, and we can't assume long is 64bit if we wan't to remain cross-platform |
| [15:05:16] | stuartm: | but yeah, using int32_t and int64_t where size matters is good practice – it doesn't always mean using int64_t since there will be circumstances where that's just a waste of memory (long lists etc) |
| [15:16:20] | dekarl1 is now known as dekarl | |
| [16:25:46] | rsiebert (rsiebert!~quassel@g224250160.adsl.alicedsl.de) has joined #mythtv | |
| [16:29:28] | rsiebert_ (rsiebert_!~quassel@g229052162.adsl.alicedsl.de) has quit (Ping timeout: 276 seconds) | |
| [16:54:06] | dgeary2 (dgeary2!~debian@pa49-187-93-5.pa.nsw.optusnet.com.au) has quit (Ping timeout: 264 seconds) | |
| [16:56:43] | dgeary2 (dgeary2!~debian@pa49-187-93-5.pa.nsw.optusnet.com.au) has joined #mythtv | |
| [17:09:28] | SteveGoodey (SteveGoodey!~steve@host86-160-206-25.range86-160.btcentralplus.com) has joined #mythtv | |
| [17:21:08] | NightMonkey (NightMonkey!~NightrMon@pdpc/supporter/professional/nightmonkey) has joined #mythtv | |
| [18:05:11] | jr3us (jr3us!~jr3us@cpe-098-026-209-039.triad.res.rr.com) has joined #mythtv | |
| [18:05:26] | jr3us: | greetings all. |
| [18:05:46] | jr3us: | I have don't some digging regarding the default values for filters for mythweb |
| [18:05:52] | jr3us: | done not dont |
| [18:06:28] | jr3us: | the defaults come from the table recordfilter |
| [18:07:06] | jr3us: | doesn't look like the default template gets touched. unless recordfilter defines the template. |
| [18:11:16] | jr3us: | now looking to see what updates recordfilter |
| [18:30:03] | jr3us: | looks like recordfilter is only modified from dbcheck.cpp |
| [18:32:06] | jr3us: | so that means if i set the value for This channel to 1, that myth web will always default to this channel. |
| [18:34:31] | jr3us: | just verified that that sets 'This channel' default to ON |
| [18:38:22] | SteveGoodey (SteveGoodey!~steve@host86-160-206-25.range86-160.btcentralplus.com) has quit (Quit: Konversation terminated!) | |
| [18:40:40] | stichnot: | Initial documentation of Themed Menus at http://www.mythtv.org/wiki/Themed_Menus , comments/feedback welcome. |
| [19:04:30] | jr3us (jr3us!~jr3us@cpe-098-026-209-039.triad.res.rr.com) has quit (Quit: Leaving) | |
| [19:06:27] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has quit (Remote host closed the connection) | |
| [19:08:02] | jarle (jarle!~jarle@70.84-234-133.customer.lyse.net) has joined #mythtv | |
| [19:30:07] | dshay (dshay!~dshay@cpe-107-10-21-164.neo.res.rr.com) has joined #mythtv | |
| [19:31:25] | len (len!~quassel@75-161-179-129.mpls.qwest.net) has joined #mythtv | |
| [19:34:00] | knightr_ (knightr_!~knightr@69-165-170-178.dsl.teksavvy.com) has joined #mythtv | |
| [19:41:14] | gigem: | jr3us: I hope you read the logs. recordfilter includes an undocumented way to specify the default filter values. It predated templates and should no longer be used. The default values should now come from the appropriate. Also, be very careful with making the "This channel" filter default to on. Search rules will not work with that filter enabled. |
| [19:42:14] | Tobbe5178: | what happens if you have both xmltv & eit turned on at the same time? which one will be used first? |
| [20:08:21] | Steve-Goodey (Steve-Goodey!~steve@host86-160-206-25.range86-160.btcentralplus.com) has joined #mythtv | |
| [20:11:50] | dekarl: | Tobbe5178: its good old "last save wins". to make it more interesting we cache unchanged EIT events and only write them if they are updated |
| [20:12:09] | Tobbe5178: | hmm |
| [20:12:10] | Tobbe5178: | ok |
| [20:12:15] | dekarl: | you can mix and match EIT/XMLTV on a video source, but never on one channel |
| [20:12:41] | Tobbe5178: | the reason for the question is that tvzon xmltv graber have only 5 days of data and that is much smaller than usual |
| [20:13:07] | Tobbe5178: | and i'm going away so if no no new data shows up i will miss stuff |
| [20:13:52] | Tobbe5178: | so i guess i have to choose from bad data or risking of not getting any data at all |
| [20:13:57] | dekarl: | which channel? |
| [20:14:14] | dekarl: | oh |
| [20:14:31] | Tobbe5178: | all channels |
| [20:14:48] | Tobbe5178: | usualy the graber have lots of data |
| [20:14:52] | Tobbe5178: | like 14 or 20 days |
| [20:15:03] | Tobbe5178: | but now there is only this month and nothing else |
| [20:16:32] | Tobbe5178: | there also have been cases recently where some channels had missing data for a day or two |
| [20:16:37] | dekarl: | I just saw it, strange |
| [20:16:41] | Tobbe5178: | and sometimes the subtitle was set to TBA |
| [20:16:51] | Tobbe5178: | and it never changed |
| [20:17:59] | Tobbe5178: | i've tried mailing the dev i found but havent received any reply |
| [20:21:21] | Tobbe5178: | i guess i'll stick with the xmltv graber and hope it will work |
| [20:21:29] | Tobbe5178: | too much trouble now to switch to eit |
| [20:22:05] | Tobbe5178: | i guess maybe i should take a look at doing an eitfixup routine for my provider to cleanup the guide data |
| [20:22:32] | Tobbe5178: | at a minimum the short event descriptor needs to be put in the description and not subtitle |
| [20:22:52] | Tobbe5178: | (subtitle is first part of the description) |
| [20:22:57] | Tobbe5178: | in eit that is |
| [20:32:36] | knightr: | stichnot, we is the current default file so that I can check if it is translated? |
| [20:35:23] | peper03 (peper03!~peper03@mythtv/developer/peper03) has quit (Ping timeout: 240 seconds) | |
| [20:37:43] | peper03 (peper03!~peper03@mythtv/developer/peper03) has joined #mythtv | |
| [20:50:38] | lux_2 (lux_2!~lux@p54ABE2BE.dip0.t-ipconnect.de) has quit (Ping timeout: 240 seconds) | |
| [21:01:07] | dekarl: | Tobbe5178: I contacted the admin of your guide |
| [21:08:13] | stichnot: | knightr: The default files are themes/menu/menu_*.xml, but those files shouldn't need translation as all the displayable strings are already translated in tv_play.cpp |
| [21:09:11] | Steve-Goodey (Steve-Goodey!~steve@host86-160-206-25.range86-160.btcentralplus.com) has quit (Quit: Konversation terminated!) | |
| [21:27:40] | knightr: | stichnot, ah, thank you! now I remember... |
| [21:28:07] | stichnot: | knightr: sorry, themes/default/menu_*.xml |
| [21:28:43] | knightr: | stichnot, np, I figured the exact path with the info you gave me... |
| [21:29:25] | knightr: | (ie the wat the files are named...) |
| [21:32:02] | NightMonkey (NightMonkey!~NightrMon@pdpc/supporter/professional/nightmonkey) has quit (Quit: Body blow! Body blow!) | |
| [21:38:24] | knightr: | what distro currently ships with Qt 5? I need to check whether something I need to add works A-OK with both Qt 4 and 5 (actually I know it's deprecated in 5 but needed in 4...) |
| [21:38:36] | knightr: | s/what/which |
| [21:52:44] | NightMonkey (NightMonkey!~NightrMon@pdpc/supporter/professional/nightmonkey) has joined #mythtv | |
| [22:21:41] | NightMonkey (NightMonkey!~NightrMon@pdpc/supporter/professional/nightmonkey) has quit (Quit: Body blow! Body blow!) | |
| [22:52:15] | jpabq_ (jpabq_!~quassel@mythtv/developer/jpabq) has quit (Remote host closed the connection) | |
| [22:52:16] | jpabq (jpabq!~quassel@mythtv/developer/jpabq) has quit (Read error: Connection reset by peer) | |
| [23:02:37] | dekarl: | jya, here's another patch to silence a compiler warning from the build bots for review, http://pastebin.com/ePWsgGc1 |
| [23:33:52] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has quit (Ping timeout: 260 seconds) | |
| [23:36:31] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.