| Tuesday, August 16th, 2011, 17:40 UTC | ||
| [17:40:25] | wagnerrp: | Captain_Murdoch: well its working, but to be honest, its a bit too convoluted for my tasts |
| [17:41:07] | wagnerrp: | through i dont really know how you would simply it and still check for inter-dependencies |
| [17:43:39] | wagnerrp: | http://www.wagnerrp.com/files/MythTV/commandlineparser/ |
| [17:44:05] | wagnerrp: | arguments are now stored in a pointer to a new class |
| [17:44:19] | wagnerrp: | the various add() methods return a pointer to the new instance |
| [17:44:46] | wagnerrp: | to which you can SetGroup(), SetParentOf(), SetChildOf(), SetRequires(), or SetBlocks() |
| [17:45:24] | wagnerrp: | each takes a string, and the cross referenced names are reconciled to the argument instances once they are all stored |
| [17:46:32] | wagnerrp: | child options are listed indented under each of their parents, and require at least one of their parents to be set on the command line to be valid |
| [17:46:49] | wagnerrp: | required arguments require all to be set, and blocks require none to be set |
| [17:47:58] | wagnerrp: | aside from that, ordering is still alphabetical |
| [17:49:43] | Anduin (Anduin!~awithers@pdpc/supporter/professional/anduin) has joined #mythtv | |
| [18:01:11] | andreax1 (andreax1!~andreaz@p57B92BB1.dip.t-dialin.net) has joined #mythtv | |
| [18:02:09] | andreax (andreax!~andreaz@p57B92D1B.dip.t-dialin.net) has quit (Ping timeout: 250 seconds) | |
| [18:05:54] | robertzaccour (robertzaccour!18b7dda1@gateway/web/freenode/ip.24.183.221.161) has joined #mythtv | |
| [18:07:44] | robertzaccour: | I'm confused about the hd-pvr setup. How do I unpack the driver? |
| [18:08:03] | wagnerrp: | this is the mythtv development channel, you want #mythtv-users |
| [18:08:55] | robertzaccour: | nobody there |
| [18:09:05] | robertzaccour: | oops typed it wrong |
| [18:09:15] | wagnerrp: | theres 168 users there, but you arent |
| [18:09:26] | robertzaccour: | I just said I typed it wrong |
| [18:10:31] | stuartm: | so std::auto_ptr is deprecated in c++11, I know there are one or two uses in the code which can easily be removed which I think is preferable to maintaining the behaviour when 99.9% of the code requires pointers to be manually freed |
| [18:12:27] | stuartm: | although it seems we're using a number of instances of QPointer which is the same thing |
| [18:12:43] | robertzaccour (robertzaccour!18b7dda1@gateway/web/freenode/ip.24.183.221.161) has left #mythtv () | |
| [18:24:03] | gigem_: | stuartm: hmm, i see an example in my --printsched output now. i'm stumped, but will keep looking. |
| [18:26:54] | Captain_Murdoch: | wagnerrp, I haven't looked over the libmythbase files, but did look over the mfdb change. it is the arg->SetBlah() stuff you're thinking is convoluted or the implementation of those methods? |
| [18:36:07] | wagnerrp: | Captain_Murdoch: the internal implementation of those methods |
| [18:36:26] | Captain_Murdoch: | wagnerrp, ah, ok. yeah, I can see how that could get hairy. |
| [18:39:25] | stoffel (stoffel!~quassel@p57B4C5A4.dip.t-dialin.net) has quit (Remote host closed the connection) | |
| [19:14:02] | taylorr (taylorr!~taylorr@cpe-173-095-144-220.nc.res.rr.com) has joined #mythtv | |
| [19:14:02] | taylorr (taylorr!~taylorr@unaffiliated/elmojo) has joined #mythtv | |
| [19:14:02] | taylorr (taylorr!~taylorr@cpe-173-095-144-220.nc.res.rr.com) has quit (Changing host) | |
| [19:21:00] | kth (kth!~kth@unaffiliated/kth) has quit (Quit: Leaving.) | |
| [19:24:22] | gigem_: | stuartm: found it. it was a case of a fix in one area obscuring a related fix in another area. |
| [19:30:28] | stuartm: | gigem_: :) |
| [19:36:06] | danielk22: | stuartm: QPointer is a bit smarter than std::auto_ptr :) |
| [19:37:05] | danielk22: | It looks like QPointer is also depreciated in Qt4 though... |
| [19:40:05] | danielk22: | Well the QPointer replacements seem to require the pointed to to be derived from QObject, so depreciated is too strong a word.. |
| [19:53:11] | brfransen (brfransen!~brfransen@216.254.250.47) has quit (Read error: Connection reset by peer) | |
| [19:55:26] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has joined #mythtv | |
| [20:06:00] | brfransen (brfransen!~brfransen@216.254.250.47) has joined #mythtv | |
| [20:39:49] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [20:40:52] | danielk22: | Beirdo: http://www.cuymedia.net/tmp/logging-patches.tbz <-- 9 patches for logging... |
| [20:41:37] | Beirdo: | OK, let me take a look |
| [20:41:57] | Beirdo: | I think the deleteItem() one you just put in may actually be needed. |
| [20:42:15] | Beirdo: | the deleteItem will decrement the use count, and only delete when everyone has used it |
| [20:42:40] | Beirdo: | without it there, we may never delete that logging item |
| [20:44:37] | Beirdo: | ahhh, never mind |
| [20:44:38] | Beirdo: | hehe |
| [20:44:49] | danielk22: | :) when we return false we reinsert it into the queue.. |
| [20:44:55] | Beirdo: | that code case is when it returns and reinserts |
| [20:44:56] | Beirdo: | correct |
| [20:45:04] | Beirdo: | heh |
| [20:45:13] | Beirdo: | been one of those weeks already |
| [20:46:27] | Beirdo: | OK, for the first patch, I don't see why this would be necessary, or even desired. |
| [20:47:28] | Beirdo: | it makes the compilation slower, and in the end really doesn't save you much more than a function call which is a push/jump/pull |
| [20:48:45] | Beirdo: | I would even venture to say that it could significantly affect gcc's capability to optimize the code |
| [20:50:18] | Beirdo: | also, moving the level < 0 check into the macro will break things |
| [20:51:04] | Beirdo: | there are places in the code where the LOG_ANY is used (-1) for checking levels, but it isn't valid for a log message |
| [20:55:07] | Beirdo: | as for the second patch... I very specifically made it a struct rather than a class as at one point (not sure if it still is the case), that type is visible to C users |
| [20:56:06] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit () | |
| [20:56:07] | danielk22: | In the first one the reason for it is so that the QString() doesn't need to be constructed when we're not going to be logging anything. |
| [20:56:22] | Beirdo: | Hmmm. |
| [20:56:37] | danielk22: | The second one doesn't make much sense till you look at 3 & 4. |
| [20:56:52] | danielk22: | It's not used in C code anywhere right now. |
| [20:56:52] | Beirdo: | I guess that could save some time, but is it really worth the extra overhead to save some overhead? |
| [20:57:02] | Beirdo: | OK, good, I'm glad that it's not :) |
| [20:59:00] | danielk22: | Beirdo: 2,3,4 yeah the overhead saved is minimal, basically the cost of memset, but it gets rid of code duplication in the initialization which tends to be a source of errors down the line. |
| [20:59:22] | Beirdo: | yeah, fair enough |
| [21:01:01] | Beirdo: | You still should explicitly cast the currentThreadId() result to uint64_t to cover all the bases... It's an "opaque" value, and we are forcing it into something we can display easier and consistently... |
| [21:01:09] | Beirdo: | at least that was the original thought |
| [21:01:56] | danielk22: | fair enough.. |
| [21:01:59] | j-rod is now known as j-rod|afk | |
| [21:02:13] | Beirdo: | might bite our butts in Windows or some crazy thing :) |
| [21:02:25] | Beirdo: | hehe |
| [21:02:32] | Beirdo: | I like #5 |
| [21:02:54] | Beirdo: | yeah, if we fit into a smaller size, it can really help at times |
| [21:03:17] | Beirdo: | although malloc gives 4k pages in Linux, so it might not help much |
| [21:03:28] | danielk22: | heh, after getting rid of the extra alloc for the message it was really close to being half a page size.. |
| [21:03:34] | Beirdo: | yeah |
| [21:03:47] | stuartm: | more old UI code bites the dust |
| [21:03:54] | Beirdo: | #6... I'm embarrassed that I missed that first 64bit assign |
| [21:04:42] | danielk22: | heh, that should have been a separate patch.. the compiler pointed it out to me. |
| [21:05:05] | Beirdo: | I can't believe I missed that :) |
| [21:05:13] | Beirdo: | ah well, it's fixed now. |
| [21:05:25] | stuarta: | \o/ |
| [21:05:41] | danielk22: | With 5 I was thinking about a custom allocator. The memory usage of the logger is very predictable.. |
| [21:05:43] | Beirdo: | I see you may have fixed the database disabling. It still seems to flap a bit for me sometimes |
| [21:05:49] | Beirdo: | yeah, it is |
| [21:06:19] | Beirdo: | we could make a pool of items and manage them ourselves |
| [21:06:32] | danielk22: | Yeah, I don't think it will save much time though, since the QString mallocs are really what take the most time in processing LOG() |
| [21:06:40] | Beirdo: | which could significantly speed it up |
| [21:06:46] | danielk22: | It might save some memory... |
| [21:06:46] | Beirdo: | ahh |
| [21:07:00] | Beirdo: | yeah, it would definitely save memory |
| [21:07:25] | danielk22: | Right, QString is why patch 1 tries to put that in the macro. |
| [21:07:27] | Beirdo: | we could change all logging to be sprintf style instead of QString :) |
| [21:07:45] | danielk22: | Well QString is really convenient... |
| [21:07:47] | Beirdo: | that would be a super-painful endeavour though |
| [21:07:52] | Beirdo: | exactly |
| [21:08:07] | Beirdo: | it crossed my mind for a very short time |
| [21:08:57] | danielk22: | In 6 I put a timer for the DB logging re-enablement because I was seeing flapping when I turned on VB_ALL logging... |
| [21:09:23] | Beirdo: | yeah, it wasn't workin quite right |
| [21:09:24] | danielk22: | It wasn't stopping because the DB was down, it just couldn't keep up :) |
| [21:09:37] | Beirdo: | I hit that HARD on some of the mythtranscode testing |
| [21:11:01] | Beirdo: | VB_FLUSH seems like an odd way to implement a flush |
| [21:11:03] | danielk22: | 7 is just an example of using VB_STDIO, it probably make more sense in the commflag console output though & it's not quite right. |
| [21:11:32] | danielk22: | how so? |
| [21:12:07] | Beirdo: | ahhh, nvm. I was thinking of the shutdown flush |
| [21:12:08] | Beirdo: | heh |
| [21:12:18] | Beirdo: | that's a output flush |
| [21:13:55] | danielk22: | ah, there is a shutdown flush() command.. but I didn't add one for the DB logger. VB_FLUSH is intended have the convenience of cout<<flush.. I was thinking of the commflag % output. |
| [21:14:11] | Beirdo: | yeah |
| [21:14:20] | Beirdo: | heh, I was just getting confused |
| [21:14:33] | Beirdo: | using VB_FLUSH to do a shutdown flush would be kinda odd |
| [21:14:35] | Beirdo: | heh |
| [21:14:53] | danielk22: | yeah :) |
| [21:15:11] | Beirdo: | how many messages are we maybe losing on shutdown? |
| [21:15:22] | Beirdo: | just looking at #8 |
| [21:15:55] | danielk22: | About 3 during a normal shutdown, a dozen when something goes wrong. |
| [21:15:56] | Beirdo: | a lot of complexity, just wondering if we are actually missing them, or if that's just protection? |
| [21:16:02] | Beirdo: | ah. OK then |
| [21:16:16] | Beirdo: | and those can well be VERY important messages too |
| [21:16:49] | danielk22: | We're missing them.. I noticed it when debugging the MThread::Cleanup(), by necessity it runs after logging is shutdown.. |
| [21:17:13] | Beirdo: | heh, yeah |
| [21:18:07] | danielk22: | When something went wrong it didn't print anything so I converted those to cout redirects.. but after this (or something like it) is in I'll convert them back to LOG(). |
| [21:18:44] | Beirdo: | in #9, I did specifically want a local cached copy of the application name so we don't waste tons of time pulling it from the core application repeatedly |
| [21:18:59] | Beirdo: | likewise the host name |
| [21:19:29] | danielk22: | Beirdo: It only does the prepare once now, unless there is an error so we don't really need to cache.. |
| [21:19:53] | danielk22: | & bind of app name and host name.. |
| [21:19:57] | Beirdo: | what you have there will work, but we are binding on every message to the values actively pulled from the classes |
| [21:20:08] | Beirdo: | Oooh |
| [21:20:13] | Beirdo: | it binds once? |
| [21:20:17] | Beirdo: | never mind :) |
| [21:20:18] | danielk22: | yep :) |
| [21:20:41] | Beirdo: | yeah, that should do the trick. I missed that it was before the loop for those binds |
| [21:20:57] | danielk22: | That's really the point of #9... now that the DB reconnect logic isn't losing prepares we don't need to do all those prepares.. |
| [21:21:10] | Beirdo: | yeah |
| [21:21:12] | Beirdo: | nice |
| [21:21:42] | Beirdo: | OK, so other than a (uint64_t) cast on the currentThread... all looks good to me :) |
| [21:21:43] | danielk22: | thx, but that was really done on sphery's encouragement. |
| [21:21:52] | Beirdo: | :) |
| [21:22:38] | Beirdo: | thanks for passing them by me. A lot of little fiddly things in there that should hopefully make it faster and more stable yet. |
| [21:22:56] | danielk22: | Great.. I'll make that change & revisit patch #1 and run with those changes on production overnight.. |
| [21:23:07] | Beirdo: | sweet. |
| [21:23:26] | danielk22: | Can you point me to an example for #1 with the VB_ALL? |
| [21:23:43] | Beirdo: | ah yes, that part :) |
| [21:24:08] | danielk22: | BTW I also changed the VERBOSE and VERBOSE_LEVEL_CHECK() macros.. |
| [21:24:12] | Beirdo: | LOG_ANY, actually |
| [21:24:35] | Beirdo: | mythtv/libs/libmythtv/avformatdecoder.cpp has a could in there |
| [21:24:41] | Beirdo: | a couple rather |
| [21:25:05] | danielk22: | VERBOSE I changed because I was getting a confusing compiler error whenever I accidentally used VERBOSE instead of LOG in my code. |
| [21:25:16] | Beirdo: | hehe, that was the intention :) |
| [21:25:26] | Beirdo: | hit you up earlier in the compile |
| [21:25:42] | Beirdo: | but this works too, will fail in link anyways |
| [21:25:45] | danielk22: | I added a logLevel >= (level) check to the VERBOSE_LEVEL_CHECK |
| [21:26:19] | Beirdo: | one sec, let me go look at that again |
| [21:26:27] | brfransen (brfransen!~brfransen@216.254.250.47) has quit (Read error: Connection reset by peer) | |
| [21:26:57] | danielk22: | Beirdo: Right, but the error just told me the LVALUE would fail at run time, which was a head scratcher... |
| [21:27:02] | Beirdo: | yeah, it's the ((_LEVEL_)>=0) part that will cause issues in those check |
| [21:27:14] | Beirdo: | haha, nice error message |
| [21:28:04] | Beirdo: | we have a few places that use LOG_ANY (-1), and the modified VERBOSE_LEVEL_CHECK will have a different result for them |
| [21:28:05] | brfransen (brfransen!~brfransen@216.254.250.47) has joined #mythtv | |
| [21:28:45] | Beirdo: | the only place the >=0 is needed is in the LOG() expansion |
| [21:30:06] | Beirdo: | oh, and something I learned years back... when using #define to make macros... it's wise to put () around the args as expanded in the macro religiously as people can do stupid things |
| [21:30:10] | Beirdo: | :) |
| [21:30:21] | danielk22: | This reminds me.. VB_STDIO & VB_FLUSH should always be enabled, but not show up on -v help.. but just setting the default mask doesn't quite wurk.. |
| [21:30:25] | Beirdo: | I notice I didn't do it myself (bad boy) |
| [21:30:52] | Beirdo: | hmmm |
| [21:31:00] | danielk22: | Beirdo: yep, I tried to do that + do {} while (0) protection and using names that won't conflict with variable names. |
| [21:31:08] | Beirdo: | :) |
| [21:31:29] | Beirdo: | yeah, it's always good to be careful |
| [21:32:05] | Beirdo: | and yeah, I like the while(0) methodology so we don't need to remove ; |
| [21:33:29] | Beirdo: | stuartm: YAY, good job removing more old cruft! :) |
| [21:34:44] | danielk22: | I mostly do that do it so if (..) MACRO_A(); else MACRO_B(); works as expected... |
| [21:35:24] | Beirdo: | danielk22: anyways, to fix the >=0, you can just move the && ((_LEVEL_)>=0) part into the if in the LOG() definitions, and all should be grand |
| [21:35:44] | danielk22: | already done :) |
| [21:35:51] | Beirdo: | yeah, it makes macros look like function calls :0 |
| [21:35:55] | Beirdo: | cool :) |
| [21:36:32] | Beirdo: | now that I have a replacement frontend in place, I can get back to fixing bugs, etc too. |
| [21:36:47] | Beirdo: | dying power supplies are evil |
| [21:37:44] | skd5aner: | quick question related to this upnp commit from mark, it says UPnP MediaServer support for mythfrontend. Does he mean to say MediaServer (as in a control point which can find other UPnP servers and provider their data) or a MediaRenderer to play back other UPnP server's content? http://www.mythtv.org/pipermail/mythtv-fireho . . . /000907.html |
| [21:38:08] | stuartm: | Beirdo: paul-h gets the credit, the only place still using those classes was the mythmusic metadata editor which he recently finished porting to mythui |
| [21:38:35] | Beirdo: | skd5aner: hmm, I'd have to take a look at what he implemented, but I thought he was working on MediaRenderer stuff |
| [21:38:46] | Beirdo: | cool |
| [21:38:54] | Beirdo: | next victim: mythgallery? :) |
| [21:39:36] | Beirdo: | skd5aner: the commit modified mediarenderer.cpp, I think he may have typoed the comment |
| [21:40:02] | skd5aner: | Beirdo: yea, that's what sort of tipped me off, plus some of the other comments about other servers and scanning |
| [21:40:30] | Beirdo: | yeah, it looks like he was adding support to connect to MediaServers |
| [21:40:38] | Beirdo: | as a MediaRenderer |
| [21:41:26] | Beirdo: | OK, I'd better go bug my coworker now |
| [22:18:33] | wagnerrp (wagnerrp!~wagnerrp_@mythtv/developer/wagnerrp) has quit (Read error: Connection reset by peer) | |
| [22:20:53] | wagnerrp (wagnerrp!~Wagner@nr-ft1-66-42-241-137.fuse.net) has joined #mythtv | |
| [22:36:15] | danielk22: | Hmmm, I wonder if the lack of activity on the MySQL related BE deadlock threads means it really is fixed in trunk now... *crosses fingers* |
| [22:39:36] | kormoc is now known as kormoc_afk | |
| [22:43:03] | kormoc_afk is now known as kormoc | |
| [22:43:37] | SteveGoodey (SteveGoodey!~steve@host86-160-43-155.range86-160.btcentralplus.com) has joined #mythtv | |
| [22:44:42] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has quit (Ping timeout: 258 seconds) | |
| [23:05:49] | SteveGoodey (SteveGoodey!~steve@host86-160-43-155.range86-160.btcentralplus.com) has quit (Remote host closed the connection) | |
| [23:09:46] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has quit (Ping timeout: 264 seconds) | |
| [23:17:58] | iamlindoro: | danielk22: Personally, I can't parse warpme's language most of the time-- I *thought* the last thing he said suggested it wasn't fixed, but then I re-read it and it sort of sounded like he hadn't updated or tried at all |
| [23:19:31] | danielk22: | iamlindoro: My understanding was he was about to try it.. and since that was a couple days ago and he was experiencing the problem pretty frequently... |
| [23:20:35] | danielk22: | iamlindoro: I think markk was also experiencing backend stability problems, which was probably the same issue.. so I think we'll know more definitively after his epic return to the UK finishes. |
| [23:24:06] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [23:46:13] | gigem (gigem!~gigem@mythtv/developer/gigem) has quit () | |
| [23:48:02] | gigem (gigem!~gigem@mythtv/developer/gigem) has joined #mythtv | |
| [23:48:18] | kormoc_afk (kormoc_afk!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
| [23:48:19] | kormoc_afk is now known as kormoc | |
| [23:50:53] | kormoc (kormoc!~kormoc@mythtv/developer/kormoc) has quit (Client Quit) | |
| [23:52:51] | kormoc_afk (kormoc_afk!~kormoc@mythtv/developer/kormoc) has joined #mythtv | |
| [23:52:52] | kormoc_afk is now known as kormoc | |
| [23:59:06] | Mousey (Mousey!~wtfisme@ross154.net) has quit (Ping timeout: 260 seconds) | |
| Wednesday, August 17th, 2011 | ||
| [00:36:41] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has quit (Quit: Leaving.) | |
| [00:46:28] | timlegge (timlegge!~tim@mctnnbsa51w-156034063103.dhcp-dynamic.FibreOp.nb.bellaliant.net) has quit (Ping timeout: 250 seconds) | |
| [01:00:19] | timlegge (timlegge!~tim@mctnnbsa51w-156034063103.dhcp-dynamic.FibreOp.nb.bellaliant.net) has joined #mythtv | |
| [01:03:59] | gigem_ (gigem_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [01:04:28] | gigem_ (gigem_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [01:19:41] | dagar (dagar!~dagar@agar.ca) has quit (Ping timeout: 240 seconds) | |
| [01:20:37] | dagar (dagar!~dagar@agar.ca) has joined #mythtv | |
| [01:24:42] | danielk22: | Heh, so far my favorite feature of C++11 is from Delphi — override — when present on a virtual function it will error out if the parent no longer provides a virtual function with that exact signature. |
| [01:27:45] | danielk22: | nullptr is nice, but the NULL macro in g++ provides the same advantage over 0 when initializing pointers. The auto type and the new template stuff just seem like the kind of "features" that will only cause grief. |
| [01:35:32] | georgespatton (georgespatton!~georgespa@166.77.103.133) has joined #mythtv | |
| [01:39:18] | georgespatton (georgespatton!~georgespa@166.77.103.133) has quit (Client Quit) | |
| [01:42:41] | georgespatton (georgespatton!~georgespa@166.77.103.133) has joined #mythtv | |
| [01:49:29] | georgespatton (georgespatton!~georgespa@166.77.103.133) has quit (Quit: Leaving) | |
| [02:34:13] | andreax1 (andreax1!~andreaz@p57B92BB1.dip.t-dialin.net) has quit (Quit: Leaving.) | |
| [02:45:11] | bruderbell (bruderbell!~bruderbel@c-174-52-174-87.hsd1.ut.comcast.net) has joined #mythtv | |
| [02:46:12] | bruderbell (bruderbell!~bruderbel@c-174-52-174-87.hsd1.ut.comcast.net) has left #mythtv () | |
| [03:09:18] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [03:34:45] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has quit (Ping timeout: 260 seconds) | |
| [03:38:27] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has joined #mythtv | |
| [03:53:41] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has quit (Ping timeout: 252 seconds) | |
| [04:15:54] | jcarlos (jcarlos!~quassel@85.137.99.76.dyn.user.ono.com) has joined #mythtv | |
| [04:57:46] | dagar (dagar!~dagar@agar.ca) has quit (Ping timeout: 264 seconds) | |
| [04:58:40] | dagar (dagar!~dagar@206.248.137.66) has joined #mythtv | |
| [05:55:46] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has joined #mythtv | |
| [05:58:37] | kth (kth!~kth@unaffiliated/kth) has joined #mythtv | |
| [06:00:37] | kth (kth!~kth@unaffiliated/kth) has quit (Client Quit) | |
| [06:15:06] | dagar (dagar!~dagar@206.248.137.66) has quit (Read error: Operation timed out) | |
| [06:19:29] | dagar (dagar!~dagar@agar.ca) has joined #mythtv | |
| [07:09:10] | kwmonroe (kwmonroe!~kwmonroe@32.97.110.58) has quit (Ping timeout: 258 seconds) | |
| [07:09:25] | kwmonroe (kwmonroe!~kwmonroe@32.97.110.58) has joined #mythtv | |
| [08:10:50] | iamlindoro (iamlindoro!~iamlindor@mythtv/developer/iamlindoro) has quit (Ping timeout: 250 seconds) | |
| [08:18:52] | stuarta: | danielk22: what about the stuff to avoid deep copies? |
| [08:19:16] | jya (jya!~jyavenard@mythtv/developer/jya) has joined #mythtv | |
| [08:27:33] | hashbang (hashbang!~alex@213-152-35-50.dsl.eclipse.net.uk) has joined #mythtv | |
| [08:38:53] | iamlindoro (iamlindoro!~iamlindor@mythtv/developer/iamlindoro) has joined #mythtv | |
| [09:05:06] | iamlindoro (iamlindoro!~iamlindor@mythtv/developer/iamlindoro) has quit (Ping timeout: 260 seconds) | |
| [09:11:59] | MJBrune (MJBrune!~MJBrune@unaffiliated/mjbrune) has joined #mythtv | |
| [09:12:27] | MJBrune: | hey |
| [09:12:39] | MJBrune: | can mythtv run files from samba shares? |
| [09:12:59] | MJBrune: | erm sorry |
| [09:13:07] | ** MJBrune moves to #mythtv-users ** | |
| [09:21:02] | Guest44869 (Guest44869!~mike@c-24-21-63-118.hsd1.or.comcast.net) has quit (Ping timeout: 258 seconds) | |
| [09:26:56] | stuartm: | danielk22: I like the sound of override |
| [09:43:06] | iamlindoro (iamlindoro!~iamlindor@mythtv/developer/iamlindoro) has joined #mythtv | |
| [10:16:17] | kth (kth!~kth@unaffiliated/kth) has joined #mythtv | |
| [10:17:27] | kth (kth!~kth@unaffiliated/kth) has quit (Client Quit) | |
| [10:38:42] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [10:41:12] | andreax (andreax!~andreaz@p57B92BB1.dip.t-dialin.net) has joined #mythtv | |
| [10:51:22] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [10:52:37] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [10:58:34] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Remote host closed the connection) | |
| [11:04:55] | kth (kth!~kth@unaffiliated/kth) has joined #mythtv | |
| [11:06:42] | kth (kth!~kth@unaffiliated/kth) has quit (Client Quit) | |
| [11:17:39] | stuartm: | danielk22: not only does override guard against mistakes but it also documents that the method is overriding a virtual in the base class, which can save some time if you're unfamiliar with the code |
| [11:17:53] | stuartm: | so there are multiple reasons to like it :) |
| [11:35:38] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [11:46:38] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Ping timeout: 250 seconds) | |
| [12:03:24] | gigem_ (gigem_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [12:03:50] | gigem_ (gigem_!~david@host103.16.intrusion.com) has joined #mythtv | |
| [12:03:50] | gigem_ (gigem_!~david@host103.16.intrusion.com) has quit (Changing host) | |
| [12:03:50] | gigem_ (gigem_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [12:23:29] | danielk22: | stuartm: re deep copies you mean being able to call other constructors or something else. The other constructor stuff can be convenient, but the limitation is easily worked around in C++03 |
| [12:24:47] | stuarta: | that was mee |
| [12:25:49] | danielk22: | stuarta: being able to write int x = 3; in the class definition will be nice and help keep multiple constructors setting the same constant consistent. |
| [12:26:44] | stuarta: | http://en.wikipedia.org/wiki/C%2B%2B0x is what i was looking at |
| [12:27:43] | stuarta: | under the core language runtime improvements / rvalue references and move constructors 2nd paragraph "A chronic performance problem...." |
| [12:29:49] | danielk22: | That both overstates the problem and the power of the C++11 solution. |
| [12:30:41] | andreax (andreax!~andreaz@p57B92BB1.dip.t-dialin.net) has quit (Quit: Leaving.) | |
| [12:31:36] | planktonboy (planktonboy!~john@82.152.114.160) has joined #mythtv | |
| [12:33:13] | danielk22: | They don't implement Qt container style shared data because the reference counting adds a 10–20% speed penalty, so all this solution does is allow you to write "vector<int> GetVector()" instead of "void GetVector(vector<int> &)" |
| [12:52:09] | stuartm: | xris: the channel icon resync, I'm sure you've told me how to do it before but I can't remember? |
| [12:54:59] | ** stuartm is running "db.pl --scan lyngsat" and hoping that it doesn't screw anything up ** | |
| [12:55:39] | stuartm: | fwiw, the hdtv icon grab seems broken |
| [12:56:29] | stuartm: | looks like they've dropped those pages |
| [13:23:58] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
| [13:28:32] | eharris (eharris!~eharris@99-179-7-82.lightspeed.austtx.sbcglobal.net) has joined #mythtv | |
| [13:36:45] | stuartm: | xris: for some reason submissions I know are in the pending queue (they are in the pending_dvb table) aren't showing up in the edit screen |
| [13:36:55] | stuartm: | s/edit/approval/ |
| [13:41:15] | andreax (andreax!~andreaz@p57B92BB1.dip.t-dialin.net) has joined #mythtv | |
| [13:46:25] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [14:12:01] | planktonboy is now known as plankgone | |
| [14:32:52] | j-rod|afk is now known as j-rod | |
| [14:40:04] | cocoa117 (cocoa117!~cocoa117@wk-29-212.guest.rdg.ac.uk) has joined #mythtv | |
| [14:50:41] | plankgone (plankgone!~john@82.152.114.160) has quit (Remote host closed the connection) | |
| [14:58:51] | stuartm: | xris: something funky is going on with the icon stuff, counts on submissions in the database are often much higher than those displayed, in one case I've found it displayed '115' in the database but just '32' on the site |
| [15:10:15] | cocoa117 (cocoa117!~cocoa117@wk-29-212.guest.rdg.ac.uk) has quit (Quit: Leaving) | |
| [15:20:59] | paul-h: | stuartm: Terra is complaining loudly about a missing image in the Video Browser mv_browse_nocover.png. Maybe it should be mv_browse_nocover_large.png? |
| [15:46:05] | stuartm: | paul-h: I'll take a look |
| [15:47:16] | stuartm: | xris: I think I'll add a timestamp column to the pending_ tables so we can weed out the oldest submissions, especially those were only one was received |
| [16:05:50] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has joined #mythtv | |
| [16:09:40] | kormoc is now known as kormoc_afk | |
| [16:26:51] | MythBuild: | build #1764 of master-linux-32bit is complete: Failure [failed compile plugins] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1764 blamelist: foobum <foobum@gmail.com > |
| [16:36:54] | wagnerrp: | jya: ^^^^ |
| [16:37:15] | iamlindoro: | wrong guy |
| [16:37:41] | wagnerrp: | oh, paul-h pushed it? |
| [16:37:49] | wagnerrp: | ok... paul-h ^^^ |
| [16:40:14] | dekarl: | stuartm: have you imported new channel icons from lyngsat? If so I'll fire up setup and attach some to their dvb channels |
| [16:42:49] | stuartm: | dekarl: yes, the database has been updated |
| [16:43:27] | stuartm: | I might setup a weekly cron job to keep it in sync |
| [16:43:32] | dekarl: | thanks! |
| [16:49:49] | kth (kth!~kth@unaffiliated/kth) has joined #mythtv | |
| [16:50:49] | dekarl: | stuartm: I can't assign logos to channels with "," in their name , shall I file a bug? (Looking at "NDR 90,3" FM station, so it's not really that important) |
| [16:51:28] | vallor (vallor!~Ponzo@pdpc/supporter/monthlygold/vallor) has quit (Remote host closed the connection) | |
| [16:52:36] | vallor (vallor!~Ponzo@pdpc/supporter/monthlygold/vallor) has joined #mythtv | |
| [16:54:55] | stuartm: | yeah, file a ticket please and we'll see if that can be fixed |
| [16:55:08] | Goga777 (Goga777!~Goga777@shpd-78-36-179-161.static.vologda.ru) has joined #mythtv | |
| [16:56:08] | dekarl: | will do |
| [16:57:42] | stuartm: | heh, after hours of approving icon submissions there are still 68,229 dvb submissions remaining |
| [16:58:06] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Read error: Connection reset by peer) | |
| [16:58:31] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [16:58:36] | zombor (zombor!~zombor_@kohana/developer/zombor) has quit (Read error: Connection reset by peer) | |
| [16:59:04] | zombor (zombor!~zombor_@kohana/developer/zombor) has joined #mythtv | |
| [17:03:02] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit (Read error: Connection reset by peer) | |
| [17:03:23] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
| [17:04:32] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has quit (Read error: Connection reset by peer) | |
| [17:04:45] | gigem_ (gigem_!~david@mythtv/developer/gigem) has quit (Remote host closed the connection) | |
| [17:04:53] | Jordack (Jordack!~jordack@h69-131-44-221.plmomi.dedicated.static.tds.net) has joined #mythtv | |
| [17:05:12] | gigem_ (gigem_!~david@host103.16.intrusion.com) has joined #mythtv | |
| [17:05:12] | gigem_ (gigem_!~david@host103.16.intrusion.com) has quit (Changing host) | |
| [17:05:12] | gigem_ (gigem_!~david@mythtv/developer/gigem) has joined #mythtv | |
| [17:10:29] | dekarl: | stuartm: are the submissions grouped by channel <-> icon or something? 70k different combinations sounds a bit high... (checking if grouping by onid:sid works would be cool. It should be valid but I've only seen data for 4 countries yet) |
| [17:10:56] | MythBuild: | build #1765 of master-linux-32bit is complete: Success [build successful] Build details are at http://code.mythtv.org/buildbot/builders/mast . . . /builds/1765 |
| [17:11:51] | stuartm: | dekarl: that's individual submissions, the actual combinations is much lower, although still unmanageably high |
| [17:14:19] | dekarl: | ah ok, then 70k isn't that much (but still alot) |
| [17:14:30] | stuartm: | the db structure and sqlite limitations makes it difficult to give stats on the number of distinct combinations |
| [17:15:23] | pheld (pheld!~heldal@cl-5.osl-01.no.sixxs.net) has quit (Quit: Leaving.) | |
| [17:15:43] | iamlindoro: | dekarl: stuartm: We have a ticket open for channel icon search that suggests it doesn't work with spaces, uncertain whether that might be somehow related to the ticket you've just opened-- purely informational |
| [17:15:46] | stuartm: | I might try to add some auto-approval logic |
| [17:17:16] | stuartm: | and sorting options to the approval screen, so that it's easier to use |
| [17:20:14] | dekarl: | iamlindoro: works ok with spaces (searching for "france 24 in english" used to return every channel that has "france" or "24" or "in" or "english" somewhere in it's name, now it returns every channel with "france" in it. good enough for me :) |
| [17:20:34] | iamlindoro: | dekarl: Yeah, I think that's what the ticket is about |
| [17:20:46] | iamlindoro: | good enough for you, but apparently not for the reporter |
| [17:20:57] | iamlindoro: | (and understandable when it's a US channel with hundreds of spinoff channels) |
| [17:21:19] | dekarl: | ohh, I'm talking about #9991 now. the data that comes down from lyngsat is mangled for channels containing "," |
| [17:35:49] | kormoc_afk is now known as kormoc | |
| [17:36:29] | Mousey (Mousey!~wtfisme@ross154.net) has joined #mythtv | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.