583076 - Fix a number of build warnings in accessible/

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Attachment: Patch v1

In particular, a number of warnings due to functions in base classes being hidden, unmatched enum values in ?: expressions, and unused variables. Patch attached.

Comment 1

[alexander :surkov (:asurkov)]

Comment on attachment 461335 [details] [diff] [review]
Patch v1


> nsAccTextChangeEvent::
>   nsAccTextChangeEvent(nsAccessible *aAccessible, PRInt32 aStart,
>                        nsAString& aModifiedText, PRBool aIsInserted,
>-                       PRBool aIsAsynch, EIsFromUserInput aIsFromUserInput) :
>-  nsAccEvent(aIsInserted ? nsIAccessibleEvent::EVENT_TEXT_INSERTED : nsIAccessibleEvent::EVENT_TEXT_REMOVED,
>-             aAccessible, aIsAsynch, aIsFromUserInput, eAllowDupes),
>-  mStart(aStart), mIsInserted(aIsInserted), mModifiedText(aModifiedText)
>+                       PRBool aIsAsynch, EIsFromUserInput aIsFromUserInput)
>+  : nsAccEvent(aIsInserted
>+               ? static_cast<PRUint32>(nsIAccessibleEvent::EVENT_TEXT_INSERTED)
>+               : static_cast<PRUint32>(nsIAccessibleEvent::EVENT_TEXT_REMOVED),

nit: do not keep operators at the begin of line, it should be 

> 
>+#include "mozilla/unused.h"
>+

> PRInt32
> nsAccessible::GetIndexInParent()
> {
>   // XXX: call GetParent() to repair the tree if it's broken.
>-  nsAccessible* parent = GetParent();
>+  mozilla::unused << GetParent();
>   return mIndexInParent;

GetParent() is enough here.

Comment 2

[alexander :surkov (:asurkov)]

(In reply to comment #1)

> nit: do not keep operators at the begin of line, it should be 

nsAccEvent(aIsInserted ?
  static_cast<PRUint32>(nsIAccessibleEvent::EVENT_TEXT_INSERTED) :
  static_cast<PRUint32>(nsIAccessibleEvent::EVENT_TEXT_REMOVED),

Comment 3

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Comment on attachment 461335 [details] [diff] [review]
Patch v1

(In reply to comment #1)
> Comment on attachment 461335 [details] [diff] [review]
> > 
> >+#include "mozilla/unused.h"
> >+
> 
> > PRInt32
> > nsAccessible::GetIndexInParent()
> > {
> >   // XXX: call GetParent() to repair the tree if it's broken.
> >-  nsAccessible* parent = GetParent();
> >+  mozilla::unused << GetParent();
> >   return mIndexInParent;
> 
> GetParent() is enough here.

Ok. (Eventually I'd like to get warnings for ignoring what getters return, but that can wait.)

(In reply to comment #2)
> (In reply to comment #1)
> 
> > nit: do not keep operators at the begin of line, it should be 
> 
> nsAccEvent(aIsInserted ?
>   static_cast<PRUint32>(nsIAccessibleEvent::EVENT_TEXT_INSERTED) :
>   static_cast<PRUint32>(nsIAccessibleEvent::EVENT_TEXT_REMOVED),

Ok.

Comment 4

[alexander :surkov (:asurkov)]

(In reply to comment #3)

> > >   // XXX: call GetParent() to repair the tree if it's broken.
> > >-  nsAccessible* parent = GetParent();
> > >+  mozilla::unused << GetParent();
> > >   return mIndexInParent;
> > 
> > GetParent() is enough here.
> 
> Ok. (Eventually I'd like to get warnings for ignoring what getters return, but
> that can wait.)

then it's ok

Comment 5

[alexander :surkov (:asurkov)]

landed on 2.0 - http://hg.mozilla.org/mozilla-central/rev/3900417b9594