Tuesday, July 29th, 2014, 00:14 UTC | ||
[00:14:32] | SteveGoodey (SteveGoodey!~steve@host109-158-49-150.range109-158.btcentralplus.com) has joined #mythtv | |
[00:31:40] | stuartm: | jya: one of the unit tests fails here, is this a known issue? "QFATAL : TestCopyFrames::NV12copy(SSE) Received signal 4" |
[00:32:32] | andreaz (andreaz!~andre_000@p57922329.dip0.t-ipconnect.de) has quit (Read error: Connection reset by peer) | |
[00:45:48] | stuartm: | MythBuild: force build cppcheck-master Updated configs |
[00:45:49] | MythBuild: | build forced [ETA 1h53m29s] |
[00:45:49] | MythBuild: | I'll give a shout when the build finishes |
[01:07:56] | stuarta (stuarta!~stuarta@mythtv/developer/stuarta) has quit (Ping timeout: 240 seconds) | |
[01:10:13] | SteveGoodey (SteveGoodey!~steve@host109-158-49-150.range109-158.btcentralplus.com) has quit (Quit: Konversation terminated!) | |
[01:12:06] | skd5aner (skd5aner!~skd5aner@125.sub-70-198-65.myvzw.com) has quit (Ping timeout: 250 seconds) | |
[01:21:10] | stuarta (stuarta!~stuarta@mythtv/developer/stuarta) has joined #mythtv | |
[01:35:47] | skd5aner (skd5aner!~skd5aner@158.sub-70-198-70.myvzw.com) has joined #mythtv | |
[01:48:46] | jya: | stuartm: don’t know why it would fail… that’s the whole point of the unit test :) |
[01:48:55] | jya: | SSE crashing is typically memory that isn’t aligned |
[01:50:24] | jya: | stuartm: if you could provide the backtrace … |
[01:57:53] | gigem: | paul-h: There was a bug that existed for a few days that caused early expiry, but I think it was only for live TV recordings. Definitely find out what versions he's running and if it's 0.27, make sure he is running the current or very near current version. Also, even though he has 13T of disk space free, if his storage groups are poorly configured, it could still cause quick expiry. An example would be |
[01:57:55] | gigem: | having the Default group, or which ever one his recordings are configured to use, pointing to a full disk while ignoring the ones with lots of free space. |
[02:48:01] | gigem (gigem!~david@mythtv/developer/gigem) has quit (Quit: WeeChat 0.4.3) | |
[02:48:25] | MythBuild: | Hey! build cppcheck-master #3158 is complete: Success [3build successful] |
[02:48:25] | ** MythLogBot http://code.mythtv.org/trac/ticket/3158 ** | |
[02:48:25] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . /builds/3158 |
[02:49:40] | gigem (gigem!~david@mythtv/developer/gigem) has joined #mythtv | |
[02:53:56] | peper03 (peper03!~peper03@mythtv/developer/peper03) has quit (Ping timeout: 260 seconds) | |
[02:54:50] | peper03 (peper03!~peper03@mythtv/developer/peper03) has joined #mythtv | |
[03:08:31] | stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has joined #mythtv | |
[03:33:12] | CrypticSquared (CrypticSquared!~CrypticSq@unaffiliated/crypticsquared) has joined #mythtv | |
[03:38:14] | dreamcat4 (dreamcat4!~dreamcat4@62.49.10.153) has quit (Quit: Lost terminal) | |
[03:54:01] | wahrhaft (wahrhaft!~quassel@cpe-76-181-12-141.columbus.res.rr.com) has quit (Ping timeout: 272 seconds) | |
[03:55:15] | wahrhaft (wahrhaft!~quassel@cpe-76-181-12-141.columbus.res.rr.com) has joined #mythtv | |
[04:01:58] | fetzerch (fetzerch!~quassel@unaffiliated/fetzerch) has quit (Ping timeout: 250 seconds) | |
[04:06:10] | sheedyz is now known as sheedy-away | |
[04:06:22] | fetzerch (fetzerch!~quassel@unaffiliated/fetzerch) has joined #mythtv | |
[04:53:57] | tgm4883 (tgm4883!uid23806@ubuntu/member/tgm4883) has joined #mythtv | |
[05:56:30] | SteveGoodey (SteveGoodey!~steve@host109-158-49-150.range109-158.btcentralplus.com) has joined #mythtv | |
[06:04:01] | SteveGoodey (SteveGoodey!~steve@host109-158-49-150.range109-158.btcentralplus.com) has quit (Quit: Konversation terminated!) | |
[06:06:33] | dekarl: | gigem, and if its one file system or multiple. there are issues when one filesystem of a SG is full, the other is empty and you combine load balancing with auto expiry. |
[06:07:11] | dekarl: | IIUIC the load balancer will decide on which disk to store the recording and then make room there |
[07:37:54] | peper03 (peper03!~peper03@mythtv/developer/peper03) has quit (Ping timeout: 260 seconds) | |
[07:38:31] | jya (jya!~jyavenard@mythtv/developer/jya) has quit (Ping timeout: 240 seconds) | |
[07:38:38] | peper03 (peper03!~peper03@mythtv/developer/peper03) has joined #mythtv | |
[07:38:50] | dblain (dblain!~dblain@mythtv/developer/dblain) has quit (Ping timeout: 260 seconds) | |
[08:05:42] | Merlin83b (Merlin83b!~Daniel@office.34sp.com) has joined #mythtv | |
[08:33:22] | jya (jya!~jyavenard@mythtv/developer/jya) has joined #mythtv | |
[08:41:15] | stuarta: | morning all |
[08:50:10] | stuarta: | MythBuild: force build cppcheck-master updated cppcheck |
[08:50:10] | MythBuild: | build forced [ETA 1h59m53s] |
[08:50:10] | MythBuild: | I'll give a shout when the build finishes |
[08:50:18] | stuarta: | 2hrs? |
[08:50:23] | stuarta: | wtf is that doing |
[08:58:20] | robink (robink!~quassel@unaffilated/robink) has quit (Ping timeout: 240 seconds) | |
[09:05:44] | robink (robink!~quassel@unaffilated/robink) has joined #mythtv | |
[09:21:44] | peper03 (peper03!~peper03@mythtv/developer/peper03) has quit (Ping timeout: 260 seconds) | |
[09:22:31] | peper03 (peper03!~peper03@mythtv/developer/peper03) has joined #mythtv | |
[09:26:27] | dekarl1 (dekarl1!~dekarl@p4FCEE914.dip0.t-ipconnect.de) has joined #mythtv | |
[09:28:36] | dekarl (dekarl!~dekarl@p4FE84502.dip0.t-ipconnect.de) has quit (Ping timeout: 250 seconds) | |
[09:30:39] | tgm4883 (tgm4883!uid23806@ubuntu/member/tgm4883) has quit (Quit: Connection closed for inactivity) | |
[10:11:36] | jya (jya!~jyavenard@mythtv/developer/jya) has quit (Quit: jya) | |
[10:24:14] | warpme (warpme!~piotro@89-79-250-31.dynamic.chello.pl) has joined #mythtv | |
[10:25:14] | warpme: | Hi * |
[10:26:53] | warpme: | It looks like there is regression in command socket handling after S3 sleep/resume. It was fixes in #7847 – but now it pop-ups again. Issue is also present when fe is send to idle.... |
[10:26:53] | ** MythLogBot http://code.mythtv.org/trac/ticket/7847 ** | |
[10:27:13] | warpme: | s/fixes/fixed/ |
[10:46:23] | MythBuild: | Hey! build cppcheck-master #3160 is complete: Success [3build successful] |
[10:46:23] | ** MythLogBot http://code.mythtv.org/trac/ticket/3160 ** | |
[10:46:23] | MythBuild: | Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . /builds/3160 |
[11:04:31] | joki (joki!~joki@p54863CB3.dip0.t-ipconnect.de) has quit (Ping timeout: 240 seconds) | |
[11:10:28] | joki (joki!~joki@p5486360A.dip0.t-ipconnect.de) has joined #mythtv | |
[11:52:48] | dreamcat4 (dreamcat4!~dreamcat4@62.49.10.153) has joined #mythtv | |
[12:04:08] | MythBuild: | build #2287 of master-debian-wheezy-64bit is complete: Failure [4failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2287 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[12:08:40] | MythBuild: | build #1978 of master-ubuntu-12_04-lts-64bit is complete: Failure [4failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1978 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[12:12:14] | MythBuild: | build #1080 of master-fedora-32bit is complete: Failure [4failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1080 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[12:12:55] | MythBuild: | build #607 of master-f20–64bit is complete: Failure [4failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/607 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[12:14:37] | MythBuild: | build #1175 of master-f19–64bit is complete: Failure [4failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1175 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[12:14:53] | warpme (warpme!~piotro@89-79-250-31.dynamic.chello.pl) has quit (Ping timeout: 240 seconds) | |
[12:15:00] | MythBuild: | build #481 of master-f20-qt5–64bit is complete: Failure [4failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/481 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[12:15:20] | stuarta: | doh! |
[12:18:52] | MythBuild: | build #2517 of master-linux-64bit-icc is complete: Failure [4failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2517 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[12:19:18] | MythBuild: | build #2160 of master-linux-64bit-clang is complete: Failure [4failed compile core] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2160 blamelist: Stuart Morgan <smorgan@mythtv.org > |
[12:23:48] | stuartm: | bonjour isn't building on my system, can't see to persuade it to either |
[12:25:00] | stuartm: | stuarta: any thoughts on trying the buildSetSummary=true for the buildbot mail notifier? |
[12:26:30] | stuartm: | wagnerrp raised a point about the slowest builders meaning that a failure email may not be sent for hours, but personally I'd rather receive one email instead of a dozen even if it takes longer to be dispatched |
[12:27:24] | stuartm: | builds that take excessive time should probably be culled? |
[12:29:08] | stuarta: | there's a timeout per build step |
[12:29:21] | andreaz (andreaz!~Andreaz@tmo-106-188.customers.d1-online.com) has joined #mythtv | |
[12:29:22] | stuarta: | but nothing that could trigger aborting a build when a different one fails |
[12:29:59] | stuarta: | do we even want email notification? |
[12:30:05] | stuartm: | buildSetSummary causes it to send one email instead of one per build |
[12:30:52] | stuartm: | stuarta: I think email notification is useful, I might miss IRC notifications that occur when I'm not here, but I look at all my email |
[12:31:28] | stuartm: | but checking my email in the morning to find dozens of 'build failed', 'build restored' emails is just annoying |
[12:31:44] | stuarta: | aye i just delete them all |
[12:31:55] | stuarta: | mainly because i'm never up to date with my inbox |
[12:33:59] | stuartm: | one email listing all the failed builders is more useful IMHO |
[12:34:20] | stuarta: | i'll look into it |
[12:34:22] | stuartm: | hence I asked in #buildbot and they pointed me at that config option |
[12:35:39] | stuartm: | /usr/include/avahi-compat-libdns_sd/dns_sd.h – configure doesn't seem to be finding it here |
[12:39:38] | MythBuild: | build #2161 of master-linux-64bit-clang is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2161 |
[12:40:40] | MythBuild: | build #2518 of master-linux-64bit-icc is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2518 |
[12:41:18] | MythBuild: | build #2288 of master-debian-wheezy-64bit is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/2288 |
[12:43:25] | MythBuild: | build #1979 of master-ubuntu-12_04-lts-64bit is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1979 |
[12:49:16] | stuartm: | gigem: http://cppcheck.sourceforge.net/devinfo/scan/error2957.html |
[12:49:41] | amessina (amessina!~amessina@2001:470:c1dc:7779:d6be:d9ff:fe8d:7c1e) has joined #mythtv | |
[12:54:39] | stuartm: | I've fixed the immediate problem, but I'm not sure whether the short strings are even used anywhere, I can't find then by grepping the source |
[12:54:50] | MythBuild: | build #608 of master-f20–64bit is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/608 |
[12:56:05] | MythBuild: | build #1081 of master-fedora-32bit is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1081 |
[12:56:40] | MythBuild: | build #1176 of master-f19–64bit is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1176 |
[13:01:36] | MythBuild: | build #482 of master-f20-qt5–64bit is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . t/builds/482 |
[13:04:04] | dreamcat4 (dreamcat4!~dreamcat4@62.49.10.153) has quit (Quit: Lost terminal) | |
[13:12:54] | stuarta: | stuartm: i found this site http://leanstack.io which is good for comparing the various different tools out there |
[13:15:00] | Jordack (Jordack!~Jordack@75-151-31-172-Michigan.hfc.comcastbusiness.net) has joined #mythtv | |
[13:15:16] | stuarta: | there are some fascinating tools out there |
[13:18:00] | stuartm: | interesting |
[13:18:23] | stuarta: | even just browsing the various categories is fascinating |
[13:28:49] | jheizer: | neat site |
[13:30:02] | stuarta: | aye |
[13:41:52] | stuartm: | first thing I thought to search for brought up no results :) |
[13:42:49] | CrypticSquared (CrypticSquared!~CrypticSq@unaffiliated/crypticsquared) has quit (Quit: Leaving...) | |
[13:43:35] | superm1 (superm1!uid4318@ubuntu/member/superm1) has joined #mythtv | |
[13:43:48] | stuartm: | guess mailing lists aren't trendy anymore |
[13:45:05] | stuarta: | hahaha |
[13:45:27] | jheizer: | lol |
[13:45:47] | jheizer: | CIS doesn't have the one we use and I don't think TeamCity is that rare. |
[13:46:02] | stuarta: | the site looks fairly new |
[13:46:33] | stuartm: | just wondering whether there was anything similar in functionality to mailman but which used less resources |
[13:47:17] | stuartm: | although OSUOSL are going to host our lists on their dedicated list server aren't they? So it wouldn't seem to matter very much |
[13:59:57] | stuarta: | not for the moment no |
[14:00:03] | stuarta: | but we can keep that as an option |
[14:06:09] | stuarta: | and does mailman really use a lot of resources? |
[14:07:53] | stuarta: | i see more resource usage from the various web backends |
[14:15:17] | stuartm: | historically it has used a lot, maybe it's calmed down more recently |
[14:17:41] | stuartm: | we've never kept a record of subscriber numbers, possibly they are down now that we've got the forum or people are reading the archive on Gossamer instead |
[14:17:45] | Anoia (Anoia!~dee@jane.earlsoft.co.uk) has left #mythtv () | |
[14:18:32] | gregL (gregL!~greg@cpe-74-76-121-109.nycap.res.rr.com) has quit (Ping timeout: 240 seconds) | |
[14:18:32] | taylorr (taylorr!~taylorr@unaffiliated/elmojo) has quit (Ping timeout: 240 seconds) | |
[14:18:59] | gregL (gregL!~greg@cpe-74-76-121-109.nycap.res.rr.com) has joined #mythtv | |
[14:19:00] | taylorr (taylorr!~taylorr@unaffiliated/elmojo) has joined #mythtv | |
[14:21:45] | stuarta: | stuartm: i'll monitor all this as i migrate it |
[14:34:13] | ** stuarta makes a note to add monitoring first ** | |
[14:39:22] | stuartm: | there's some awful code in AudioOutputPulse |
[14:39:45] | stuartm: | AudioOutputPulseAudio::ChooseHost() is a highlight so far |
[14:40:50] | stuartm: | if ( !(!host || *host == 0 || strcmp(host,"default") == 0)) |
[14:41:44] | stuartm: | or as most people would write it |
[14:41:55] | stuartm: | if (host && *host != 0 && strcmp(host,"default") != 0) |
[14:42:23] | stuarta: | ugh, that's like a c coder's first experiments in c++ |
[14:43:04] | stuartm: | even then, it needs refactoring, not sure why it's using a char pointer at all |
[14:53:19] | stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has quit (Ping timeout: 272 seconds) | |
[15:00:42] | MythBuild (MythBuild!~MythBuild@alcor.mythtv.org) has quit (Remote host closed the connection) | |
[15:00:53] | MythBuild (MythBuild!~MythBuild@140.211.167.130) has joined #mythtv | |
[15:08:02] | MythBuild (MythBuild!~MythBuild@140.211.167.130) has quit (Remote host closed the connection) | |
[15:08:13] | MythBuild (MythBuild!~MythBuild@140.211.167.130) has joined #mythtv | |
[15:24:37] | stuarta: | mingw build slave has been removed |
[15:24:52] | stuarta: | stuartm: i've also set that property on the mailNotifier |
[15:30:17] | stuartm: | thanks, guess I need to break the build now to see if it works ;) |
[15:30:30] | stuarta: | hahaha |
[15:45:13] | MythBuild (MythBuild!~MythBuild@140.211.167.130) has quit (Remote host closed the connection) | |
[15:45:24] | MythBuild (MythBuild!~MythBuild@140.211.167.130) has joined #mythtv | |
[15:51:49] | stuartm: | there are a lot of warnings generated by xine_demux_sputext.cpp/h, I wonder if we should be looking to sync with an upstream version |
[16:18:52] | andreaz (andreaz!~Andreaz@tmo-106-188.customers.d1-online.com) has quit (Read error: Connection reset by peer) | |
[16:37:43] | stichnot (stichnot!~stichnot@216.239.45.68) has joined #mythtv | |
[16:37:43] | stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has joined #mythtv | |
[16:37:43] | stichnot (stichnot!~stichnot@216.239.45.68) has quit (Changing host) | |
[16:48:57] | dekarl1 is now known as dekarl | |
[16:57:09] | Merlin83b (Merlin83b!~Daniel@office.34sp.com) has quit (Quit: Leaving) | |
[16:57:52] | stuartm: | stuarta: actually it's good practice to initialise a buffer to empty before use since it then avoids issues with reading garbage beyond the end of written data |
[16:58:53] | stuartm: | worst case scenario an exploit can use that to read sensitive data from memory if the code also fails to check that the requested read is less than any writes |
[17:01:05] | stuartm: | and for MythTV that really isn't very likely, but it's better to get into the habit of null'ing a buffer immediately after it's created |
[17:01:40] | ** stuartm notices that stuarta's commit message went to say exactly that ** | |
[17:02:55] | stuartm: | stuarta: sorry for the lecture, the text was truncated, didn't see the last four words :) |
[17:11:14] | stuartm: | stuarta: the cppcheck team are trialing a new system a bit like coverity where they'll generate the report for selected projects, devs can triage the results and anything marked as a false positive will get looked at by them, long story short they are now including mythtv – http://cppcheck.sourceforge.net/devinfo/scan/ http://cppcheck.sourceforge.net/devinfo/scan/ . . . -mythtv.html |
[17:11:25] | stuartm: | I've been working through it today |
[17:15:45] | SteveGoodey (SteveGoodey!~steve@host109-158-49-150.range109-158.btcentralplus.com) has joined #mythtv | |
[17:48:25] | gigem: | stuartm: I don't think that's the right URL you wanted me to look at. |
[17:55:16] | gregL (gregL!~greg@cpe-74-76-121-109.nycap.res.rr.com) has quit (Ping timeout: 250 seconds) | |
[18:06:03] | stuartm: | gigem: it was earlier, but the url got re-used when the report was updated :/ |
[18:07:48] | stuartm: | this was the issue – https://code.mythtv.org/cgit/mythtv/commit/?i . . . 28249dc45ef2 |
[18:08:28] | gregL (gregL!~greg@23.105.7.63) has joined #mythtv | |
[18:08:28] | stuartm: | type.toLower() == "template" || type.toLower() == "template" |
[18:08:37] | stuartm: | appeared to be a copy/paste issue |
[18:09:38] | stuartm: | but I was then wondering where the short forms – "override", "template", "findweekly" etc came from and whether they were still required |
[18:13:53] | gigem: | stuartm: I believe they come through the services API. To be honest, I really didn't understand it all completely when I last made the template and other rule type changes. Consequently, I could have easily gotten parts of it wrong. |
[18:16:26] | stuartm: | if that's the case I think they can be removed, the services API should only accept values identical to those it gives out and those are the ones in toRawString() |
[18:26:59] | Seeker` (Seeker`!~cjo20@unaffiliated/seeker) has quit (Remote host closed the connection) | |
[18:27:12] | Seeker` (Seeker`!~cjo20@unaffiliated/seeker) has joined #mythtv | |
[18:35:06] | gigem: | stuartm: To be clear, I was referring to the checks with the toLower(). Themes can use short names, but I'm not sure if it's the same short names the API uses. |
[18:54:20] | FrankD_Work (FrankD_Work!~frankd@resv-195.noc.cv.net) has quit (Ping timeout: 240 seconds) | |
[18:55:26] | andreaz (andreaz!~andre_000@p57922329.dip0.t-ipconnect.de) has joined #mythtv | |
[19:07:02] | gregL (gregL!~greg@23.105.7.63) has quit (Ping timeout: 240 seconds) | |
[19:08:57] | Merlin83b (Merlin83b!~Daniel@office.34sp.com) has joined #mythtv | |
[19:15:38] | gregL (gregL!~greg@23.105.7.72) has joined #mythtv | |
[19:19:08] | Merlin83b (Merlin83b!~Daniel@office.34sp.com) has quit (Quit: Leaving) | |
[19:45:44] | Merlin83b (Merlin83b!~Daniel@office.34sp.com) has joined #mythtv | |
[19:48:16] | Merlin83b (Merlin83b!~Daniel@office.34sp.com) has quit (Client Quit) | |
[19:50:28] | superm1 (superm1!uid4318@ubuntu/member/superm1) has quit (Quit: Connection closed for inactivity) | |
[19:50:34] | jpharvey: | cd usr |
[19:54:08] | dekarl: | stuartm, what does "intentional code" mean? Maybe I'm missing the point of unused private accessors like http://cppcheck.sourceforge.net/devinfo/scan/error2918.html |
[19:56:09] | stuartm: | dekarl: in that case I'd use "Intentional code", code which is intentionally unused, probably because it was added to support future development/features |
[19:56:27] | stuarta: | i disagree |
[19:56:43] | stuarta: | from a static analysis point of view, it is unused, so it's not a false positive |
[19:57:04] | stuarta: | static tools do not understand intent |
[19:57:13] | stuartm: | we're not marking as a false postive, but a 'true positive: intentional code' |
[19:57:35] | stuartm: | i.e. cppcheck is correct to flag it, but it's not actually a bug |
[19:57:38] | stuarta: | hmm true |
[19:58:29] | stuartm: | cppcheck is warning just in case it was meant to be used and was overlooked, that's more often going to be the case with unused variables but they are doing it for functions as well |
[19:58:41] | dekarl: | I think its more like a bug because the accessor is not used, e.g. here https://github.com/MythTV/mythtv/blob/master/ . . . tor.cpp#L282 |
[19:58:51] | dekarl: | But I do see your point |
[20:00:16] | dekarl: | looks like "someone started to add private accessors, but had to stop before getting to implementing the actual usage" half finished refactoring |
[20:00:26] | stuartm: | certainly if it was my code I'd fix it to use the Get/Set methods |
[20:00:56] | dekarl: | Looking at the contributors its up for grabs :D |
[20:01:03] | stuarta: | it's yours |
[20:01:17] | stuartm: | but at least for now I still come down on that being 'unfinished' (intentional) rather than 'broken' (bug) |
[20:01:29] | stuartm: | stuarta: :p |
[20:02:10] | Chutt_ (Chutt_!~ijr@cpe-76-190-199-73.neo.res.rr.com) has joined #mythtv | |
[20:02:12] | stuartm: | dekarl, stuarta: I nominate stichnot since he was the last one to touch subtitle code and he can actually test it since he's in the US |
[20:02:18] | stuarta: | \o/ |
[20:02:30] | stichnot: | wat |
[20:02:38] | stichnot: | reading scrollback now :) |
[20:02:42] | dekarl: | ;) |
[20:02:43] | stuarta: | :-p |
[20:03:08] | Chutt (Chutt!~ijr@2605:a000:1208:c08c:79a6:f255:ad71:b6a0) has quit (Ping timeout: 240 seconds) | |
[20:08:43] | rhpot1991 (rhpot1991!~rhpot1991@ubuntu/member/rhpot1991) has quit (Ping timeout: 272 seconds) | |
[20:10:12] | dekarl: | stuartm, the "Variable 'i' is assigned a value that is never used." was a bug in qobjectdefs.h when I last looked into it. got fixed in QT by adding a "i=i;" to Q_OBJECT |
[20:11:02] | rhpot1991 (rhpot1991!~rhpot1991@2001:4968:202:3:21ee:d0fe:a97:8015) has joined #mythtv | |
[20:11:05] | rhpot1991 (rhpot1991!~rhpot1991@2001:4968:202:3:21ee:d0fe:a97:8015) has quit (Changing host) | |
[20:11:05] | rhpot1991 (rhpot1991!~rhpot1991@ubuntu/member/rhpot1991) has joined #mythtv | |
[20:11:19] | stuartm: | dekarl: yeah, already discussed that with the lead cppcheck dev |
[20:12:56] | stuartm: | for now we'll leave it as false positive, even if it's a QT bug it's outside the scope of the code being checked and therefore probably shouldn't be reported as an issue |
[20:13:38] | dekarl: | ok, so "not *our* bug". |
[20:13:41] | stuartm: | but the cppcheck team can make the ultimate determination |
[20:14:43] | stuartm: | dekarl: basically, cppcheck shouldn't be reporting issues in libraries we're using just the code in our project |
[20:17:13] | stuartm: | I'm leaving the 'no copy constructor' warnings for last since they need to be individually checked in greater detail than most other reported issues |
[20:17:46] | stuartm: | some may be bugs, others won't need a copy constructor and can be marked as intentional |
[20:18:10] | stuartm: | actually implementing copy constructors for them all could take a while |
[20:18:45] | dekarl: | obviously the "i=i;" fix has its own issues... https://code.mythtv.org/buildbot/builders/mas . . . 20%287372%29 |
[20:38:58] | gregL (gregL!~greg@23.105.7.72) has quit (Ping timeout: 264 seconds) | |
[20:51:12] | SteveGoodey (SteveGoodey!~steve@host109-158-49-150.range109-158.btcentralplus.com) has quit (Quit: Konversation terminated!) | |
[20:54:02] | gregL (gregL!~greg@cpe-74-76-121-109.nycap.res.rr.com) has joined #mythtv | |
[20:59:44] | Jordack (Jordack!~Jordack@75-151-31-172-Michigan.hfc.comcastbusiness.net) has quit () | |
[21:18:11] | arescorpio (arescorpio!~arescorpi@201-26-245-190.fibertel.com.ar) has joined #mythtv | |
[21:22:20] | stuarta: | oh good ftp.osuosl.org is available via http as well as ftp |
[21:23:17] | stuartm: | pfft, you can't transfer files over http! |
[21:23:34] | stuarta: | i was more thinking of the theme downloader |
[21:23:50] | stuartm: | oooh |
[21:23:51] | beata (beata!~beata@pool-108-22-244-125.bltmmd.fios.verizon.net) has joined #mythtv | |
[21:24:02] | stuarta: | we push all the themes up there |
[21:24:21] | stuartm: | yeah, good idea |
[21:24:29] | stuarta: | less to host ourselves |
[21:26:10] | stuarta: | not to mention a decent chunk of disk space |
[21:26:24] | stuarta: | which we are limited on with the new vm |
[21:33:49] | dekarl: | stuarta btw seen that? http://www.mythtv.org/pipermail/mythtv-users/ . . . /366117.html |
[21:35:10] | ** stuarta goes to insult osuosl ** | |
[21:37:23] | stuarta: | dekarl: nope, but i did see an increase in unusual errors in the mail logs from pflogsumm about 2 days ago |
[21:45:49] | stuarta: | hmm boat load of emails never got delivered and a bunch more backed up |
[21:48:44] | rsiebert_ (rsiebert_!~quassel@g225036076.adsl.alicedsl.de) has quit (Ping timeout: 250 seconds) | |
[21:50:23] | gregL (gregL!~greg@cpe-74-76-121-109.nycap.res.rr.com) has quit (Ping timeout: 240 seconds) | |
[21:56:56] | rsiebert (rsiebert!~quassel@g225036076.adsl.alicedsl.de) has joined #mythtv | |
[22:02:10] | stuartm: | sigh |
[22:03:13] | stuarta: | indeed |
[22:10:26] | stuarta: | fyi support case raised |
[22:16:35] | gregL (gregL!~greg@cpe-74-76-121-109.nycap.res.rr.com) has joined #mythtv | |
[22:23:08] | stuartm: | MythBuild: force build cppcheck-master Updated config yet again |
[22:23:09] | MythBuild: | build forced [ETA 1h56m22s] |
[22:23:09] | MythBuild: | I'll give a shout when the build finishes |
[22:24:44] | MythBuild: | build #3167 of cppcheck-master is complete: Failure [4failed shell shell_1] Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . /builds/3167 |
[22:35:46] | stuartm: | MythBuild: force build cppcheck-master Hello, are you alive? |
[22:35:51] | MythBuild: | The build has been queued, I'll give a shout when it starts |
[22:37:21] | wahrhaft (wahrhaft!~quassel@cpe-76-181-12-141.columbus.res.rr.com) has quit (Ping timeout: 256 seconds) | |
[22:47:56] | dblain (dblain!~dblain@mythtv/developer/dblain) has joined #mythtv | |
[22:58:03] | dblain: | stuarta: master-win8-msvc-2010–32bit buildbot config is not pointing to new uri. |
[22:58:21] | stuarta: | dblain: s/not/now ?? |
[22:58:42] | dblain: | :( yes, now |
[22:58:51] | stuarta: | :) thanks |
[22:59:41] | dekarl: | stuarta, btw mizar has no reverse dns either |
[23:00:18] | stuarta: | dekarl: was raised in the same support request |
[23:02:29] | dekarl: | :) |
[23:02:30] | wahrhaft (wahrhaft!~quassel@cpe-76-181-12-141.columbus.res.rr.com) has joined #mythtv | |
[23:20:59] | arescorpio (arescorpio!~arescorpi@201-26-245-190.fibertel.com.ar) has quit (Excess Flood) | |
[23:29:16] | andreaz (andreaz!~andre_000@p57922329.dip0.t-ipconnect.de) has quit (Read error: Connection reset by peer) | |
[23:31:22] | stichnot (stichnot!~stichnot@mythtv/developer/stichnot) has quit (Ping timeout: 240 seconds) | |
[23:33:57] | rsiebert_ (rsiebert_!~quassel@g225009184.adsl.alicedsl.de) has joined #mythtv | |
[23:35:22] | MythBuild: | build #3168 of cppcheck-master is complete: Success [3build successful] Build details are at http://code.mythtv.org/buildbot/builders/cppc . . . /builds/3168 |
[23:35:23] | MythBuild: | build forced [ETA 1h28m08s] |
[23:35:23] | MythBuild: | I'll give a shout when the build finishes |
[23:36:38] | rsiebert (rsiebert!~quassel@g225036076.adsl.alicedsl.de) has quit (Ping timeout: 250 seconds) | |
[23:44:59] | tgm4883 (tgm4883!uid23806@gateway/web/irccloud.com/x-njswhqvhmhzbibgi) has joined #mythtv | |
[23:45:00] | tgm4883 (tgm4883!uid23806@gateway/web/irccloud.com/x-njswhqvhmhzbibgi) has quit (Changing host) | |
[23:45:00] | tgm4883 (tgm4883!uid23806@ubuntu/member/tgm4883) has joined #mythtv | |
[23:49:58] | Gibby_ (Gibby_!~Gibby@184.170.249.223) has quit (Ping timeout: 260 seconds) | |
[23:50:10] | Gibby (Gibby!~Gibby@184.170.249.223) has joined #mythtv |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.