Re: still problems in cvs log parser for graph view

Am 11.03.2004 um 10:12 schrieb Brunen Karl-Heinz:

> Hi Jens,
>
> This is a known issue in the actual stable releases. If you update your
> code base you will see that this problem is solved. The problem was 
> that
> e.g. rev 1.6 has no description (commit message). So the parser
> regonized revision 1.5 as the revision description.

Doh! I updated my code base yesterday, but the version I built and 
which I'm now using was still built using the old code base.
I remembered seeing some cvsgui-log messages about the log parser, but 
I forgot that I didn't update until after I built that version.
I'll try again with the latest code base...

Thanks
</jum>

> -----Original Message-----
> From: Jens Miltner [mailto:[email protected]]
> Sent: Thursday, March 11, 2004 10:06 AM
> To: [email protected]
> Subject: [cvsgui-dev] still problems in cvs log parser for graph view
>
>
> While verifying my latest fixes for large graphs in MacCvs, I stumbled
> across a problem with incorrect log descriptions being dumped in Graph
> view:
> The file I checkes has about 770 revisions on the trunk plus a few
> branches and quite a few tags.
>
> One problem is that revision 1.5 seems to be skipped: I see 1.4 and the
> next one is 1.6.
> When I click on rev 1.6, I get the following log message written to the
> console window:
>
>> ----------------------------
>> Revision : 1.6
>> Date : 1996/7/16 7:56:0
>> Author : 'dodo'
>> State : 'Exp'
>> Lines : + 3 -1
>> Description :
>> ----------------------------
>> revision 1.5
>> date: 1996/07/10 07:21:00;  author: martin;  state: Exp;  lines: +7 -0
>> Added CodeResModuleLoaderTerminate
>
> (looks like 1.5 isn't correctly recognized while parsing the log and is
> 'merged' into 1.6)
>
> Also, when clicking on the first revision (rev 1.1), I get the
> following incorrect log description:
>
>> ----------------------------
>> Revision : 1.1
>> Date : 1996/6/28 7:42:0
>> Author : 'martin'
>> State : 'Exp'
>> Lines : + 0 0
>> Description :
>> ----------------------------
>> revision 1.542.2.2
>> date: 2002/04/15 13:17:49;  author: jens;  state: Exp;  lines: +5 -2
>> #pragma warning -> #warning
>
>
> Again, another revision seems to be 'merged' in (and as expected,
> 1.542.2.2 is missing from the graph).
>
> Here's the excerpt from the plain cvs log for this file for rev 
> 1.4-1.7:
>
>
>> ----------------------------
>> revision 1.7
>> date: 1996/07/25 12:56:00;  author: martin;  state: Exp;  lines: +18
>> -14
>> Added Support for foobar
>>
>>
>>
>> ----------------------------
>> revision 1.6
>> date: 1996/07/16 07:56:00;  author: dodo;  state: Exp;  lines: +3 -1
>>
>>
>>
>>
>> ----------------------------
>> revision 1.5
>> date: 1996/07/10 07:21:00;  author: martin;  state: Exp;  lines: +7 -0
>> Added CodeResModuleLoaderTerminate
>>
>>
>>
>> ----------------------------
>> revision 1.4
>> date: 1996/07/09 16:15:00;  author: martin;  state: Exp;  lines: +12
> -2
>> Hardcoded return value of foo
>>
>>
>>
>> ----------------------------
>
>
> and here's an excerpt around rev 1.542.2.2.2:
>
>
>> ----------------------------
>> revision 1.2
>> date: 1996/07/04 10:49:00;  author: martin;  state: Exp;  lines: +24
>> -10
>> CommManager is now delete at the end
>>
>>
>>
>> ----------------------------
>> revision 1.1
>> date: 1996/06/28 07:42:00;  author: martin;  state: Exp;
>>
>>
>>
>>
>> ----------------------------
>> revision 1.542.2.2
>> date: 2002/04/15 13:17:49;  author: jens;  state: Exp;  lines: +5 -2
>> #pragma warning -> #warning
>> ----------------------------
>> revision 1.542.2.1
>> date: 2002/04/11 13:42:58;  author: jens;  state: Exp;  lines: +14 -11
>> adjusted to CWPro 8 compilers (MPTP fincal candidates)
>> ----------------------------
>>
>
>
> Since I don't know too much about the graph log parser, maybe Karl or
> Jerzy have an idea what's going wrong?
> Probably because for some reason, there's an empty comment for those
> revisions. Is this something that's legal, or is this somehow a strange
> corruption in our cvs repository?
> In any event, our parser should probably be smart enough to handle
> those situations...
>
> </jum>



------------------------ Yahoo! Groups Sponsor ---------------------~-->
Upgrade to 128-bit SSL Security!
http://us.click.yahoo.com/LPJzrA/yjVHAA/TtwFAA/NhFolB/TM
---------------------------------------------------------------------~->

 
Yahoo! Groups Links

<*> To visit your group on the web, go to:
     http://groups.yahoo.com/group/cvsgui-dev/

<*> To unsubscribe from this group, send an email to:
     [email protected]

<*> Your use of Yahoo! Groups is subject to:
     http://docs.yahoo.com/info/terms/