From: SourceForge.net <no...@so...> - 2011-03-26 14:05:06
|
Bugs item #1963382, was opened at 2008-05-13 23:14 Message generated for change (Comment added) made by maartenbrock 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: Closed >Resolution: Fixed 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: Maarten Brock (maartenbrock) Date: 2011-03-26 15:05 Message: I don't know when it was fixed, but SDCC 3.0.2 #6360 outputs the correct line in the .cdb file. ---------------------------------------------------------------------- Comment By: Borut Ražem (borutr) Date: 2009-02-22 17:39 Message: The first part is fixed, the second (.cdb reference) is not, so I left it opened. Borut ---------------------------------------------------------------------- Comment By: Maarten Brock (maartenbrock) Date: 2009-02-19 12:55 Message: Borut, Is this fixed as you set it? If so, in which revision and why is this not closed then? If not, then please remove the fixed Group setting. And what does Resolution Accepted mean here for you? Maarten ---------------------------------------------------------------------- Comment By: Serge Malo (serge_malo) Date: 2008-09-03 16: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 09: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 08: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 |