[sdcc-devel] [ sdcc-Bugs-1963382 ] Listing File - C file comment is incorrect

[SourceForge.net <no...@so...>]

Bugs item #1963382, was opened at 2008-05-13 17:14
Message generated for change (Comment added) made by serge_malo
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100599&aid=1963382&group_id=599

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: C-Front End
Group: fixed
Status: Open
Resolution: Accepted
Priority: 5
Private: No
Submitted By: Serge Malo (serge_malo)
Assigned to: Borut Ražem (borutr)
Summary: Listing File - C file comment is incorrect

Initial Comment:
SDCC command: sdcc --debug lst_problem_example.c
SDCC version 2.8.0, under Windows XP.

The problem is inside the listing (.lst and .rst) files generated.
When compiling the included example (lst_problem_example.c), the C lines comment inside the listing file seems incorrect.

Inside the listing file, we see the same C line comment twice, like this:
159 ;	lst_problem_example.c:22: l_uiCounter = 0;
171 ;	lst_problem_example.c:22: l_uiCounter = 0;

In my opinion, line 171 is correct. But the line 159 should have had the next comment:
159 ;	lst_problem_example.c:14: if (P1_4 == 0)

Also, note that the .cdb has no reference to the C line number #14. This might be related to the same problem.


----------------------------------------------------------------------

>Comment By: Serge Malo (serge_malo)
Date: 2008-09-03 10:42

Message:
Logged In: YES 
user_id=2082376
Originator: YES

In the .cdb file, I think that we should see this line (C-Line Record):
L:C$lst_problem_example.c$14$3$3:7A

It should be written right after the line
L:A$lst_problem_example$160:7A

Thanks,
Serge




----------------------------------------------------------------------

Comment By: Borut Ražem (borutr)
Date: 2008-08-24 03:57

Message:
Logged In: YES 
user_id=568035
Originator: NO

I forgot to mention that I applied the patch 2050245: Fix # 1963382
http://sourceforge.net/tracker/?func=detail&atid=300599&aid=2050245&group_id=599

The regression test was not added since there is no way to automatically
check the correctness of the .lst file.

Borut

----------------------------------------------------------------------

Comment By: Borut Ražem (borutr)
Date: 2008-08-24 02:58

Message:
Logged In: YES 
user_id=568035
Originator: NO

Parch applied in svn revision #5219.

Serge, can you be more specific about .cdb issue? How the missing
reference should look like? I'm not an expert in .cdb format :-(

Borut

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100599&aid=1963382&group_id=599