← Back to team overview

unity-api-bugs team mailing list archive

[Bug 1282567] [NEW] libindicator/indicator-service-manager.c:561]: (style) Same expression on both sides of '&&'.

 

Public bug reported:

Source code is

        if (new_name != NULL && prev_name != NULL && new_name[0] != 0 &&
prev_name != 0 && g_strcmp0(prev_name, new_name) != 0) {

I think

        if (new_name != NULL && prev_name != NULL && new_name[0] != 0 &&
prev_name[0] != 0 && g_strcmp0(prev_name, new_name) != 0) {

was probably intended.

** Affects: libindicator
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Unity API
bugs, which is subscribed to libindicator.
https://bugs.launchpad.net/bugs/1282567

Title:
  libindicator/indicator-service-manager.c:561]: (style) Same expression
  on both sides of '&&'.

Status in Libindicator:
  New

Bug description:
  Source code is

          if (new_name != NULL && prev_name != NULL && new_name[0] != 0
  && prev_name != 0 && g_strcmp0(prev_name, new_name) != 0) {

  I think

          if (new_name != NULL && prev_name != NULL && new_name[0] != 0
  && prev_name[0] != 0 && g_strcmp0(prev_name, new_name) != 0) {

  was probably intended.

To manage notifications about this bug go to:
https://bugs.launchpad.net/libindicator/+bug/1282567/+subscriptions


Follow ups

References