View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0000790LDMudLPC Languagepublic2011-10-27 11:322018-01-30 04:59
Reporter_xtian_ Assigned Tozesstra  
PrioritylowSeveritytweakReproducibilityN/A
Status resolvedResolutionfixed 
Summary0000790: officially deprecate "static" (in variables and functions)
DescriptionI thought, I'd just put this up on the agenda:

pro deprecation:
- completely different use of "static" in other languages confuses a lot of programmers
- in variables: has been replaced by "nosave" for ages
- in functions: has no real unique use. Security mechanisms are better implemented via stack-based approach (see comments in https://ldmud.eu/mantis/view.php?id=751)

con deprecation:
- mudlibs have to be changed. not a lot of programmers left.
TagsNo tags attached.
External Data (URL)

Activities

zesstra

2011-10-27 11:58

administrator   ~0002070

Pheeewww. While I am usually in favor of clean-up, I think, I am very reluctant in this case:
1) MG has about 45000 variables+function which use static. That is a lot and probably similar in other muds.
2) protected and static are not completely equivalent. There are still applications where I deliberately use static, because I want that call_other(this_object, ...) can call the function.
Also, I am actually not sure at the moment, if you can execute protected functions in this_object() with command() as it is possible with static...

zesstra

2011-10-31 13:43

administrator   ~0002071

Setting to feedback until we got feedback from several people.

_xtian_

2011-11-01 17:34

reporter   ~0002075

Yes, with less and less programmers we should not burden us with uneccesary incompatible changes, you are right.
But maybe just a comment in the documentation would be nice? (you shouldn't use this in new code)

ldmud-bugs

2014-02-23 20:23

viewer   ~0002224

Fix committed in revision 9f018133b1c1c5b473a751d862addb563e0710b0 to master branch (see changeset 934 for details). Thank you for reporting!

ldmud-bugs

2014-02-23 20:26

viewer   ~0002226

Fix committed in revision 632d68bf9427f177312b38461690923790302efd to umaster-3.3 branch (see changeset 936 for details). Thank you for reporting!

zesstra

2018-01-29 19:59

administrator   ~0002315

Fix committed in revision 632d68bf9427f177312b38461690923790302efd to master-3.3 branch (see changeset 2196 for details). Thank you for reporting!

zesstra

2018-01-29 22:57

administrator   ~0002366

Fix committed in revision 632d68bf9427f177312b38461690923790302efd to master-3.3 branch (see changeset 3540 for details). Thank you for reporting!

zesstra

2018-01-30 04:59

administrator   ~0002417

Fix committed in revision 9f018133b1c1c5b473a751d862addb563e0710b0 to master branch (see changeset 3851 for details). Thank you for reporting!

Issue History

Date Modified Username Field Change
2011-10-27 11:32 _xtian_ New Issue
2011-10-27 11:58 zesstra Note Added: 0002070
2011-10-31 13:43 zesstra Note Added: 0002071
2011-10-31 13:43 zesstra Assigned To => zesstra
2011-10-31 13:43 zesstra Status new => feedback
2011-11-01 17:34 _xtian_ Note Added: 0002075
2011-11-01 17:34 _xtian_ Status feedback => assigned
2014-02-23 20:23 ldmud-bugs Source_changeset_attached => ldmud.git master 9f018133
2014-02-23 20:23 ldmud-bugs Note Added: 0002224
2014-02-23 20:23 ldmud-bugs Assigned To zesstra => ldmud-bugs
2014-02-23 20:23 ldmud-bugs Status assigned => resolved
2014-02-23 20:23 ldmud-bugs Resolution open => fixed
2014-02-23 20:26 ldmud-bugs Source_changeset_attached => ldmud.git umaster-3.3 632d68bf
2014-02-23 20:26 ldmud-bugs Note Added: 0002226
2018-01-29 19:59 zesstra Source_changeset_attached => ldmud.git master 9f018133
2018-01-29 19:59 zesstra Source_changeset_attached => ldmud.git master-3.3 632d68bf
2018-01-29 19:59 zesstra Note Added: 0002315
2018-01-29 19:59 zesstra Assigned To ldmud-bugs => zesstra
2018-01-29 22:57 zesstra Source_changeset_attached => ldmud.git master 9f018133
2018-01-29 22:57 zesstra Source_changeset_attached => ldmud.git master-3.3 632d68bf
2018-01-29 22:57 zesstra Note Added: 0002366
2018-01-30 04:59 zesstra Source_changeset_attached => ldmud.git master 9f018133
2018-01-30 04:59 zesstra Note Added: 0002417