https://code.cor-lab.de/https://code.cor-lab.de/favicon.ico?14019720732014-10-22T20:20:33ZOpen Source Collaboration PlatformRobotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=60852014-10-22T20:20:33ZR. Haschkerhaschke@techfak.uni-bielefeld.de
<ul><li><strong>Status</strong> changed from <i>Feedback</i> to <i>Resolved</i></li><li><strong>Assignee</strong> changed from <i>J. Wienke</i> to <i>R. Haschke</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>90</i></li></ul><p>- extract + demangle C++ function names on Linux <a href="https://code.cor-lab.de/projects/rsc/repository/revisions/43ee009e514bd1cc4cdb7c0cb475894789275f57" class="changeset" title="extracting and demangling function names from raw stack trace">43ee009</a><br />- removed class DebugTools, expose functions createBacktrace() + formatBacktrace() directly <a href="https://code.cor-lab.de/projects/rsc/repository/revisions/38c6e40996dddfd00bc6497e20ba608222ada8b2" class="changeset" title="removed DebugTools class, directly expose functions createBacktrace() + formatBacktrace()">38c6e40</a><br />- added Windows implementation for createBacktrace(): please verify on Windows machine! <a href="https://code.cor-lab.de/projects/rsc/repository/revisions/78798d1ec36b34b9f3abe4199d7a66d4b79b7ce3" class="changeset" title="Windows implementation for createBacktrace() taken from http://www.codeproject.com/Articles/1113...">78798d1</a></p>
<p>source branch: enhancement-2063</p> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=60872014-10-23T13:07:17ZJ. Wienkejwienke@techfak.uni-bielefeld.de
<ul><li><strong>Category</strong> set to <i>Debug Tools</i></li><li><strong>Status</strong> changed from <i>Resolved</i> to <i>In Progress</i></li></ul><p>This issue cannot be resolved as long as things aren't in the master branch. ;)</p>
<p>Thanks for the work. I am fine with the general API changes, however, there are several formal issues that need to be resolved before we can integrate this into master.</p>
<ol>
<li>Your feature branch is based on the URI feature branch. It needs to be rebased on master without the URI changes. If you rename it to <code>enhancement-2063</code>, it will also automatically the jenkins merge simulator jobs so that you can get feedback from jenkins on your changes (assuming there is no new bug in jenkins for this feature).</li>
<li>Please do not use inline functions defined in the headers. These end up being compiled in the client programs and not in the library. Hence, internal changes to RSC will require a recompilation of client code, which effectively increases coupling and breaks separation of concerns.</li>
<li>Please document under which assumptions the demangle functions works (e.g. compiler). Any chance to get the code more readable? E.g. by using real c++ strings.</li>
<li>I am note sure whether we can simply include the <code>StackWalker</code> as it is to not violate license agreements. We need to find this out.</li>
<li>In any case, the windows code needs to adhere to our coding standards. In all 3 windows-related files, tabs and windows line ends are used, our license headers are missing and indentation and line break rules are off.</li>
<li>Please either integrate the required features of <code>MyStackWalker</code> directly into the original <code>StackWalker</code> and limit its functionality to what we actually require or give this class a more descriptive name to explain the purpose why it exists. Adapting the <code>StackWalker</code> to the functionality we need, might also be sufficient to move it further apart from the original code base so that we declare this as our own code under our license. I am not sure about this.</li>
</ol>
<p>Apart from the general licensing issue, would you adapt your branch accordingly?</p> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=60882014-10-23T15:04:22ZR. Haschkerhaschke@techfak.uni-bielefeld.de
<ul></ul><p>I can work on that some time.<br />I suggest to keep the StackWalker code as is. I didn't touched anything there yet and I don't want to.</p> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=60942014-10-24T08:26:10ZJ. Wienkejwienke@techfak.uni-bielefeld.de
<ul></ul><p>Robert Haschke wrote:</p>
<blockquote>
<p>I can work on that some time.<br />I suggest to keep the StackWalker code as is. I didn't touched anything there yet and I don't want to.</p>
</blockquote>
<p>So, I'd propose to move the windows stracktrace support to a new issue and feature branch so that we can continue with the API changes without having to discuss the license issues. Moreover, this also feels more like a separate feature.</p> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=60952014-10-24T09:07:20ZR. Haschkerhaschke@techfak.uni-bielefeld.de
<ul></ul><p>I worked on all the points you pointed out. However, I didn't touched StackWalker itself.<br />You should decide how to deal with the BSD license of that code or extract the neccessary functionality. Actually StackWalker is much more powerful than required in rsc context: It can create stacktraces of different threads or even processes.</p> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=65582015-02-11T00:54:06ZR. Haschkerhaschke@techfak.uni-bielefeld.de
<ul><li><strong>Assignee</strong> changed from <i>R. Haschke</i> to <i>J. Wienke</i></li></ul> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=65612015-02-11T01:10:08ZR. Haschkerhaschke@techfak.uni-bielefeld.de
<ul><li><strong>Blocks</strong> <i><a href="/issues/2051" class="issue tracker-2 status-2 priority-4 priority-default">Feature #2051</a>: Qt-based graphical event logger</i> added</li></ul> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=66012015-02-24T09:33:01ZS. Wredeswrede@cor-lab.uni-bielefeld.de
<ul><li><strong>Target version</strong> set to <i>rsb-0.12</i></li></ul> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=68992015-04-27T09:16:49ZJ. Wienkejwienke@techfak.uni-bielefeld.de
<ul><li><strong>Target version</strong> changed from <i>rsb-0.12</i> to <i>rsb-1.0</i></li></ul> Robotics Systems Commons - Enhancement #2063: simplify clumsy usage of DebugToolshttps://code.cor-lab.de/issues/2063?journal_id=85072016-06-30T10:38:29ZJ. Wienkejwienke@techfak.uni-bielefeld.de
<ul><li><strong>Assignee</strong> deleted (<del><i>J. Wienke</i></del>)</li></ul>