<div dir="ltr"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 25, 2015 at 2:33 PM, Ilia K <span dir="ltr"><<a href="mailto:ki.stfu@gmail.com" target="_blank">ki.stfu@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>oops..</div>-<span style="font-size:12.8px"><a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a></span><div><span style="font-size:12.8px">+<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a></span></div></div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Fri, Sep 25, 2015 at 2:30 PM, Hafiz Abid Qadeer <span dir="ltr"><<a href="mailto:abidh.haq@gmail.com" target="_blank">abidh.haq@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sounds reasonable to me.<div><div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 25, 2015 at 8:02 AM, Ilia K <span dir="ltr"><<a href="mailto:ki.stfu@gmail.com" target="_blank">ki.stfu@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi guys!</div><div><br></div><div>I found the following things that can be simplified:</div><div>1. Add CMIUtilString(std::string) ctor:</div><div>```</div><div>CMIUtilString(std::string().c_str()) -> CMIUtilString(std::string())</div><div>```</div><div>Patch: <a href="http://reviews.llvm.org/D13158" target="_blank">http://reviews.llvm.org/D13158</a></div><div>2. Rename CMICmnBase::SetErrorDescriptionn to CMICmnBase::SetErrorDescriptionFormat<br></div><div>3. Allow to pass unformatted arguments (i.e. format string + tuple of parameters) to CMICmnBase::SetErrorDescription:</div><div>```</div><div>SetErrorDescription(CMIUtilString::Format(MIRSRC(IDS_CMDFACTORY_ERR_INVALID_CMD_NAME), vMiCmd.c_str()))<br></div><div>```</div><div>and then remove CMICmnBase::SetErrorDescriptionFormat.</div><div>4. Allow to pass unformatted arguments to CMICmdBase::SetError:</div><div>```</div><div>SetError(CMIUtilString::Format(MIRSRC(IDS_CMD_ERR_OPTION_NOT_FOUND), m_cmdData.strMiCmd.c_str(), m_constStrArgThread.c_str()));<br></div><div>```</div><div><br></div><div>Thoughts?</div><span><font color="#888888"><div><br></div>-- <br><div><div dir="ltr">- Ilia</div></div>
</font></span></div>
</blockquote></div><br></div></div></div></div>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div><div dir="ltr">- Ilia</div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">- Ilia</div></div>
</div>