Patch to remove some warnings

[Allen Winter]

On Tuesday, December 19, 2006 05:09:40 PM Leonardo Stern wrote:
> Greetings,
> 
> Could someone apply the patch in the end of this message ? (to remove
> some warnings about "*/" inside the code)
> 
I like Q_UNUSED() for situations like this.
Which also helps keep doxygen quiet.


> regards,
> Leonardo Stern
> 
> Index: kdeui/tests/kblendtest.cpp
> ===================================================================
> --- kdeui/tests/kblendtest.cpp	(revision 615015)
> +++ kdeui/tests/kblendtest.cpp	(working copy)
> @@ -23,7 +23,7 @@
>      setPalette(palette);
>  }
> 
> -void KBlendWidget::paintEvent(QPaintEvent */*ev*/)
> +void KBlendWidget::paintEvent(QPaintEvent * /*ev*/)
>  {
>      QTime time;
>      int it, ft;
> Index: khtml/ecma/kjs_audio.cpp
> ===================================================================
> --- khtml/ecma/kjs_audio.cpp	(revision 615015)
> +++ khtml/ecma/kjs_audio.cpp	(working copy)
> @@ -102,7 +102,7 @@
>      return getStaticValueSlot<Audio, DOMObject>(exec, &AudioTable,
> this, propertyName, slot);
>  }
> 
> -ValueImp *Audio::getValueProperty(ExecState */*exec*/, int token) const
> +ValueImp *Audio::getValueProperty(ExecState * /*exec*/, int token) const
>  {
>      switch (token) {
>        case Onerror:
> @@ -151,7 +151,7 @@
>      }
>  }
> 
> -void Audio::notifyFinished(khtml::CachedObject */*co*/)
> +void Audio::notifyFinished(khtml::CachedObject * /*co*/)
>  {
>      if (!m_cs) return;
> 
> Index: khtml/ecma/kjs_window.cpp
> ===================================================================
> --- khtml/ecma/kjs_window.cpp	(revision 615015)
> +++ khtml/ecma/kjs_window.cpp	(working copy)
> @@ -2504,7 +2504,7 @@
>    return "[object FrameArray]";
>  }
> 
> -ValueImp* FrameArray::callAsFunction(ExecState *exec, ObjectImp
> */*thisObj*/, const List &args)
> +ValueImp* FrameArray::callAsFunction(ExecState *exec, ObjectImp *
> /*thisObj*/, const List &args)
>  {
>      //IE supports a subset of the get functionality as call...
>      //... basically, when the return is a window, it supports that,
> otherwise it
> Index: kjsembed/kjsembed/filedialog_binding.cpp
> ===================================================================
> --- kjsembed/kjsembed/filedialog_binding.cpp	(revision 615015)
> +++ kjsembed/kjsembed/filedialog_binding.cpp	(working copy)
> @@ -30,7 +30,7 @@
> 
>  using namespace KJSEmbed;
> 
> -KJS::JSValue *callGetExistingDirectory( KJS::ExecState *exec,
> KJS::JSObject */*self*/, const KJS::List &args )
> +KJS::JSValue *callGetExistingDirectory( KJS::ExecState *exec,
> KJS::JSObject * /*self*/, const KJS::List &args )
>  {
>      QWidget *parent = KJSEmbed::extractObject<QWidget>(exec, args, 0, 0);
>      QString caption = KJSEmbed::extractVariant<QString>(exec, args,
> 1, QString());
> @@ -40,7 +40,7 @@
>      return KJS::String( QFileDialog::getExistingDirectory(parent,
> caption, dir, options) );
>  }
> 
> -KJS::JSValue *callGetOpenFileName( KJS::ExecState *exec,
> KJS::JSObject */*self*/, const KJS::List &args )
> +KJS::JSValue *callGetOpenFileName( KJS::ExecState *exec,
> KJS::JSObject * /*self*/, const KJS::List &args )
>  {
>      QWidget *parent = KJSEmbed::extractObject<QWidget>(exec, args, 0, 0);
>      QString caption = KJSEmbed::extractVariant<QString>(exec, args, 1, "");
> @@ -52,7 +52,7 @@
>      return KJS::String( QFileDialog::getOpenFileName(parent, caption,
> dir, filter, 0, options) );
>  }
> 
> -KJS::JSValue *callGetOpenFileNames( KJS::ExecState *exec,
> KJS::JSObject */*self*/, const KJS::List &args )
> +KJS::JSValue *callGetOpenFileNames( KJS::ExecState *exec,
> KJS::JSObject * /*self*/, const KJS::List &args )
>  {
>      QWidget *parent = KJSEmbed::extractObject<QWidget>(exec, args, 0, 0);
>      QString caption = KJSEmbed::extractVariant<QString>(exec, args,
> 1, QString());
> @@ -66,7 +66,7 @@
>      return convertToValue(exec, fileNames);
>  }
> 
> -KJS::JSValue *callGetSaveFileName( KJS::ExecState *exec,
> KJS::JSObject */*self*/, const KJS::List &args )
> +KJS::JSValue *callGetSaveFileName( KJS::ExecState *exec,
> KJS::JSObject * /*self*/, const KJS::List &args )
>  {
>      QWidget *parent = KJSEmbed::extractObject<QWidget>(exec, args, 0, 0);
>      QString caption = KJSEmbed::extractVariant<QString>(exec, args,
> 1, QString());
> Index: kjsembed/kjsembed/iosupport.cpp
> ===================================================================
> --- kjsembed/kjsembed/iosupport.cpp	(revision 615015)
> +++ kjsembed/kjsembed/iosupport.cpp	(working copy)
> @@ -35,20 +35,20 @@
>      return KJS::Null();
>  }
> 
> -KJS::JSValue * callPrintLn( KJS::ExecState *exec, KJS::JSObject
> */*self*/, const KJS::List &args )
> +KJS::JSValue * callPrintLn( KJS::ExecState *exec, KJS::JSObject *
> /*self*/, const KJS::List &args )
>  {
>      (*KJSEmbed::conout()) << args[0]->toString(exec).qstring() << endl;
>      return KJS::Null();
>  }
> 
> -KJS::JSValue * callDebug( KJS::ExecState *exec, KJS::JSObject
> */*self*/, const KJS::List &args )
> +KJS::JSValue * callDebug( KJS::ExecState *exec, KJS::JSObject *
> /*self*/, const KJS::List &args )
>  {
>      //(*KJSEmbed::conerr())  << "Debug: " <<
> args[0]->toString(exec).qstring() << endl;
>      qDebug()  << "Debug: " << args[0]->toString(exec).qstring();
>      return KJS::Null();
>  }
> 
> -KJS::JSValue * callReadLine( KJS::ExecState *exec, KJS::JSObject
> */*self*/, const KJS::List &args )
> +KJS::JSValue * callReadLine( KJS::ExecState *exec, KJS::JSObject *
> /*self*/, const KJS::List &args )
>  {
>      Q_UNUSED(exec);
>      Q_UNUSED(args);
> @@ -56,7 +56,7 @@
>      return KJS::String( line );
>  }
> 
> -KJS::JSValue * callSystem( KJS::ExecState *exec, KJS::JSObject
> */*self*/, const KJS::List &args )
> +KJS::JSValue * callSystem( KJS::ExecState *exec, KJS::JSObject *
> /*self*/, const KJS::List &args )
>  {
>      QProcess systemProcess;
>      QStringList processArgs = args[0]->toString(exec).qstring().split( ' ' );
> Index: kjsembed/kjsembed/object_binding.cpp
> ===================================================================
> --- kjsembed/kjsembed/object_binding.cpp	(revision 615015)
> +++ kjsembed/kjsembed/object_binding.cpp	(working copy)
> @@ -44,7 +44,7 @@
>      return m_name;
>  }
> 
> -KJS::UString ObjectBinding::toString(KJS::ExecState */*exec*/) const
> +KJS::UString ObjectBinding::toString(KJS::ExecState * /*exec*/) const
>  {
>      return KJS::UString( typeName() );
>  }
> 
> 



-- 
This is my KDE signature