[PATCH 1/2] FAQ: update the website address and call it Wiki

[PATCH 1/2] FAQ: update the website address and call it Wiki

[Jonathan Neuschäfer]

Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
 FAQ |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/FAQ b/FAQ
index 4cad826..290de0e 100644
--- a/FAQ
+++ b/FAQ
@@ -88,7 +88,7 @@ A.  Yeah, well...  It parses a fairly complete subset of "extended C" as
 
 Q.  What other sparse resources are available?
 
-A.  Website: http://www.kernel.org/pub/software/devel/sparse/
+A.  Wiki: http://sparse.wiki.kernel.org/index.php/Main_Page
 
     Mailing list: linux-sparse@vger.kernel.org
     See http://vger.kernel.org/vger-lists.html#linux-sparse for subscription
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 2/2] ptrlist.c: fix a typo in a comment

[Jonathan Neuschäfer]

Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
---
 ptrlist.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ptrlist.c b/ptrlist.c
index 2620412..5dc1117 100644
--- a/ptrlist.c
+++ b/ptrlist.c
@@ -35,7 +35,7 @@ int ptr_list_size(struct ptr_list *head)
  *
  * The array to linearize into (second argument) should really
  * be "void *x[]", but we want to let people fill in any kind
- * of pointer array, so let's just call it "void *".
+ * of pointer array, so let's just call it "void **".
  */
 int linearize_ptr_list(struct ptr_list *head, void **arr, int max)
 {
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] ptrlist.c: fix a typo in a comment

[Christopher Li]

On Mon, Jun 18, 2012 at 2:39 PM, Jonathan Neuschäfer
<j.neuschaefer@gmx.net> wrote:
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>

Both 1 and 2 applied.

Chris
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] ptrlist.c: fix a typo in a comment

[Jonathan Neuschäfer]

On Tue, Jun 26, 2012 at 12:55:40AM -0700, Christopher Li wrote:
> On Mon, Jun 18, 2012 at 2:39 PM, Jonathan Neuschäfer
> <j.neuschaefer@gmx.net> wrote:
> > Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> 
> Both 1 and 2 applied.
> 
> Chris

Thanks!

Jonathan
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] ptrlist.c: fix a typo in a comment

[Mauro Dreissig]

On 06/18/2012 06:39 PM, Jonathan Neuschäfer wrote:
> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
> ---
>  ptrlist.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/ptrlist.c b/ptrlist.c
> index 2620412..5dc1117 100644
> --- a/ptrlist.c
> +++ b/ptrlist.c
> @@ -35,7 +35,7 @@ int ptr_list_size(struct ptr_list *head)
>   *
>   * The array to linearize into (second argument) should really
>   * be "void *x[]", but we want to let people fill in any kind
> - * of pointer array, so let's just call it "void *".
> + * of pointer array, so let's just call it "void **".
>   */
>  int linearize_ptr_list(struct ptr_list *head, void **arr, int max)
>  {
> 

Hi!

It looks like the comment was left there by mistake. I belive the idea
was to call the second argument void * to avoid warnings when passing
other_type ** to the function without having to cast it to void **.

Reading the comment now only gives the idea that declaring the argument
as void ** instead of void *[] gives different semantics, which is not the case. 

Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] ptrlist.c: fix a typo in a comment

[Mauro Dreissig]

On 08/01/2012 11:59 PM, Mauro Dreissig wrote:
> On 06/18/2012 06:39 PM, Jonathan Neuschäfer wrote:
>> Signed-off-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
>> ---
>>  ptrlist.c |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/ptrlist.c b/ptrlist.c
>> index 2620412..5dc1117 100644
>> --- a/ptrlist.c
>> +++ b/ptrlist.c
>> @@ -35,7 +35,7 @@ int ptr_list_size(struct ptr_list *head)
>>   *
>>   * The array to linearize into (second argument) should really
>>   * be "void *x[]", but we want to let people fill in any kind
>> - * of pointer array, so let's just call it "void *".
>> + * of pointer array, so let's just call it "void **".
>>   */
>>  int linearize_ptr_list(struct ptr_list *head, void **arr, int max)
>>  {
>>
> 
> Hi!
> 
> It looks like the comment was left there by mistake. I belive the idea
> was to call the second argument void * to avoid warnings when passing
> other_type ** to the function without having to cast it to void **.
> 
> Reading the comment now only gives the idea that declaring the argument
> as void ** instead of void *[] gives different semantics, which is not the case. 
> 
> Mauro
> 

Ah, I think I get it.. You cant cast something to "void *[]". sry :P
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html