[PATCH] - Adding timestamps to terminal output

Mark Einon mark.einon at gmail.com
Mon Feb 28 11:36:50 UTC 2011


On 20 February 2011 11:38, Adam Lackorzynski <adam at os.inf.tu-dresden.de> wrote:
> Hi,
>
> On Sat Feb 19, 2011 at 15:48:41 +0000, Mark Einon wrote:
>> On 19 February 2011 14:03, Adam Lackorzynski <adam at os.inf.tu-dresden.de> wrote:
>> > I applied it and further modified it. I like it more when the timestamp
>> > always prepends each line. What do you think?
>>
>> I did consider that approach, and it isn't hard to change to use it,
>> or even make it configurable. I decided against it because:
>>
>> 1. The log files produced are greatly increased in size (especially
>> for an overnight run).
>> 2. Terminals that have limited width (e.g 80 lines) get wrapped /
>> truncated lines more often
>> 3. Each line printed would go through the process of formatting the
>> timestamp, which may cause issues on slower machines - although
>> thinking about it again, you only have to format the timestamp string
>> when the time changes, so this can be mitigated.
>
> Ok. So I made the timestamping a three value thing now, adding a 'once
> per second' choice. That should cover both options.

Great, seems to work well. I've attached a patch to change the help
string from 'on/off' to 'toggle' which is more accurate. It's not
totally transparent what it does, but it suffices...

Cheers,

   Mark
>
>
>
> Adam
> --
> Adam                 adam at os.inf.tu-dresden.de
>  Lackorzynski         http://os.inf.tu-dresden.de/~adam/
>
> --
> minicom-devel mailing list <minicom-devel at lists.alioth.debian.org>
> http://lists.alioth.debian.org/mailman/listinfo/minicom-devel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: timestamp_help_change.patch
Type: text/x-patch
Size: 626 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/minicom-devel/attachments/20110228/7c71380f/attachment.bin>


More information about the minicom-devel mailing list