I do agree that an additional option would indeed be confusing, there are no really good options for us here.
The option you've chosen will provide a full log, but the layout will not be very readable, because the 'control lines' will have the same prefix (i.e. indentation) as the stacktrace lines, making them difficult to spot.
>From a functional perspective I think maybe the best solution would be that the 'control lines' get the same prefix as the normal log lines instead (and not the same as the stacktrace lines). I tried this first, but found this very difficult to code without making a lot of changes to how the appender works. It might require to much effort just to fix a readability/indentation problem, I won't blame anyone if it doesn't get fixed :)
As for testing, unfortunately I have moved on to a new client and will not be able to test the fix properly, but I'll forward this to the guys (the girl actually) that maintains this code now, hopefully she will pick this up where I left.
The option you've chosen will provide a full log, but the layout will not be very readable, because the 'control lines' will have the same prefix (i.e. indentation) as the stacktrace lines, making them difficult to spot.
>From a functional perspective I think maybe the best solution would be that the 'control lines' get the same prefix as the normal log lines instead (and not the same as the stacktrace lines). I tried this first, but found this very difficult to code without making a lot of changes to how the appender works. It might require to much effort just to fix a readability/indentation problem, I won't blame anyone if it doesn't get fixed :)
As for testing, unfortunately I have moved on to a new client and will not be able to test the fix properly, but I'll forward this to the guys (the girl actually) that maintains this code now, hopefully she will pick this up where I left.