Page MenuHomeDevCentral
Differential D821

Refactor .tcl DCC command
ClosedPublic
Actions

Authored by dereckson on Jan 12 2017, 05:14.
Tags
None
Referenced Files
F25183998: D821.diff
Fri, Apr 3, 19:45
Unknown Object (File)
Thu, Apr 2, 16:05
Unknown Object (File)
Wed, Apr 1, 23:38
Unknown Object (File)
Wed, Apr 1, 14:55
Unknown Object (File)
Sat, Mar 28, 09:18
Unknown Object (File)
Wed, Mar 25, 22:33
Unknown Object (File)
Sat, Mar 21, 18:40
Unknown Object (File)
Sat, Mar 21, 03:12
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rVIPER1dab92b91711: Refactor .tcl DCC command
Summary
  • separate concern of decide if we need to log in a standalone proc
  • add genpass to the commands not to log
  • unit tests
Test Plan

make test

Diff Detail

Repository
rVIPER ViperServ scripts
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
should_log_tcl_command
Build Status
Buildable 1259
Build 1507: arc lint + arc unit

Event Timeline

dereckson created this revision.Jan 12 2017, 05:14
dereckson accepted this revision.Jan 12 2017, 05:16
This revision is now accepted and ready to land.Jan 12 2017, 05:16
Closed by commit rVIPER1dab92b91711: Refactor .tcl DCC command (authored by dereckson, committed by dereckson). · Explain WhyJan 12 2017, 05:16
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
16 lines
tests/
36 lines

Diff 2078

View Options

Tech.tcl

Loading...
View Options

tests/Tech.test

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator