[beta11b] learning and unlearning shows equi/energy/etc bar

[keekooceeaou]

After learning talent category (gained from eating something ), and then some talents in it (to see their effects), I unlearned everything and exited Talents Levelup screen.
This caused Equi bar to appear permanently (even after exitting/running game again).
The same applies to Negative energy bar (hymns talent tree from quest reward).

[darkgod]

Yes ..

[Final Master]

I think what he's saying is he opened the level up talent screen, learned the category and then put points into the respective talents, then exited using escape rather than enter. Exiting with escape doesn't confirm your choices while enter does. If that is the case then that should be a bug that the bars would appear and stay.

[Feanor.81]

Uhm, I'm pretty sure that exiting hitting Escape still confirms the choices made.

[Gwai]

I can confirm that escape confirms one's choices, but I thinkn this bug happens if you right click the chosen talent to cancel also. (I can't check that because my current character does not have access to any such talents yet.)

[yufra]

DG, maybe we could use a reference counter for each pool? I have attached a patch against revision 1308.

[Feanor.81]

Yes, and IMO this is not a bug, it's just the way the talent system is implemented: bars are automatically added the first time you learn a talent which requires some kind of energy pool you don't have access to yet; and it doesn't make difference if you then unlearn the talent (i.e. talent levels are increased and decreased accordingly as you make choices in the level up screen, and changes do not take effect only after confirming and exiting the dialog).

To solve this issue there would be the need to introduce a check, triggered upon unlearning any talent, to search for other talents that are using the same energy pool; if none is found, the energy pool is removed. Not immediate, but still feasible I think.

Edit:

DG, maybe we could use a reference counter for each pool? I have attached a patch against revision 1308.

Oh, I see yufra was proceeding in the same direction while I was writing my response

[darkgod]

Thanks

[yufra]

Edit:

DG, maybe we could use a reference counter for each pool? I have attached a patch against revision 1308.

Oh, I see yufra was proceeding in the same direction while I was writing my response

Can we add a ninja class to ToME?

[keekooceeaou]

Can we add a ninja class to ToME?

And Ninja PIRATES!








Sorry, I had to.

[darkgod]

Lol, ninja are unlikely, the pirates could be when I add Umbar

[Shoob]

Well, you could say shadowblades are ninjas

[madmonk]

Lol, ninja are unlikely, the pirates could be when I add Umbar

Corsairs, yay!