[Mesa-stable] [PATCH] st/clover: add space between < and ::

Vedran Miletić vedran at miletic.net
Wed Apr 19 16:12:55 UTC 2017


On 04/19/2017 01:05 PM, Emil Velikov wrote:
> From: Emil Velikov <emil.velikov at collabora.com>
> 
> As pointed out by compiler
> 
> ./llvm/codegen.hpp:52:22: error: ‘<::’ cannot begin a template-argument list [-fpermissive]
> ./llvm/codegen.hpp:52:22: note: ‘<:’ is an alternate spelling for ‘[’. Insert whitespace between ‘<’ and ‘::’
> 
> Cc: Francisco Jerez <currojerez at riseup.net>
> Cc: <mesa-stable at lists.freedesktop.org>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
> Supersedes: HACK: st/clover: automake: set -fpermissive for clover/llvm
> 
> Commit message is a bit ... any suggestions will be appreciated.
> ---
>  src/gallium/state_trackers/clover/llvm/codegen.hpp         |  2 +-
>  src/gallium/state_trackers/clover/llvm/codegen/bitcode.cpp |  4 ++--
>  src/gallium/state_trackers/clover/llvm/codegen/common.cpp  |  6 +++---
>  src/gallium/state_trackers/clover/llvm/codegen/native.cpp  |  2 +-
>  src/gallium/state_trackers/clover/llvm/compat.hpp          |  8 ++++----
>  src/gallium/state_trackers/clover/llvm/metadata.hpp        | 12 ++++++------
>  6 files changed, 17 insertions(+), 17 deletions(-)
> 

Commit message looks fine to me. If you still want to reword it, then
remove mention of the compiler and say something along the lines of:
"C++11 requires space between < and :: as <: is an alternate spelling
for [ (see standard documents ... and ...)."

I see you caught all the occurrences.

Reviewed-by: Vedran Miletić <vedran at miletic.net>

-- 
Vedran Miletić
vedran.miletic.net


More information about the mesa-stable mailing list