<div dir="ltr"><div dir="ltr">On Sun, Jul 12, 2020 at 12:44 AM Lukas Bulwahn <<a href="mailto:lukas.bulwahn@gmail.com">lukas.bulwahn@gmail.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jul 11, 2020 at 5:44 PM Mrinal Pandey <<a href="mailto:mrinalmni@gmail.com" target="_blank">mrinalmni@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The usage of "capture group (...)" in the immediate condition after `&&`<br>
results in `$1` being uninitialized. This eventually crashes the script.<br>
<br></blockquote><div><br></div><div>It does not really crash it, right? It just emits a warning.</div></div></div></blockquote><div><br></div><div>Sir,</div><div><br></div><div>Yes. I will modify the line accordingly.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Fix this by placing the capture group in the condition before `&&`.<br>
Thus, `$1` can be initialized to the text it matches thereby setting it<br>
to the desired and required value.<br>
<br></blockquote><div><br></div><div>Maybe you can look when this bug was introduced?</div></div></div></blockquote><div><br></div><div>The bug was first introduced with the commit `e518e9a59ec3` when the block was</div><div>added to the script. It has been like that since then.</div><div>Should I add this detail too in the commit message?<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div>Lukas</div></div></div>
</blockquote></div></div>