| Friday, May 14th, 2010, 00:06 UTC | ||
| [00:06:03] | abqjp (abqjp!~abqjp@97-119-180-26.albq.qwest.net) has joined #mythtv-theming | |
| [00:30:15] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has quit (Ping timeout: 260 seconds) | |
| [00:35:38] | gbutters (gbutters!~gbutters@67.217.215.18) has joined #mythtv-theming | |
| [00:52:53] | gbutters (gbutters!~gbutters@67.217.215.18) has quit (Ping timeout: 252 seconds) | |
| [00:57:23] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has joined #mythtv-theming | |
| [01:06:03] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has quit (Ping timeout: 240 seconds) | |
| [01:11:40] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has joined #mythtv-theming | |
| [02:24:37] | gregl (gregl!~greg@cpe-69-204-191-116.nycap.res.rr.com) has quit (Quit: Leaving) | |
| [03:30:52] | mrand (mrand!~mrand@ubuntu/member/mrand) has left #mythtv-theming () | |
| [04:23:00] | mrand (mrand!~mrand@ubuntu/member/mrand) has joined #mythtv-theming | |
| [06:38:37] | brfransen (brfransen!~brfransen@adrianDHCP-47.216-254-250.iw.net) has quit (Quit: brfransen) | |
| [07:34:56] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has quit (Ping timeout: 240 seconds) | |
| [07:36:37] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has joined #mythtv-theming | |
| [07:43:07] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has quit (Ping timeout: 248 seconds) | |
| [07:47:54] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has joined #mythtv-theming | |
| [07:56:26] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has quit (Ping timeout: 240 seconds) | |
| [07:56:44] | gbutters (gbutters!~gbutters@ip98-163-245-35.no.no.cox.net) has joined #mythtv-theming | |
| [08:46:43] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has joined #mythtv-theming | |
| [11:17:08] | Kew (Kew!~kew@4304ds1-hs.0.fullrate.dk) has quit (Ping timeout: 252 seconds) | |
| [11:56:23] | Kew (Kew!~kew@4304ds1-hs.0.fullrate.dk) has quit (Ping timeout: 260 seconds) | |
| [11:56:44] | stuartm (stuartm!~gbee@cpc2-derb9-0-0-cust872.leic.cable.ntl.com) has joined #mythtv-theming | |
| [11:56:44] | Mode for #mythtv-theming by ChanServ!ChanServ@services. : +v stuartm | |
| [11:56:46] | gbee (gbee!~gbee@cpc2-derb9-0-0-cust872.leic.cable.ntl.com) has quit (Ping timeout: 245 seconds) | |
| [13:37:31] | brfransen (brfransen!~brfransen@adrianDHCP-47.216-254-250.iw.net) has joined #mythtv-theming | |
| [13:50:38] | gbutters: | iamlindoro: Will you apply this for me and backport it to fixes? http://mythbuntu.pastebin.com/p5q0U5Rp |
| [13:51:05] | iamlindoro: | gbutters, ah, heh, yeah, sure |
| [13:52:07] | iamlindoro: | gbutters, done |
| [13:52:23] | gbutters: | iamlindoro: thank you |
| [13:52:41] | iamlindoro: | no problem |
| [14:08:04] | gregl (gregl!~greg@cpe-69-204-191-116.nycap.res.rr.com) has joined #mythtv-theming | |
| [14:50:44] | psipsi_ is now known as psipsi | |
| [14:57:01] | psipsi (psipsi!~psipsi@rrcs-24-227-36-210.se.biz.rr.com) has quit (Read error: Connection reset by peer) | |
| [15:10:47] | iamlindoro: | stuartm: Better not make any fixes, nobody's allowed to touch, look at, or think about anything that is vaguely audio related ATM under penalty of whiny e-mail and IRC sulking |
| [15:18:38] | stuartm: | heh |
| [15:35:22] | simonckenyon (simonckenyon!~kvirc@195.7.61.12) has quit (Remote host closed the connection) | |
| [16:51:42] | iamlindoro: | OK, I'm convinced, MythVideo is getting a metadata map, I can't stand to have it fall behind themingwise :) |
| [16:52:38] | stuartm: | heh |
| [16:54:11] | stuartm: | http://svn.mythtv.org/trac/changeset/24667 |
| [17:11:40] | iamlindoro: | UI in MythVideo makes me sad |
| [17:11:47] | iamlindoro: | Someday I'll be smart enough to fix it all |
| [17:30:38] | stuartm: | :/ |
| [17:31:46] | iamlindoro: | stuartm: Just the weird non-standard mapping of values/confusing namespace things/CopyMetadataToUI stuff |
| [17:32:14] | iamlindoro: | It's just literally the only place in all of Myth that does what it does, and seems deliberately confusing at times |
| [17:32:48] | Captain_Murdoch: | iamlindoro, just rewrite it when you move it into programs/mythfrontend. :) |
| [17:33:02] | stuartm: | iamlindoro: ah, yeah, I hate it, I don't really understand why Anduin insists on obfuscating everything in pointless wrappers etc |
| [17:33:09] | iamlindoro: | heh |
| [17:33:30] | iamlindoro: | well using the map will allow me to rip out a fair amount, and I'm moving a bunch of the random helper stuff to videoutils.cpp |
| [17:33:41] | stuartm: | Captain_Murdoch: I re-wrote it already, but Anduin just undid it all the moment I was finished |
| [17:34:28] | Captain_Murdoch: | ah. |
| [17:34:41] | iamlindoro: | Well he doesn't seem to have much investment in it these days, so maybe we can get it shaped up again |
| [17:35:10] | iamlindoro: | s/we/I/ |
| [17:35:15] | Captain_Murdoch: | was a pain to get the triggered fanart loading working in there compared to how easy it was in Watch Recordings. |
| [17:37:31] | stuartm: | I respect the work Anduin does, I just don't like the way he does it, everything seems to add a layer of complexity that serves no apparent purpose and makes maintaining and extending the code a huge pain |
| [18:11:38] | iamlindoro: | Untangling this is not easy at all-- Think the text map is probably working now, but the handling of images needs a rewrite-- on the plus side, this experience should make applying the same principles to MythNetvision easy |
| [18:11:56] | iamlindoro: | And having Music/Video/Netvision all using maps sets a good example for plugin authors |
| [18:14:25] | mrand: | Heh. The way you all are talking about confusing names and unneeded wrappers and such is the way I feel about C++ ;-) |
| [18:19:06] | iamlindoro: | There's no reason C++ can't be perfectly easy to read |
| [18:19:19] | iamlindoro: | variable and function naming doesn't have to be insane |
| [18:19:49] | stuartm: | mrand: maybe you've seen a few bad examples ;) Anyone seeing mythvideo might be put off for life, but it's not representative of C++ |
| [18:20:44] | stuartm: | iamlindoro: another pet hate of mine is poor variable naming and in particular, one letter names, e.g. s for string or e for event |
| [18:21:18] | iamlindoro: | I try to kepp my variables to plain english nouns, my functions to describe what it does |
| [18:21:26] | iamlindoro: | though I'm no expert, it makes sense to me |
| [18:21:37] | stuartm: | i for iterator is tolerable, but only when it's a numeric iterator in a small (readable) loop |
| [18:22:52] | stuartm: | iamlindoro: that's exactly how it should be, unfortunately not always how it's taught in schools/universities or practised in companies |
| [18:23:19] | iamlindoro: | Heh, I still have no idea what "CheckedSet" is supposed to mean |
| [18:23:59] | stuartm: | :p Sometimes it's hard to provide a clear and concise name for things |
| [18:24:10] | mrand: | exactly my point... EvenReallyLongNames don't help me most of the time because stuff is so abstracted. |
| [18:24:18] | stuartm: | I'm sure there are cases in mythui where that's the case, though I can't take the blame for all of them |
| [18:24:45] | mrand: | I'm not talking about myth code in general... I'm generalizing about about most C++. |
| [18:24:52] | iamlindoro: | stuartm: No, this is an anduin function in MythVideo |
| [18:25:11] | iamlindoro: | It's his Handle-All-UI-types function |
| [18:25:50] | stuartm: | iamlindoro: ah, ok, it seemed familiar so I thought it might be one of my ugly creations |
| [18:26:37] | iamlindoro: | It's an overloaded function for each ui type that takes the item, resets it, and loads the new one |
| [19:11:53] | iamlindoro: | I think I am going to cheat this a little-- I'll let the existing stuff go on handling images and statetypes and just move the text stuff into the map/handle it in a more straightforward way. Makes the conversion much simpler. |
| [19:13:53] | iamlindoro: | Once I can figure out how to extricate the rest I'll finish the conversion |
| [19:39:50] | stuartm: | not everything is necessarily suited to being in the map, in ProgramInfo/MythMusic it's just the metadata and IMHO that works well |
| [19:40:57] | iamlindoro: | stuartm: Yeah, I'd just like to exorcise the complicated code that confuses me to this date |
| [19:41:00] | iamlindoro: | er to this day |
| [19:41:27] | iamlindoro: | It'd be nice for someone who understands any of the other plugins to have some idea what was going on in MV |
| [19:46:41] | stuartm: | at the risk of banging on about this too often, if there is overlap between the three then we should re-use the same names for things, e.g. 'lenmins/lentime' is used in ProgramInfo for the length/runtime, mythmusic currently uses 'length' and mythvideo 'runtime' |
| [19:58:58] | iamlindoro: | stuartm: I spent a good deal of time aligning them before .22, that one may just have escaped me |
| [19:59:11] | iamlindoro: | ie all the plot->description, etc. |
| [19:59:26] | iamlindoro: | modified mythgame too, and modeled netvision after all the existing ones |
| [20:00:50] | stuartm: | yeah, I remember, and it's not a criticism, just a reminder to everyone that we should try to harmonise things as much as possible |
| [20:01:30] | iamlindoro: | If you see those, please feel free to let me know and I am happy to make changes |
| [21:18:54] | i_is_cat (i_is_cat!~i_is_cat@S010600179a22e379.ed.shawcable.net) has quit (Remote host closed the connection) | |
| [21:20:36] | gbee_ (gbee_!~gbee@cpc2-derb9-0-0-cust872.leic.cable.ntl.com) has joined #mythtv-theming | |
| [21:20:37] | Mode for #mythtv-theming by ChanServ!ChanServ@services. : +v gbee_ | |
| [21:20:49] | stuartm (stuartm!~gbee@cpc2-derb9-0-0-cust872.leic.cable.ntl.com) has quit (Ping timeout: 265 seconds) | |
| [21:24:01] | gbee_ is now known as gbee | |
| [21:34:10] | iamlindoro: | Will need to write a treatise on templates/maps when I get these commits in |
| [21:34:19] | iamlindoro: | That's a spot I keep meaning to get to in the docs |
| [21:35:37] | iamlindoro: | Or at least put a heading on the tables I have the mapped values in indicating that they can be used in templates with one another |
| [22:46:30] | iamlindoro: | Woo hoo: http://www.fecitfacta.com/templatemap.png |
| [22:47:01] | iamlindoro: | Now if I can just get it to clear properly when there's no metadata I'll be in business |
| [23:33:26] | rooaus (rooaus!~cameron@ppp118-209-130-8.lns20.mel6.internode.on.net) has left #mythtv-theming () | |
| [23:35:46] | rooaus (rooaus!~cameron@ppp118-209-130-8.lns20.mel6.internode.on.net) has joined #mythtv-theming | |
| [23:39:49] | iamlindoro: | gbee, I don't suppose there's an easy way to reset all textareas on a screen? |
| [23:48:39] | iamlindoro: | Want to be able to clear all the textareas when a folder is selected, but the only way I have found so far is to send the tomap function a null metadata object, which has its own problems |
| [23:58:26] | natanojl (natanojl!~jonatan@c83-252-237-63.bredband.comhem.se) has quit (Ping timeout: 240 seconds) | |
IRC Logs collected by
BeirdoBot.
Please use the above link to report any bugs.
