IRC log for #arl on 20090524

00:00.15TorhalPOOPY!
00:01.54ZhinjioTorhal!
00:03.18Zhinjioyou didn't by any chance figure out why I'm getting memcreep in those two addons, eh?
00:16.50TorhalOh, I didn't delve too far yet. I was doing other stuff with the libs
00:18.51TorhalZhinjio: Next commit will fix clipping of the bottoms of letters such as "g" and "p", as well as introduce two more optional SetCell() arguments - padding and maxwidth
00:19.21TorhalWhen you set a maxwidth, if your text is longer than the width it will wrap.
00:19.58TorhalBut I definitely wanna see why those two addons are blowing QTip up like a fucking blimp.
00:20.47TorhalAckis: Did you want me to make a new section in CHANGES.txt for RC9?
00:44.46ZhinjioI was wondering about the clipping
00:48.51ZhinjioI'll probably use padding regularly
00:48.57Zhinjiothinks about the mem bloat
00:49.04Zhinjiohopefully, I'll get time to look at that tonight
01:49.36TorhalZhinjio: I've significantly reduced the impact...I'm going psycho on the lib's caching mechanics. There's still a leak somewhere, though.
01:49.50TorhalZhinjio: Also, you're not releasing the tooltip with AllXP.
01:49.53Torhal:D
01:50.51TorhalI've gotten it down from ~1Kib or more gain to exactly 0.539KiB gain.
01:51.01TorhalWhich is still fucking huge.
01:52.25TorhalThough, that's apparently with every LDB plugin that uses LQT so you're out of the danger zone
01:52.30TorhalLeak on my side.
01:55.02TorhalZhinjio: Ok, update: Broker_Extractor and Broker_AllXP are acquiring their tooltip when loaded, and never releasing it.
01:56.40Zhinjionods.
01:57.00ZhinjioI seem to recall we had discussed this, and you said it wasn't necessary really, since release wasn't ever truly releasing anyway
01:57.05Zhinjiobut I guess you've fixed that now?
01:57.09TorhalIndeed.
01:57.22TorhalI'm hammering down all the leaks now.
01:57.45TorhalYour other QTip addons acquire/release normally, though, so...:D
02:17.47ZhinjioI wrote my code taking into account all the ways it was broken!
02:18.00Zhinjionow you have to go all crazy and FIX stuff. Fiend.
02:30.06TorhalHaha
02:33.06Zhinjioalright, so how about qtc?
02:33.14Zhinjioyou said I shouldn't bother releasing there too, right?
02:33.42Zhinjioor something about hiding doing the release for you, or something
02:41.14TorhalAutoHide releases.
02:41.37TorhalYou could just set autohide on Extractor and AllXP if you wanted.
02:41.49TorhalGet rid of your mouseover checks.
02:42.11TorhalI've pushed the new lib versions, BTW.
02:43.05ZhinjioI don't use qtc on anything except SKG (yet)
02:43.22TorhalAutoHide is in the core.
02:43.29Zhinjioaha
02:43.31TorhalQTC just inherits it
02:43.35ZhinjioI see
02:44.35TorhalI just severely limited memory growth by adding table caches. Yay me.
02:44.56TorhalThough there's still that annoying 0539KiB growth every time you Acquire.
02:45.08Torhals/0539/0.539/
02:45.27TorhalGC snatches it up, but I'd rather it not be there at all.
02:51.24Zhinjionods.
04:34.24TorhalPoopy?
04:43.33Zhinjioheh
04:46.12TorhalAckis: You, sir, are an idle bastard.
04:48.40Zhinjiohe's too busy collecting crap on facebook
04:48.42Zhinjiogrins.
04:48.55TorhalI thought, perhaps, he found a way to lick himself.
04:49.54Ackishuh?
04:49.59TorhalBWAH!
04:50.08Ackisif i found a way to licm myself you'd never see me again
04:50.16TorhalI asked you a few hours ago: Did you want me to add an RC9 entry in CHANGES.txt?
04:50.18TorhalHeh
04:50.21Ackisyes pls
04:50.25Torhalkk
05:18.49*** join/#arl pompy (n=Mike@c-68-38-45-3.hsd1.nj.comcast.net)
05:18.49*** mode/#arl [+v pompy] by ChanServ
06:03.36TorhalIs it sad when code that is meant to ensure a full library upgrade breaks the upgrade path?
06:03.40Torhalcackles.
06:07.05Zhinjiosomething tells me I am really happy to wait a bit before relinking new library versions
06:07.23TorhalNah
06:07.35TorhalAdirelle added some code which I told him was unnecessary, and it broke shit
06:07.44TorhalSo I commented it out in case he wanted to fix it
06:07.46TorhalHeh
06:08.49TorhalI switched back to the single heap system, and he added code to make sure the two heaps were reclaimed. I told him there shouldn't be addons with active tooltips at that point, and if there were a few orphaned frames wouldn't hurt anything.
06:08.56*** join/#arl BWMerlin (n=chatzill@58.174.148.250)
06:08.56*** mode/#arl [+v BWMerlin] by ChanServ
06:09.03Torhalshrugs.
07:21.22Zhinjiodo we need to sell tickets to the Adirelle v Torhal cage match?
07:28.19pompylol ut oh
08:15.17*** join/#arl Jim-Bim (n=thorben_@mnsr-4db1617d.pool.einsundeins.de)
08:15.17*** mode/#arl [+o Jim-Bim] by ChanServ
09:17.24*** join/#arl BWMerlin (n=chatzill@58.174.148.250)
09:17.24*** mode/#arl [+v BWMerlin] by ChanServ
09:42.16BWMerlinin the ARL recipe database
09:42.44BWMerlinif a trainer is missing from a recipe do i need to add it in order or just tack it onto the end
09:46.58pompyum, doesnt matter what order
09:50.50BWMerlincool
09:51.14BWMerlini havent set up svn access to ARL so ill just be happy to break my copy
09:52.33pompyfeel free to keep submitting more tickets of trainer scans when u come across them, ima fix a bunch today.
10:13.45BWMerlinwill do
10:13.54BWMerlini dont play much atm
12:57.50*** join/#arl aesir (n=Aesir@24.238.95.142.res-cmts.sth.ptd.net)
12:57.50*** mode/#arl [+v aesir] by ChanServ
15:06.25TorhalZhinjio: :P
16:03.19*** join/#arl Ackis (n=asdf@WoWUIDev/WoWAce/ARL/Troll/Ackis)
16:03.19*** mode/#arl [+o Ackis] by ChanServ
16:26.20pompyTorhal: ping
16:27.12Torhalpompy: Ponglets
16:27.54pompyhttp://www.wowinterface.com/downloads/info11020-BrokerFactions.html#comments    http://www.wowace.com/projects/arl/tickets/606-update-lib-qtip-1-0/
16:28.47Torhalpompy: Factions needs to update LibQTip.
16:28.57TorhalNewest revision squashes that bug.
16:29.09pompylol yeah i assumed that
16:29.29TorhalAdirelle added code for the lib upgrade path which broke the upgrade path.
16:29.31TorhalHeh
16:29.31Ackisyah I saw that ticket myself
16:29.52Ackisi was like let my minions handle it! :D
16:29.53pompyi jus wanted to link it to u since ur the author, u can inform them ;p
16:29.56pompyha
16:30.02pompyu see teh addonloader one ackis?
16:30.08Ackisyah
16:30.25Ackisdon't really care
16:30.29pompyi knew it
16:30.32pompylolol
16:30.43pompy;)
16:30.45Torhalpompy: What's the issue there?
16:31.13pompyjust was pointing out the ticket someone posted
16:32.33pompyah nm, tinyu and kastorz looks like they informed them lol
16:33.13Ackiswhat do you guys think  about addonloader?
16:33.17TorhalI use it.
16:33.29Ackisis it good?
16:33.33pompyi havent messed with it what so ever
16:33.34TorhalYah, if used right.
16:33.50pompyshould i just put in a comment to that ticket.. 'see tinyu and kastorz comment on wowi" and then close the ticket? lol
16:33.58TorhalLike, you can make it so that ARL never loads if the user never opens their tradeskill window.
16:34.12Ackisyou can do that w/o addonloader
16:34.21AckisI had it done before and it fucked shit up w/ the scan button
16:34.47TorhalIt's easier with AddonLoader. Which is why there _is_ an AddonLoader. :)
16:34.57pompylol
16:35.13Ackisloadwith: tradeskillui == fail
16:35.19Ackisaddonloader shit fucked stuff up too
16:35.43TorhalYou gotta do it right You can't just slap "support" in and expect it to work as intended.
16:36.03TorhalAnywho, both QTip tooltips acquire when needed and release when done.
16:36.17Ackisthat's about the extent of the effort I want to put into it
16:36.21TorhalHeh
16:36.30pompyackis is lazy, remember that ;p
16:36.35Ackisno pompy
16:36.38Ackisthere's just no need
16:36.49AckisARL only does work when the tradeskill window is opened up
16:36.52Ackisand that's minimal
16:36.56pompyyeahh
16:37.00Ackis99.999% of the work is done when you press the button
16:37.44TorhalARL made me change LibQTipClick-1.1
16:37.44Ackisit's the same reason why I'm not disembedding it
16:37.54Ackisis that a good or bad thing? :(
16:38.01pompythe libs?
16:38.26TorhalIt's good. The highlighter wasn't working, so I changed it to use the highlighter frame directly instead of its texture, and set its strata to TOOLTIP
16:38.35TorhalWorks fine now.
16:39.02Ackisahh cool
16:39.41TorhalAnywho: Some people want AddonLoader because they don't want ARL using 2MB RAM unless they're actively using it.
16:39.50TorhalI personally don't give a shit
16:39.57TorhalBut I can understand if they use a gimp system
16:40.08pompyAckis: shjould i jus say what u pretty much said about addonloader support for arl, or u wanna handle that?
16:40.39Ackismeh if one of you guys want to add support, all the power to you, however I just don't feel it's worth the effort
16:40.41TorhalIf you absolutely don't want support, cool. Otherwise, I might look at it.
16:40.49TorhalOk. I may, I may not.
16:40.53pompylol
16:41.05TorhalMight be edumacational.
16:41.07Torhal:D
16:41.09Ackisheh
16:41.20Ackisit's just like splitting up the databases, not worth the effort
16:42.07Ackis@project arl
16:42.08RepoAckis: http://www.wowace.com/projects/arl/. Ackis Recipe List. Game: WoW. Ackis (Manager/Author), Jim-Bim (Author), Torhal (Author), Zhinjio (Author), Elkano (Tester), AesirRising (Ticket Manager/Documenter/Translator), BWMerlin (Ticket Manager/Documenter), pompachomp (Ticket Manager/Documenter/Translator). Updated: 77 seconds ago. Tickets: 22/608
16:43.22pompyAckis: jw, how come u put translator for me? lol
16:43.43Ackisso you can access the local thing
16:43.48pompyahh
16:46.20TorhalOk, gonna take my boy to see some waterfalls. Latah.
16:46.31Ackislol have fun
17:00.33aesirtorhal: qtip r87 http://dev.pastey.net/114810-3ann  Just hovering down the list of recipes.  Is something like this specific to a particular recipe (Do I need to try to find the one that trips it up)?
17:01.42pompyaesir: is it easy to reproduce?
17:02.37aesirDon't know.  I have to clear the swatter log between each error, as they get hidden after the first, so I may only see it once a session.
17:03.08aesirthat's sort of why I'm asking if I should try to drill down to a unique recipe/tradeskill in the list.
17:04.55pompyi believe r88 fixed that
17:05.11pompy(5/24/2009 12:45:56 PM) Repo: libqtip-1-0: Torhal * r88 LibQTip-1.0.lua:
17:05.11pompy(5/24/2009 12:45:59 PM) Repo: In ReleaseTooltip(): Remove the tooltip from the layoutCleaner registry.
17:05.33pompypossibly
19:13.11pompyAckis: ping
21:03.17*** join/#arl Kraftman (n=kraftman@81.96.207.69)
21:03.23Kraftman^^
21:38.43Torhalaesir: It's fixed in r88
21:39.24aesirgot it thanks. And thanks for the responses Pompyachompy.
21:42.50pompylol
21:42.52pompynp
21:43.56pompyhmm i wonder if http://www.wowhead.com/?npc=14401 should be added to libbabbleboss
22:20.56Zhinjioyo
22:23.29*** join/#arl BWMerlin (n=chatzill@58.174.148.250)
22:23.29*** mode/#arl [+v BWMerlin] by ChanServ
22:28.14TorhalZhinjio: YO!
23:41.29BWMerlinpompy:
23:42.02BWMerlinhttp://pastebin.com/d3d895b3f
23:42.07BWMerlinyou broke ARL
23:42.42pompyit didnt break it
23:43.13pompyyou dnot see the mobs name in the tooltip?
23:43.20pompydon't*

Generated by irclog2html.pl Modified by Tim Riker to work with infobot.