* [PATCH 1/2] FAQ: update the website address and call it Wiki
@ 2012-06-18 21:39 Jonathan Neuschäfer
2012-06-18 21:39 ` [PATCH 2/2] ptrlist.c: fix a typo in a comment Jonathan Neuschäfer
0 siblings, 1 reply; 6+ messages in thread
From: Jonathan Neuschäfer @ 2012-06-18 21:39 UTC (permalink / raw)
To: linux-sparse; +Cc: 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
^ permalink raw reply related [flat|nested] 6+ messages in thread* [PATCH 2/2] ptrlist.c: fix a typo in a comment
2012-06-18 21:39 [PATCH 1/2] FAQ: update the website address and call it Wiki Jonathan Neuschäfer
@ 2012-06-18 21:39 ` Jonathan Neuschäfer
2012-06-26 7:55 ` Christopher Li
2012-08-02 2:59 ` Mauro Dreissig
0 siblings, 2 replies; 6+ messages in thread
From: Jonathan Neuschäfer @ 2012-06-18 21:39 UTC (permalink / raw)
To: linux-sparse; +Cc: 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
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 2/2] ptrlist.c: fix a typo in a comment
2012-06-18 21:39 ` [PATCH 2/2] ptrlist.c: fix a typo in a comment Jonathan Neuschäfer
@ 2012-06-26 7:55 ` Christopher Li
2012-06-26 19:59 ` Jonathan Neuschäfer
2012-08-02 2:59 ` Mauro Dreissig
1 sibling, 1 reply; 6+ messages in thread
From: Christopher Li @ 2012-06-26 7:55 UTC (permalink / raw)
To: Jonathan Neuschäfer; +Cc: linux-sparse
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
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/2] ptrlist.c: fix a typo in a comment
2012-06-18 21:39 ` [PATCH 2/2] ptrlist.c: fix a typo in a comment Jonathan Neuschäfer
2012-06-26 7:55 ` Christopher Li
@ 2012-08-02 2:59 ` Mauro Dreissig
2012-08-02 14:56 ` Mauro Dreissig
1 sibling, 1 reply; 6+ messages in thread
From: Mauro Dreissig @ 2012-08-02 2:59 UTC (permalink / raw)
To: linux-sparse
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
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 2/2] ptrlist.c: fix a typo in a comment
2012-08-02 2:59 ` Mauro Dreissig
@ 2012-08-02 14:56 ` Mauro Dreissig
0 siblings, 0 replies; 6+ messages in thread
From: Mauro Dreissig @ 2012-08-02 14:56 UTC (permalink / raw)
To: linux-sparse
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
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-08-02 14:56 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-06-18 21:39 [PATCH 1/2] FAQ: update the website address and call it Wiki Jonathan Neuschäfer
2012-06-18 21:39 ` [PATCH 2/2] ptrlist.c: fix a typo in a comment Jonathan Neuschäfer
2012-06-26 7:55 ` Christopher Li
2012-06-26 19:59 ` Jonathan Neuschäfer
2012-08-02 2:59 ` Mauro Dreissig
2012-08-02 14:56 ` Mauro Dreissig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).