Loading ...
Sorry, an error occurred while loading the content.

Re: [pcgen_developers] CI Upgrade and Code Checks

Expand Messages
  • Andrew Wilson
    ... Generally ignoring exceptions is a bad idea so the PMD/Checkstyle warning ... warranted IMO. ... Fair enough, what severity of thing should I log it as?
    Message 1 of 12 , Jun 6 4:05 PM
    • 0 Attachment
      On 6 June 2011 23:08, James Dempsey <jdempsey@...> wrote:
      > One of the files I'm working on is complaining about an empty catch block.  Thing 
      > is it's commented, so someone's put a bit of thought into it.  Do we really care 
      > about this?  if not how do I get rid of the warning.  do we have a suppression 
      > mechanism that we're using?

      One technique I have used in the past is to add a log entry to the catch block. 
      Generally ignoring exceptions is a bad idea so the PMD/Checkstyle warning is 
      warranted IMO.

      Fair enough, what severity of thing should I log it as? 

      andrew
    Your message has been successfully submitted and would be delivered to recipients shortly.