|AnalysisInfo.txt logic missing when opening GCDA file [message #1689368]
||Mon, 23 March 2015 12:16
| Sanda Dedeic
Registered: November 2012
In linux tools v.1.2.1 there was a logic that was looking for AnalysisInfo.txt file when opening the GCDA file. Path to a binary file was resolved from there if such file exist. If not, first binary found within the project was used instead.
However, starting from v.2.0.0 this logic was removed, i.e. first binary found is used all the time.
We highly depend on AnalysisInfo mechanism because we have several binaries within the same project and it is not always the first one that we want to open.
Does anyone know why AnalisysInfo part was removed?
Is there some similar mechanism to achieve the same behavior?
GIT log for this change doesn't say to much: "o.e.l.gcov: Code review:refs/changes/60/10260/2"
Thank you in advance for any help.
[Updated on: Mon, 23 March 2015 14:34]
Report message to a moderator
|Re: AnalysisInfo.txt logic missing when opening GCDA file [message #1689759 is a reply to message #1689466]
||Tue, 24 March 2015 09:06
| Xavier Raynaud
Registered: February 2012
Hi Sanda Dedeic,|
I'm the author of this patch.
You're absolutely right, the logic around "AnalysisInfo.txt" has been removed (2 years ago!)
I removed it, because it was never documented (you have to read the source code to discover how it works), and therefore I think nobody uses it.
I was wrong, ans I'm a bit disappointed to discover the side effect of this patch.
As far as I know, there is no workaround for now.
Of course, the dialog box allows you to choose another binary file (not always the first). But I understand it can be annoying to change the default binary each time.
However, I suggest the following actions:
1/ Improve the algorithm that finds the default binary.
Instead of providing the first binary found, we can search for a binary with gcov information, corresponding to the file you have selected.
2/ Implements a "memory".
If, for a given file, you have selected another binary, it will be memorized, and selected by default for the subsequent attempts.
What's your opinion about these 2 actions ?
If it's OK for you, I will submit patches.
Powered by FUDForum
. Page generated in 0.02322 seconds