Token Change Proposals for 5.13 Alpha - Revision history

Karianna at 14:25, 4 August 2008

2008-08-04T14:25:27Z

Karianna at 13:08, 23 July 2008

2008-07-23T13:08:45Z

Karianna at 13:07, 23 July 2008

2008-07-23T13:07:57Z

Karianna at 13:02, 23 July 2008

2008-07-23T13:02:34Z

Karianna at 12:49, 23 July 2008

2008-07-23T12:49:28Z

Karianna at 12:48, 23 July 2008

2008-07-23T12:48:47Z

Karianna at 12:48, 23 July 2008

2008-07-23T12:48:32Z

Karianna at 12:46, 23 July 2008

2008-07-23T12:46:25Z

Karianna at 12:44, 23 July 2008

2008-07-23T12:44:44Z

Karianna at 12:44, 23 July 2008

2008-07-23T12:44:25Z

@@ Line 96: / Line 96: @@
 '''Also, there have been various discussions about PRERACETYPE and finishing that cleanup:'''
-[http://tech.groups.yahoo.com/group/pcgen_experimental/message/6281]
+[http://tech.groups.yahoo.com/group/pcgen_experimental/message/6281 http://tech.groups.yahoo.com/group/pcgen_experimental/message/6281]
 The problem is that PRERACETYPE internally references deprecated code (getCritterType()) implying that the structure needs to change. So I need to understand more about whether this can be elimianted in favor of PRERACE:1,RACETYPE= or whether another solution is required.

@@ Line 19: / Line 19: @@
 * Complicated: This is a proposed change that eliminates complicated behavior
-You may also wish to see:
+'''You may also wish to see:'''
-[[Token Change Proposals for 5.15 Alpha|5.15 Alpha Token Change Proposals]]
+* [[Token Change Proposals for 5.15 Alpha|5.15 Alpha Token Change Proposals]]
 '''The following are the actual proposed changes to tokens:'''
@@ Line 94: / Line 94: @@
 |}
-Also, there have been various discussions about PRERACETYPE and finishing that cleanup:
+'''Also, there have been various discussions about PRERACETYPE and finishing that cleanup:'''
-http://tech.groups.yahoo.com/group/pcgen_experimental/message/6281
+[http://tech.groups.yahoo.com/group/pcgen_experimental/message/6281]
 The problem is that PRERACETYPE internally references deprecated code (getCritterType()) implying that the structure needs to change. So I need to understand more about whether this can be elimianted in favor of PRERACE:1,RACETYPE= or whether another solution is required.
-Additional items for 5.13, issues being worked on, solutions TBD:
+'''Additional items for 5.13, issues being worked on, solutions TBD:'''
-) Retroactivity, much of this issue is discussed in this thread: http://tech.groups.yahoo.com/group/pcgen_experimental/message/6864
+# Retroactivity, much of this issue is discussed in this thread: http://tech.groups.yahoo.com/group/pcgen_experimental/message/6864
-) Prerequisite behavior, this is surfaced starting at this message: http://tech.groups.yahoo.com/group/pcgen_experimental/message/8133
+# Prerequisite behavior, this is surfaced starting at this message: http://tech.groups.yahoo.com/group/pcgen_experimental/message/8133
-) A 6.0 compatibility checker (built out of the CDOM branch) must be completed.
+# A 6.0 compatibility checker (built out of the CDOM branch) must be completed.

@@ Line 1: / Line 1: @@
 '''The following are the philosophies underlying these token change proposals for the 5.13 cycle:'''
-(1) Elimination of magical values: Values like -1 that represent unlimited should be replaced by "UNLIMITED" or a String otherwise explicitly identifying the meaning of the special value.
+# Elimination of magical values: Values like -1 that represent unlimited should be replaced by "UNLIMITED" or a String otherwise explicitly identifying the meaning of the special value.
-(2) Elimination of duplicate code by consolidating tokens
+# Elimination of duplicate code by consolidating tokens
-(3) Completion of Ability Object changes
+# Completion of Ability Object changes
-(4) Preparation for CDOM parsing of tokens
+# Preparation for CDOM parsing of tokens
-(5) Performance (in one or two cases)
+# Performance (in one or two cases)
-(6) Elimination of complicated, undocumented interactions
+# Elimination of complicated, undocumented interactions
 '''The following are the definitions of the Reason in the token change proposals:'''
