This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 2/5] Clean pthread functions namespaces for C11 threads


On Mon, 10 Aug 2015, Juan Manuel Torres Palma wrote:

> @@ -501,7 +501,7 @@ allocate_stack (const struct pthread_attr *attr, struct pthread **pdp,
>  	    size += pagesize_m1 + 1;
>  #endif
>  
> -	  mem = mmap (NULL, size, prot,
> +	  mem = __mmap (NULL, size, prot,
>  		      MAP_PRIVATE | MAP_ANONYMOUS | MAP_STACK, -1, 0);

In cases such as this, you need to reindent followign lines of the call so 
they remain correctly indented after the function name has got longer.

> +weak_alias(__pthread_cancel, pthread_cancel);

Missing space before '('.

-- 
Joseph S. Myers
joseph@codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]