~compiz-team/compiz/compiz.fix_1006335

« back to all changes in this revision

Viewing changes to plugins/scalefilter/src/scalefilter.cpp

  • Committer: Daniel van Vugt
  • Author(s): smspillaz
  • Date: 2012-05-28 06:08:02 UTC
  • mfrom: (3225.4.2 compiz.clang-warnings)
  • Revision ID: daniel.van.vugt@canonical.com-20120528060802-1nnxgrg2amsieb01
Fixes warnings that happen on clang. Clang picked up a number of things by
default, including:

1. Memset by the size of a pointer rather than by the struct itself
2. inline functions in header files declared only
3. needless casting (although a leaked KWinAdapter is just insane, not sure
   how to fix that ..)
4. Unnecessary checks for < 0 with unsigned int
5. Member methods hiding virtual functions because they were never overridden
   correclty
6. Unused isAncestorTo in decor
7. ; after if statement with block proceeding in some places
8. Incrementing a bool

Show diffs side-by-side

added added

removed removed

Lines of Context:
548
548
    sScreen (ScaleScreen::get (s))
549
549
{
550
550
    xim = XOpenIM (s->dpy (), NULL, NULL, NULL);
551
 
    if (xim)
552
 
        xic = XCreateIC (xim, XNClientWindow, s->root (), XNInputStyle,
553
 
                         XIMPreeditNothing  | XIMStatusNothing, NULL);
 
551
    //if (xim)
 
552
        //xic = XCreateIC (xim, XNClientWindow, s->root (), XNInputStyle,
 
553
        //               XIMPreeditNothing  | XIMStatusNothing, NULL);
554
554
 
555
555
    if (xic)
556
556
        setlocale (LC_CTYPE, "");