summaryrefslogtreecommitdiff
path: root/clformat/src/main
diff options
context:
space:
mode:
authorBenjamin J. Culkin <bjculkin@mix.wvu.edu>2020-11-26 12:47:37 -0400
committerBenjamin J. Culkin <bjculkin@mix.wvu.edu>2020-11-26 12:47:37 -0400
commit8933de7f646f0565edf700aa2f2fcab06d639855 (patch)
tree7da4bf9f88477986a8af239a81138d7f499076eb /clformat/src/main
parent03de62016afa4e392f32069ec28ad58ee38699da (diff)
Fix some failing tests
This fixes two failing tests * One was that when refactoring AestheticDirective, I accidentally broke doing padding before the value. * The other was that when allowing for using multiple ` on a directive, I accidentally broke parsing for ` directives
Diffstat (limited to 'clformat/src/main')
-rw-r--r--clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java39
-rw-r--r--clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java7
2 files changed, 37 insertions, 9 deletions
diff --git a/clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java b/clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java
index 856589c..3dfb45b 100644
--- a/clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java
+++ b/clformat/src/main/java/bjc/utils/ioutils/format/directives/AestheticDirective.java
@@ -83,23 +83,50 @@ class AestheticEdict implements Edict {
StringBuilder work = new StringBuilder();
- char padchar = padcharPar.asChar(itemTape, "padding character", "A", ' ');
+ char padchar = padcharPar.asChar(itemTape,
+ "padding character", "A", ' ');
- int mincol = mincolPar.asInt(itemTape, "minimum column count", "A", 0);
- int colinc = colincPar.asInt(itemTape, "padding increment", "A", 1);
- int minpad = minpadPar.asInt(itemTape, "minumum amount of padding", "A", 0);
+ int mincol = mincolPar.asInt(itemTape,
+ "minimum column count", "A", 0);
+ int colinc = colincPar.asInt(itemTape,
+ "padding increment", "A", 1);
+ int minpad = minpadPar.asInt(itemTape,
+ "minumum amount of padding", "A", 0);
- if (padBefore) doPadding(work, padchar, mincol, colinc, minpad);
+ // if (padBefore) doPadding(work, padchar, mincol, colinc, minpad);
work.append(itemString);
- if (!padBefore) doPadding(work, padchar, mincol, colinc, minpad);
+ // if (!padBefore) doPadding(work, padchar, mincol, colinc, minpad);
+
+ String padding = createPadding(work, padchar, mincol, colinc, minpad);
+ if (padBefore) work.insert(0, padding);
+ else work.append(padding);
formCTX.writer.write(work.toString());
itemTape.right();
}
+ private String createPadding(
+ StringBuilder work,
+ char padchar,
+ int mincol,
+ int colinc,
+ int minpad)
+ {
+ StringBuilder padding = new StringBuilder();
+ for (int i = 0; i < minpad; i++) padding.append(padchar);
+
+ for (int i = work.length(); i < mincol; i++) {
+ for (int k = 0; k < colinc; k++) {
+ padding.append(padchar);
+ }
+ }
+
+ return padding.toString();
+ }
+
private void doPadding(StringBuilder work, char padchar, int mincol, int colinc,
int minpad) {
for (int i = 0; i < minpad; i++) work.append(padchar);
diff --git a/clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java b/clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java
index 8c16917..0d1d0df 100644
--- a/clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java
+++ b/clformat/src/main/java/bjc/utils/ioutils/format/directives/DecimalDirective.java
@@ -80,11 +80,12 @@ class DecimalEdict implements Edict {
}
}
- StringBuffer work = new StringBuffer();
+ //StringBuffer work = new StringBuffer();
- numForm.format(itemTape.item(), work, ZERO_FIELD);
+ //numForm.format(itemTape.item(), work, ZERO_FIELD);
- formCTX.writer.write(work.toString());
+ // formCTX.writer.write(work.toString());
+ formCTX.writer.write(numForm.format(itemTape.item()));
itemTape.right();
}
}