>>> On 27.02.17 at 06:09, <haozhong.zh...@intel.com> wrote: > The current production build works fine even though those ASSERT's are > violated.
This - without any evidence - is rather weak an argument. Code inspection, otoh, pretty clearly tells us the ASSERT()s aren't needed, as there is no dependency on the asserted expressions to be true. Therefore, while the change itself is fine, may I ask for a better commit message? Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel