Hello,<div>      Another patch, containing all the changes for the GnuReadLine -&gt; LineReader abstraction.</div><div><br></div><div>Thanks,</div><div>Jonathan<br><br><div class="gmail_quote">On Tue, Mar 10, 2009 at 10:48 PM, Jonathan Chambers <span dir="ltr">&lt;<a href="mailto:joncham@gmail.com">joncham@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Martin,<div>     Thanks for reviewing. I&#39;ll send in some small, easy to review patches.</div><div><br></div><div>Thanks,</div>
<div>Jonathan<div><div></div><div class="h5"><br><br><div class="gmail_quote">On Tue, Mar 10, 2009 at 2:16 PM, Martin Baulig <span dir="ltr">&lt;<a href="mailto:martin@novell.com" target="_blank">martin@novell.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jonathan,<br>
<br>
thanks a lot for this patch.<br>
<br>
Just a few comments:<br>
<br>
* Please don&#39;t use DebuggerConfiguration.IsRunningOnWindows -<br>
  DebuggerConfiguration is for storing persistent state and things<br>
  the user can configure.<br>
  I think the right place for this is Inferior.IsRunningOnWindows,<br>
  the Inferior can also auto-detect this.<br>
<br>
* DebuggerOptions.InferiorArguments must be kept platform-neutral<br>
  and thus we can&#39;t add null at the end there.  The OS-specific code<br>
  should do that.<br>
<br>
* We shouldn&#39;t use any `#ifndef WIN32&#39; conditionals in<br>
  backend/server/library.c.  Instead, we should add new methods to<br>
  the `InferiorVTable&#39; and have a custom implementation in<br>
  x86-ptrace.c and x86-windows.c<br>
<br>
* I like the GnuReadLine -&gt; LineReader abstraction :-)<br>
<br>
* Regarding ThreadManager, if we need larger changes there, it may<br>
  make sense to make `ThreadManager&#39; an abstract base class and have<br>
  a custom implementation for Linux and Windows.<br>
<br>
Martin<br>
<font color="#888888"><br>
--<br>
Martin Baulig - <a href="mailto:martin@novell.com" target="_blank">martin@novell.com</a><br>
Novell GmbH, Nördlicher Zubringer 9-11, 40470 Düsseldorf<br>
GF: Dr. Jürgen Müller, Sylvia Geil, Felix Imendörffer; HRB 21108 (AG Düsseldorf)<br>
<br>
<br>
</font></blockquote></div><br></div></div></div>
</blockquote></div><br></div>