-Syntax: This is a proposed syntax change to improve clarity and eliminate a magical value that is not clearly identified
+* Syntax: This is a proposed syntax change to improve clarity and eliminate a magical value that is not clearly identified
-Consistency: This is a proposed change to improve consistency between tokens
+* Consistency: This is a proposed change to improve consistency between tokens
-Duplicate: This is a proposed change to eliminate an alternate way of achieving an identical result
+* Duplicate: This is a proposed change to eliminate an alternate way of achieving an identical result
-Upgrade: This is a proposed change based on upgrades made to PCGen to keep the application consistent
+* Upgrade: This is a proposed change based on upgrades made to PCGen to keep the application consistent
-Nonfunctioning: This is a proposed change due to the token not functioning in the current rev of PCGen
+* Nonfunctioning: This is a proposed change due to the token not functioning in the current rev of PCGen
-Unnecessary: This is a token that provides insights, but requires no processing by the code
+* Unnecessary: This is a token that provides insights, but requires no processing by the code
-Performance: This is a proposed change that will reducing parsing complexity and thus improve speed
+* Performance: This is a proposed change that will reducing parsing complexity and thus improve speed
-Complicated: This is a proposed change that eliminates complicated behavior
+* Complicated: This is a proposed change that eliminates complicated behavior
 You may also wish to see:

@@ Line 25: / Line 25: @@
 {| style="width:100%" border="1"
-!width="50"|Token LST File !! Status !! Token Name !! Current Syntax !! Syntax Change !! Future Syntax !! Reason !! Justification for Change !! Discussion/Approval/FREQ
+!width="50"|Token LST File
 |-
 |All || SVN 3034 || All Relevant tokens || TOKEN:x|PRExxx|x || Disallow embedded PRE, require PRE at end || TOKEN:x|x|PRExxx || Consistency, Performance || Clarity and error checking ||

@@ Line 25: / Line 25: @@
 {| style="width:100%" border="1"
-! Token LST File !! Status !! Token Name !! Current Syntax !! Syntax Change !! Future Syntax !! Reason !! Justification for Change !! Discussion/Approval/FREQ
+!width="50"|Token LST File !! Status !! Token Name !! Current Syntax !! Syntax Change !! Future Syntax !! Reason !! Justification for Change !! Discussion/Approval/FREQ
 |-
 |All || SVN 3034 || All Relevant tokens || TOKEN:x|PRExxx|x || Disallow embedded PRE, require PRE at end || TOKEN:x|x|PRExxx || Consistency, Performance || Clarity and error checking ||

← Older revision		Revision as of 12:48, 23 July 2008
Line 23:		Line 23:

	'''The following are the actual proposed changes to tokens:'''		'''The following are the actual proposed changes to tokens:'''
−
−
−	~~{\| border="1" cellpadding="2"~~
−	~~!width="50"\|Name~~
−	~~!width="225"\|Effect~~
−	~~!width="225"\|Games Found In~~
−
−

	{\| style="width:100%" border="1"		{\| style="width:100%" border="1"

@@ Line 24: / Line 24: @@
 '''The following are the actual proposed changes to tokens:'''
 {| border="1"
 ! Token LST File !! Status !! Token Name !! Current Syntax !! Syntax Change !! Future Syntax !! Reason !! Justification for Change !! Discussion/Approval/FREQ
 |-
@@ Line 33: / Line 41: @@
 | Race || SVN 3029 || '''MFEAT''' || MFEAT:x || Deprecate || none || Removed || MFEAT was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
 |-
-| Race || SVN 3029 || '''HITDICE** || HITDICE:x || Deprecate || none || Removed || HITDICE was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
+| Race || SVN 3029 || '''HITDICE''' || HITDICE:x || Deprecate || none || Removed || HITDICE was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
 |-
 | All || SVN 3029 || '''PREDEFAULTMONSTER''' || PREDEFAULTMONSTER:x || Deprecate || none || Removed || PREDEFAULTMONSTER was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]

@@ Line 29: / Line 29: @@
 |All || SVN 3034 || All Relevant tokens || TOKEN:x|PRExxx|x || Disallow embedded PRE, require PRE at end || TOKEN:x|x|PRExxx || Consistency, Performance || Clarity and error checking ||
 |-
-| Race || SVN 3029 || !INIT || INIT:x || Deprecate || BONUS:COMBAT|INITIATIVE|x || Nonfunctioning, Duplicate || INIT: does not function in the code || [[http://sourceforge.net/tracker/index.php?func=detail&aid=1696679&group_id=25576&atid=384722|FREQ 1696679]]
+| Race || SVN 3029 || '''INIT''' || INIT:x || Deprecate || BONUS:COMBAT|INITIATIVE|x || Nonfunctioning, Duplicate || INIT: does not function in the code || [[http://sourceforge.net/tracker/index.php?func=detail&aid=1696679&group_id=25576&atid=384722|FREQ 1696679]]
 |-
-| Race || SVN 3029 || **MFEAT** || MFEAT:x || Deprecate || none || Removed || MFEAT was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
+| Race || SVN 3029 || '''MFEAT''' || MFEAT:x || Deprecate || none || Removed || MFEAT was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
 |-
-| Race || SVN 3029 || **HITDICE** || HITDICE:x || Deprecate || none || Removed || HITDICE was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
+| Race || SVN 3029 || '''HITDICE** || HITDICE:x || Deprecate || none || Removed || HITDICE was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
 |-
-| All || SVN 3029 || **PREDEFAULTMONSTER** || PREDEFAULTMONSTER:x || Deprecate || none || Removed || PREDEFAULTMONSTER was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
+| All || SVN 3029 || '''PREDEFAULTMONSTER''' || PREDEFAULTMONSTER:x || Deprecate || none || Removed || PREDEFAULTMONSTER was part of the Default Monster Code || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1613771&group_id=25576&atid=384722|FREQ 1613771]]
 |-
-| Class || SVN 3029 || **BAB** || BAB:x || Deprecate || BONUS:xxx || Nonfunctioning, Duplicate || BAB: is present in the code but will always fail (not valid today) ||
+| Class || SVN 3029 || '''BAB''' || BAB:x || Deprecate || BONUS:xxx || Nonfunctioning, Duplicate || BAB: is present in the code but will always fail (not valid today) ||
 |-
-| Global || SVN 3032 || **REMOVE:FEAT** || REMOVE:FEAT(x,x)n || Change to | syntax like ADD || REMOVE:FEAT|[x|]y,y || Consistency || This was not changed for consistency when ADD and AUTO were updated ||
+| Global || SVN 3032 || '''REMOVE:FEAT''' || REMOVE:FEAT(x,x)n || Change to | syntax like ADD || REMOVE:FEAT|[x|]y,y || Consistency || This was not changed for consistency when ADD and AUTO were updated ||
 |-
-| Global || SVN 3029 || **MOVEA** || MOVEA:x || Deprecate || use BONUS:... || Duplicate || This was rated an ancient Token by Tir || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1689591&group_id=25576&atid=384722FREQ|1689591]]
+| Global || SVN 3029 || '''MOVEA''' || MOVEA:x || Deprecate || use BONUS:... || Duplicate || This was rated an ancient Token by Tir || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1689591&group_id=25576&atid=384722FREQ|1689591]]
 |-
 | Global || SVN 3033 || KNOWNSPELLS || KNOWNSPELLS:.CLEARx || Put | after .CLEAR || KNOWNSPELLS:.CLEAR|x || Syntax || KNOWNSPELLS:.CLEARLEVEL=0 is just strange || [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/6293|_exp 6293]]
