mnemonics icon indicating copy to clipboard operation
mnemonics copied to clipboard

Fixed Issue #1, Duplicate Code for t and vr.

Open jayharris opened this issue 11 years ago • 5 comments

Fixed the issue with vr templates replicating, rather than getting the type definitions. Moved the t templates to a new UntypedMemberTemplates group, as the test templates will not be appended with type definitions. Also, renamed the template from t to tm (Test Method) to allow for future test fixture, setup, and teardown templates.

Did not regenerate the Download files.

jayharris avatar Sep 10 '13 16:09 jayharris

Does this fix the following bug?

vrb generates

private readonly type fieldname;

instead of

private readonly bool fieldname;

angularsen avatar Feb 14 '14 13:02 angularsen

Yes. Any of the vr* templates will render the proper type value instead of type.

It also renames the t mnemonic for test methods to tm so that additional test-based mnemonics can be added.

jayharris avatar Feb 14 '14 13:02 jayharris

This addresses #1 and is the beginning of correcting #5

jayharris avatar Feb 14 '14 13:02 jayharris

I recently discovered that nb and ob also exists that are somewhat overlapping.

private bool fieldname = false;
private readonly bool fieldname = false;

Do we need both vr* and o* ?

I do miss some simple documentation that lists all the cases and what variations are supported for each, but that is a separate issue.

angularsen avatar Feb 14 '14 14:02 angularsen

I think "vb" and "vrb" for a field declarations with and without readonly modifier is intuitive, but "nb" and "ob" is not. Maybe adapting the same naming, such as "nb" and "nrb" for declarations with default value assignment, with/without readonly modifier?

angularsen avatar Feb 14 '14 14:02 angularsen