-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/49035/
-----------------------------------------------------------

Review request for geode, Grace Meilen and Jinmei Liao.


Bugs: GEODE-1565
    https://issues.apache.org/jira/browse/GEODE-1565


Repository: geode


Description
-------

GEODE-1565: add quotes around any value starting with hyphen

* refactor OptionJFormatter into HyphenFormatter to format all values starting 
with hypen


Diffs
-----

  
geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/GfshParser.java
 0bf2f66 
  
geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/parser/jopt/JoptOptionParser.java
 04590ed 
  
geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/remote/CommandProcessor.java
 b62f922 
  
geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/util/HyphenFormatter.java
 PRE-CREATION 
  
geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/util/OptionJFormatter.java
 0e3265a 
  
geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/GfshParserIntegrationTest.java
 17e78a5 
  
geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/GemfireDataCommandsDUnitTest.java
 553b7a4 
  
geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/util/HyphenFormatterTest.java
 PRE-CREATION 
  
geode-core/src/test/java/com/gemstone/gemfire/management/internal/cli/util/OptionJFormatterTest.java
 12dcd5e 
  
geode-web/src/test/java/com/gemstone/gemfire/management/internal/cli/commands/CommandOverHttpDUnitTest.java
 cad3f31 

Diff: https://reviews.apache.org/r/49035/diff/


Testing
-------

new tests in:

* HyphenFormatterTest
* GfshParserIntegrationTest
* GemfireDataCommandsDUnitTest

precheckin


Thanks,

Kirk Lund

Reply via email to