@@ Line 49: / Line 49: @@
 | Equipment || SVN 3027 || RANGE || RANGE:x || Eliminate Magical Value (-) || RANGE:x || Syntax ||   || [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7172|_exp 7172]]
 |-
-| EquipmentModifier || SVN 3029 || **ADDPROF** || ADDPROF:X.Y || Deprecate || none || Capability || The implementation of this token in the code makes the assumption that proficiencies are still in the (Exotic) format, which according to [[https://sourceforge.net/mailarchive/message.php?msg_id=45BBB226.3000208%40comcast.net|this discussion]] is no longer valid. ||
+| EquipmentModifier || SVN 3029 || '''ADDPROF''' || ADDPROF:X.Y || Deprecate || none || Capability || The implementation of this token in the code makes the assumption that proficiencies are still in the (Exotic) format, which according to [[https://sourceforge.net/mailarchive/message.php?msg_id=45BBB226.3000208%40comcast.net|this discussion]] is no longer valid. ||
 |-
-| Global || SVN 3031/4526 || **PREHD** || PREHD:x-y PREHD:x+ PREHD:x- (undocumented) || Remove Undocumented Feature || PREHD:x-y PREHD:x+ || Consistency, Clarity || An unclosed range should be 4+ not 4- ||
+| Global || SVN 3031/4526 || '''PREHD''' || PREHD:x-y PREHD:x+ PREHD:x- (undocumented) || Remove Undocumented Feature || PREHD:x-y PREHD:x+ || Consistency, Clarity || An unclosed range should be 4+ not 4- ||
 |-
-| Equipment || SVN 3810 || **CONTAINS** || varies || -1 changes to "UNLIM" || varies || Syntax, Known Error || Fix an issue where one item is capacity 1 and another capacity -1 resulting in total allowed items being zero (can't add anything to the container) Eliminate -1 as a magical value || [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7756|exp_7756]]
+| Equipment || SVN 3810 || '''CONTAINS''' || varies || -1 changes to "UNLIM" || varies || Syntax, Known Error || Fix an issue where one item is capacity 1 and another capacity -1 resulting in total allowed items being zero (can't add anything to the container) Eliminate -1 as a magical value || [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7756|exp_7756]]
 |-
-| Global || SVN up to 4875/4876 || **CHOOSE** || varies || Deprecate || See FREQ || Consistency || Previously approved || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1690306&group_id=25576&atid=384722|FREQ 1690306]] [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7753|exp_7753]] [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7754|exp_7754]]
+| Global || SVN up to 4875/4876 || '''CHOOSE''' || varies || Deprecate || See FREQ || Consistency || Previously approved || [[https://sourceforge.net/tracker/index.php?func=detail&aid=1690306&group_id=25576&atid=384722|FREQ 1690306]] [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7753|exp_7753]] [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7754|exp_7754]]
 |-
-| Global || SVN 3057 || **CHOOSE** || CHOOSE:WEAPONFOCUS || Deprecate || TBD (new syntax of CHOOSE:FEAT=) || Consistency || Previously approved || [[http://sourceforge.net/tracker/index.php?func=detail&aid=1683322&group_id=25576&atid=384722|FREQ 1683322]]
+| Global || SVN 3057 || '''CHOOSE''' || CHOOSE:WEAPONFOCUS || Deprecate || TBD (new syntax of CHOOSE:FEAT=) || Consistency || Previously approved || [[http://sourceforge.net/tracker/index.php?func=detail&aid=1683322&group_id=25576&atid=384722|FREQ 1683322]]
 |-
-| Global || SVN up to 4875/4876 || **CHOOSE** || CHOOSE:x (weapon) || Deprecate || CHOOSE:PROFICIENCY || Consistency || Previously approved || [[http://sourceforge.net/tracker/index.php?func=detail&aid=1631983&group_id=25576&atid=384722|FREQ 1631983]]
+| Global || SVN up to 4875/4876 || '''CHOOSE''' || CHOOSE:x (weapon) || Deprecate || CHOOSE:PROFICIENCY || Consistency || Previously approved || [[http://sourceforge.net/tracker/index.php?func=detail&aid=1631983&group_id=25576&atid=384722|FREQ 1631983]]
 |-
-| Class || SVN 3344 || **FEATAUTO** || FEATAUTO:x|x FEATAUTO:.CLEAR.x || Remove Cross-Level Interaction || AUTO:FEAT || Complicated || Currently, FEATAUTO can be used on different Class Level lines, and the .CLEAR behavior is expected to (or I shoudl say 'does' - not sure if people expect it) bridge across those different class levels. This is extremely complicated behavior to replicate, and behavior that we do not replicate elsewhere (except in SA, and we may remove that) || [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7773|exp_7773]]
+| Class || SVN 3344 || '''FEATAUTO''' || FEATAUTO:x|x FEATAUTO:.CLEAR.x || Remove Cross-Level Interaction || AUTO:FEAT || Complicated || Currently, FEATAUTO can be used on different Class Level lines, and the .CLEAR behavior is expected to (or I shoudl say 'does' - not sure if people expect it) bridge across those different class levels. This is extremely complicated behavior to replicate, and behavior that we do not replicate elsewhere (except in SA, and we may remove that) || [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7773|exp_7773]]
 |-
-| Global || SVN up to 4832 || **SA** || SA:.CLEAR. || Remove Cross-Level Interaction Deprecate .CLEAR behavior (complex, use Ability object) || SAB:x || Complicated || Currently, SA can be used on different Class Level lines, and the .CLEAR behavior is expected to bridge across those different class levels. This is extremely complicated behavior to replicate once Class Levels are separate objects || [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7773|exp_7773]]
+| Global || SVN up to 4832 || '''SA''' || SA:.CLEAR. || Remove Cross-Level Interaction Deprecate .CLEAR behavior (complex, use Ability object) || SAB:x || Complicated || Currently, SA can be used on different Class Level lines, and the .CLEAR behavior is expected to bridge across those different class levels. This is extremely complicated behavior to replicate once Class Levels are separate objects || [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7773|exp_7773]]
 |-
 | Global || SVN 3826 || MOVECLONE || MOVECLONE:x,xn,y,yn || Simplify || MOVECLONE:x,y,yn || Complicated || Fixes order of operations issues in movement || [[http://sourceforge.net/tracker/index.php?func=detail&aid=1707366&group_id=25576&atid=384722|FREQ 1707366]] [[http://tech.groups.yahoo.com/group/pcgen_experimental/message/7754|exp_7754]]