Skip to content

Conversation

@PieterVdc
Copy link
Member

No description provided.

{"COST", 6, field(game.conf.magic_conf.power_cfgstats[0].cost[6]), 0, LONG_MIN,ULONG_MAX, NULL, value_default, assign_default},
{"COST", 7, field(game.conf.magic_conf.power_cfgstats[0].cost[7]), 0, LONG_MIN,ULONG_MAX, NULL, value_default, assign_default},
{"COST", 7, field(game.conf.magic_conf.power_cfgstats[0].cost[8]), 0, LONG_MIN,ULONG_MAX, NULL, value_default, assign_default},
{"POWER", -1, field(game.conf.magic_conf.power_cfgstats[0].strength), 0, LONG_MIN,ULONG_MAX, NULL, value_powercost, assign_powercost},
Copy link
Contributor

@walt253 walt253 Apr 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any support for negative value? I think most power will not work at all currently if one give them a negative power value?🤔

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

euh strength is a long, but didn't look further in to how it's used, no different then before my refactors

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah! Nevermind then, a bit strange for it to be a long but ok.

@Loobinex
Copy link
Member

I tried SET_POWER_CONFIGURATION(POWER_HEAL_CREATURE,9,10)

And I get this log error:
Error: set_power_configuration_check(line 72): Unknown power variable

@walt253 walt253 requested review from Spatulade and removed request for Spatulade July 23, 2025 22:04
@Loobinex
Copy link
Member

I tried SET_POWER_CONFIGURATION(POWER_HEAL_CREATURE,9,10)

And I get this log error: Error: set_power_configuration_check(line 72): Unknown power variable

Sorry. This is obviously because I am missing a variable in that script command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants