All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adam Jackson <ajax@redhat.com>
To: Sateesh Kavuri <sateesh.kavuri@intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH] Android port of intel-gpu-tools
Date: Tue, 10 Jan 2012 10:03:38 -0500	[thread overview]
Message-ID: <4F0C534A.50100@redhat.com> (raw)
In-Reply-To: <1326170701-22099-1-git-send-email-sateesh.kavuri@intel.com>

On 1/9/12 11:45 PM, Sateesh Kavuri wrote:
> Added support for Android. Changes include fixes for compilation issues
>   related to Android using an older version of GCC compiler (ver 4.3.3)
>   while the latest version of intel-gpu-tools confirms to GCC ver 4.5.2
>   (C99 standard functions), using functions like getline(). Fixed such
>   functions, header dependencies for android and added an Android.mk file.

I can understand avoiding C99 functions that android doesn't have, but 
this kind of thing:

> +            #ifdef ANDROID
> +	    int i;
> +	    for (i = 1; i<  len; i++) {
> +            #else
>   	    for (int i = 1; i<  len; i++) {
> +            #endif

is silly.  Does gcc -std=c99 on android seriously not cope with this?

- ajax

  parent reply	other threads:[~2012-01-10 15:03 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-10  4:45 [PATCH] Android port of intel-gpu-tools Sateesh Kavuri
2012-01-10 12:47 ` Daniel Vetter
2012-01-16 18:25   ` Chad Versace
2012-01-16 18:36     ` Daniel Vetter
2012-01-16 18:53       ` Chad Versace
     [not found]         ` <4F147225.3000604-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2012-01-16 19:52           ` [Intel-gfx] " Daniel Stone
2012-01-16 19:43       ` Eugeni Dodonov
2012-01-10 15:03 ` Adam Jackson [this message]
2012-01-10 16:50   ` Kavuri, Sateesh
2012-01-16 17:58     ` Chad Versace
2012-01-11  7:54 ` Kenneth Graunke

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F0C534A.50100@redhat.com \
    --to=ajax@redhat.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=sateesh.kavuri@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.