<div class="gmail_quote">On 9 February 2011 12:28, Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

Hi all,<br>
attached patch is the initial support for .pushsection / .popsection.<br>
To do this properly requires a change in the MCStreamer API. One<br>
approach is attached. I think a better idea would be to split the<br>
current SwitchSection() into two parts: the common code to manage<br>
CurSection / PrevSection and a virtual function ChangeSection that gets<br>
called if the new CurSection differs from the old one. There are a bunch<br>
of bugs with the handling of changing to the current section again.<br></blockquote><div><br></div><div>Why does this need a second SwitchSection method and why does it need to take the previous as an argument? Why not just take the current one, then switch to the new one?</div>

<div><br></div><div>Also, did you check that a .s which consists entirely of ".previous" gives a nice error? GAS says:</div><div><div>x.s:1: Warning: .previous without corresponding .section; ignored</div></div>

<div><br></div><div>Nick</div><div><br></div></div>