On Wed, 2016-05-18 at 19:59 +0300, Pekka Jääskeläinen wrote: > Hi Joseph, > > Thanks for the comments. Updated&rebased patch attached. Hopefully I > didn't miss any diags.
It looks like the attachment doesn't contain the patch; on unzipping I just see a 27 byte file reading "The BRIG frontend itself." A similar problem seems to have happened with the update to 1/4: https://gcc.gnu.org/ml/gcc-patches/2016-05/msg01385.html where the "patch" just has the descriptive text, but not the actual changes. Looks like you need to resend both. Hope this is helpful; good luck Dave > On Wed, May 18, 2016 at 3:20 AM, Joseph Myers < > jos...@codesourcery.com> wrote: > > This patch has many improperly formatted diagnostic messages (e.g. > > starting with capital letters, ending with '.' or failing to use %q > > for > > quoting). I also note cases where you use %lu as a format for > > size_t, > > which is not correct (you'd need to add pretty-print.c support for > > %zu > > before you could use that, however). > > > > -- > > Joseph S. Myers > > jos...@codesourcery.com