Commit 0dce471
committed
Include the index in the property name when formatting a SequenceValue
I had previously hesitated a lot to include the index and when improving the example code in the README I stumbled on a problem with Log4View: it can't handle XML with two identical properties (no error is displayed but no log is displayed at all either).
Before this commit (unreadable with Log4View):
```
<log4net:properties>
<log4net:data name="Args" value="--first-argument" />
<log4net:data name="Args" value="--second-argument" />
</log4net:properties>
```
After this commit (no problem with Log4View):
```
<log4net:properties>
<log4net:data name="Args[0]" value="--first-argument" />
<log4net:data name="Args[1]" value="--second-argument" />
</log4net:properties>
```1 parent 6cae94a commit 0dce471
File tree
2 files changed
+5
-5
lines changed- src
- tests
2 files changed
+5
-5
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
291 | 291 | | |
292 | 292 | | |
293 | 293 | | |
294 | | - | |
| 294 | + | |
295 | 295 | | |
296 | | - | |
| 296 | + | |
297 | 297 | | |
298 | 298 | | |
299 | 299 | | |
| |||
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | | - | |
4 | | - | |
5 | | - | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
0 commit comments