<div dir="ltr"><div><div><div>Hi,<br></div>  I'd like some feedback on this patch. Coverity detects an uncaught exception code path for OutOfBoundsException which eventually leads to SBuf.toUpper. However in single-threaded mode the exception cannot be thrown by the loop calling that code.<br></div>Is it reasonable to explicitly catch that exception as I do in the attached patch or should we rather triage and mark as false positives in coverity all uses of this method? Or is there a more effective way to obtain the same result?<br><br></div>Thanks<br clear="all"><div><div><div><div><div><br>-- <br><div class="gmail_signature">    kinkie</div>
</div></div></div></div></div></div>