<blockquote>
<p>are there any notes on the logic of this attempt to fix the "New Window" function</p>
</blockquote>

<p>I like the end result.</p>

<p>It would be easier to review (and look back to later) if it was broken down into the general stages of the change, like in a few different commits. I like the fact that it uses the <code>spawn_*()</code> API now, instead of <code>g_find_program_in_path()</code> + <code>utils_spawn_*()</code> (change 1), and agree with the changes from <a href="https://github.com/b4n" class="user-mention">@b4n</a>'s PR that it incorporates getting the installed/resource binary path (change 2, without using his commit+attribution), but the stuff with the options lookup table, and new globals, functions, and data types (change 3..N) seems like a lot of complexity (though it may indeed be warranted), and the part that guards out the <code>is_osx_bundle()</code> seems unrelated (change N+1).</p>

<p>Because of the above I have not done a thorough code review and likely won't merge it myself, but I'm +1 for the end result in general.</p>

<blockquote>
<p>The ultimate Right Thing for that will be to open a new window in the same process, but that will take time, and we have to patch "New Window" somehow until then.</p>
</blockquote>

<p>Agree, there's no point holding off on an intermediate solution until we endeavour to improve the underlying implementation enough to support the ideal solution.</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/geany/geany/pull/637#issuecomment-137294359">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABDrJzz2YCokte6JLUcGDuJN36KHM4rWks5ot5bKgaJpZM4F1KFb.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/geany/geany/pull/637#issuecomment-137294359"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>