mudlet-makers team mailing list archive

[Felyza Wishbringer <felyza@xxxxxxxxx>]

Public bug reported:

TTrigger.cpp, around 330.

    if( rc < 0 )
    {
        switch(rc)
        {
             return false;
        }
    }

Throws a warning.

    if( rc < 0 )
    {
             return false;
    }

Does not.


While you're there.... afew more lines down has 
    if( rc < 0 )
    {
        return false;
    }
...which will never be reached.

** Affects: mudlet
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Mudlet
Makers, which is subscribed to Mudlet.
https://bugs.launchpad.net/bugs/1055977

Title:
  Switch statement has no use.

Status in Mudlet the MUD client:
  New

Bug description:
  TTrigger.cpp, around 330.

      if( rc < 0 )
      {
          switch(rc)
          {
               return false;
          }
      }

  Throws a warning.

      if( rc < 0 )
      {
               return false;
      }

  Does not.


  While you're there.... afew more lines down has 
      if( rc < 0 )
      {
          return false;
      }
  ...which will never be reached.

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