<div dir="ltr"><div dir="ltr">Hi<div><br></div><div>Thanks for sending this to devel@. There are some minor issues and then you can send an updated version.</div><div><br></div><div>(1) Title of patch shouldn't reference psxtmonce02 since we decided to just add the case to psxtmonce01.</div><div>Something like this: "psxtmonce01: Add second use of once variable case". </div><div><br></div><div>(2) psxtmonce01/Makefile.am is already deleted. So you need to update your tree and update the patch</div><div>to reflect it is no longer in the tree.</div><div><br></div><div>(3) the change to the psxtmtests.csv should also change the name of the test exe holding the case from</div><div>psxtmonce02 to psxtmonce01.</div><div><br></div><div>The code itself looks good. Minor stuff.</div><div><br></div><div>When you use git format-patch, add the -vN argument and it will make it clear that it is vN of the patch.</div><div>This would be v2. Something like "git format-patch -v2 master"</div><div><br></div><div>Thanks.</div><div><br></div><div>--joel</div></div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Nov 10, 2018 at 9:24 AM Himanshu Sekhar Nayak <<a href="mailto:himanshuwindows8.1@gmail.com">himanshuwindows8.1@gmail.com</a>> wrote:<br></div><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>I have added a new test suite for review of psxtmonce02.</div><div>Here is the patch file you can find with this email.</div><div><br></div><div>Thanks</div><div>Himanshu Sekhar Nayak<br></div></div>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div>