Comments about SmartRes

Comment #15

no-reply@curseforge.com (myrroddin) — Sun, 13 Sep 2009 22:39:36 +0000

Fixed, I hope. That was my lack of understanding the branch/tag svn commit system, sorry.

Comment #14

no-reply@curseforge.com (Pneumatus) — Sun, 13 Sep 2009 01:21:34 +0000

6.04 will not work stand-alone as it's .toc has been mangled because you merged a no-lib toc in. Arrowmaster has committed a fixed version in r117.

Also, tag the repo to make releases, don't just rename alpha zips and change the release type :<

Comment #13

no-reply@curseforge.com (myrroddin) — Sat, 11 Apr 2009 00:38:19 +0000

See my answer on Curse.com for your error, Kelem.

Comment #12

no-reply@curseforge.com (Kelem) — Sun, 05 Apr 2009 19:50:53 +0000

[2009/04/05 15:49:29-127-x1]: SmartRes-6.00\SmartRes.lua:20: Cannot find a library instance of SharedMedia-1.0. BigWigs-2.0 r5048\Libs\AceLibrary\AceLibrary.lua:490: in function `AceLibrary' SmartRes-6.00\SmartRes.lua:20: in main chunk

[2009/04/05 15:49:29-127-x1]: SmartRes-6.00\SmartResOptions.lua:3: Cannot find a library instance of SharedMedia-1.0. BigWigs-2.0 r5048\Libs\AceLibrary\AceLibrary.lua:490: in function `AceLibrary' SmartRes-6.00\SmartResOptions.lua:3: in main chunk

Comment #11

no-reply@curseforge.com (myrroddin) — Sun, 15 Feb 2009 08:46:39 +0000

Weird, it shouldn't be out of date, and I'm not seeing any such flags. SmartRes does work with WoW 3.0.x and higher. I will check again after 3.10 comes out to see if there are any API changes.

Plus I am still working on SmartRes2 in the little spare time I have. SR2 will eventually replace the original SR.

Comment #10

no-reply@curseforge.com (Kelem) — Sun, 01 Feb 2009 02:27:24 +0000

Addon still shows out of date, so that might be some peoples issue.

Comment #9

no-reply@curseforge.com (myrroddin) — Sun, 23 Nov 2008 03:58:09 +0000

People might have seen the Release version up. I get no errors at all, which is why I tagged r103 as a Release. Everything works, including Revive and Death Knights, the keybindings to FuBar.

Of course, you can't auto-res someone as Blizzard won't allow addons to target, and haven't since about patch 1.10 or so. I use the manual target for released players, and the auto-res key for normal corpses, and in both cases casts the appropriate res spell on them and puts the bars on my screen.

Comment #8

no-reply@curseforge.com (myrroddin) — Sun, 16 Nov 2008 17:04:30 +0000

Zidomo, thank you very much! I've been having too much fun with WotlK to code atm, but I'll get back to it soon enough, and check out what you fixed :-)

The other thing I was thinking of doing was rewriting to use LibResComm, but we will see.

I did notice that Ace2 has been updated, so that makes it a bit easier, but yes, upgrading libraries is a good idea.

Comment #7

no-reply@curseforge.com (Zidomo) — Mon, 10 Nov 2008 11:17:41 +0000

Uploaded r100 a short while ago. Has Druid Revive & Death Knight support like you had intended to put in, Myrroddin, but its actually tested & working now.

Removed Babble-Spell-2.2 & replaced it with GetSpellInfo info. Basically used your code for the GetSpellInfo stuff, Myrroddin, but removed apostrophes that caused problems. As well, didn't add in your other changes that broke the mod (its currently all Ace2/old libraries).

Once WotLK arrives this week and SmartRes continues to work without errors in it, I'll bump the TOC and tag it for release. Unless you'd like to do that Myrroddin :).

Yes, this should be updated to Ace3/LibBars/etc.. But until working code is provided by someone (I'm a professional software tester by trade, not a coder), may as well continue to use what works now.

Comment #6

no-reply@curseforge.com (Zidomo) — Fri, 07 Nov 2008 16:59:31 +0000

"Alpha2" (which is actually the working circa-WoW 2.4.3 r87) does not contain the new Revive spell. Only Priest Resurrection, Shaman Ancestral Spirit & Paladin Redemption (the three OOC reses pre-WoW 3.0). I may add it in when I have a level 12+ Druid to test on.

And a note: the only reason I rolled back your non-working changes to the previous version (in Alpha2), Myrroddin, was because that earlier version still works in WoW 3.0.3 live.

If the earlier version r87 (the last one by Poull) was also broken, I would not have done anything ;). But as it works without issues for many people (including in my testing), decided it was better to have a working version out there than one that didn't work at all.

GL with your changes; hope you are able to update it (properly) in the future :).