[PATCH xf86-video-modesetting 1/2] Staticise output names

[Thierry Reding]

On Mon, Nov 25, 2013 at 09:05:18AM -0800, Aaron Plattner wrote:
> On 11/25/13 03:49, Thierry Reding wrote:
> >This array isn't used anywhere outside this file, so it can be made
> >static.
> >
> >Signed-off-by: Thierry Reding <treding at nvidia.com>
> >---
> >  src/drmmode_display.c | 32 ++++++++++++++++----------------
> >  1 file changed, 16 insertions(+), 16 deletions(-)
> >
> >diff --git a/src/drmmode_display.c b/src/drmmode_display.c
> >index 773ec4c1c441..9965409a0307 100644
> >--- a/src/drmmode_display.c
> >+++ b/src/drmmode_display.c
> >@@ -961,22 +961,22 @@ static int subpixel_conv_table[7] = { 0, SubPixelUnknown,
> >  				      SubPixelVerticalBGR,
> >  				      SubPixelNone };
> >
> >-const char *output_names[] = { "None",
> >-			       "VGA",
> >-			       "DVI",
> >-			       "DVI",
> >-			       "DVI",
> >-			       "Composite",
> >-			       "S-video",
> >-			       "LVDS",
> >-			       "CTV",
> >-			       "DIN",
> >-			       "DisplayPort",
> >-			       "HDMI",
> >-			       "HDMI",
> >-			       "TV",
> >-			       "eDP",
> >-			       "Virtual"
> >+static const char *output_names[] = { "None",
> 
> While you're at it, can you make it static const char *const as well?

Yes, good idea. I've sent an updated version.

Thanks,
Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20131126/246ea9da/attachment.pgp>