All of lore.kernel.org
 help / color / mirror / Atom feed
From: Johannes Obermayr <johannesobermayr@gmx.de>
To: dri-devel@lists.freedesktop.org, marcin.slusarz@gmail.com
Subject: libdrm: Fix some warnings reported by clang's scan-build tool [try 2]
Date: Fri, 13 Jul 2012 20:34:20 +0200	[thread overview]
Message-ID: <1342204466-15460-1-git-send-email-johannesobermayr@gmx.de> (raw)
In-Reply-To: <20120713164750.GA31630@joi.lan>


Am Freitag, 13. Juli 2012, 18:47:50 schrieb Marcin Slusarz:
> On Fri, Jul 13, 2012 at 05:49:12PM +0200, Johannes Obermayr wrote:
> > 
> > Patches 1 to 4 were sent to mesa-dev.
> 
> And you chose to ignore most of my comments.
> Fine. Don't expect further reviews from me.
> 
> Marcin

Patch 1 and 2:
- Adapted
- I want to keep proposed easier to read "switch" case

Patch 3:
- Resend
- Waiting on your response: http://lists.freedesktop.org/archives/mesa-dev/2012-June/023456.html

Patch 4 and 5:
- Splitted
- http://llvm.org/bugs/show_bug.cgi?id=13358 (forgot to split and to add 'drmFree(list);')
- The 'more if's case' seems better to me

Patch 6:
- Resend

Marcin, not that I ignore comments. But sometimes I want to hear also opinions from (some more) other people.
I hope I can calm the waves ...

Johannes

  reply	other threads:[~2012-07-13 18:36 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-13 15:49 libdrm: Fix some warnings reported by clang's scan-build tool Johannes Obermayr
2012-07-13 15:49 ` [PATCH 1/5] libkms/intel.c: Fix a memory leak and a dead assignment as well as cleanup code a bit Johannes Obermayr
2012-07-13 15:49 ` [PATCH 2/5] libkms/nouveau.c: Fix a memory leak and " Johannes Obermayr
2012-07-13 15:49 ` [PATCH 3/5] nouveau/nouveau.c: Fix two memory leaks Johannes Obermayr
2012-07-13 15:49 ` [PATCH 4/5] xf86drm.c: Make more code UDEV unrelevant and fix a memory leak Johannes Obermayr
2012-07-13 15:49 ` [PATCH 5/5] modetest.c: Add return 0 in bit_name_fn(res) macro Johannes Obermayr
2012-07-13 16:47 ` libdrm: Fix some warnings reported by clang's scan-build tool Marcin Slusarz
2012-07-13 18:34   ` Johannes Obermayr [this message]
2012-07-13 18:34     ` [PATCH 1/6] libkms/intel.c: Fix a memory leak and a dead assignment as well as make some code easier to read Johannes Obermayr
2012-07-13 18:34     ` [PATCH 2/6] libkms/nouveau.c: Fix a memory leak and " Johannes Obermayr
2012-07-13 18:34     ` [PATCH 3/6] nouveau/nouveau.c: Fix two memory leaks Johannes Obermayr
2012-07-13 18:34     ` [PATCH 4/6] xf86drm.c: Make more code UDEV unrelevant Johannes Obermayr
2012-07-13 18:34     ` [PATCH 5/6] xf86drm.c: Fix two memory leaks Johannes Obermayr
2012-07-13 18:34     ` [PATCH 6/6] modetest.c: Add return 0 in bit_name_fn(res) macro Johannes Obermayr

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=1342204466-15460-1-git-send-email-johannesobermayr@gmx.de \
    --to=johannesobermayr@gmx.de \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=marcin.slusarz@gmail.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.