[haiku-bugs] Re: [Haiku] #10224: Changed some dynamic casts to reinterpret casts

  • From: "bonefish" <trac@xxxxxxxxxxxx>
  • Date: Thu, 21 Nov 2013 20:25:56 -0000

#10224: Changed some dynamic casts to reinterpret casts
-------------------------------------+----------------------------
   Reporter:  Ezodev                 |      Owner:  anevilyak
       Type:  bug                    |     Status:  new
   Priority:  normal                 |  Milestone:  R1
  Component:  Applications/Debugger  |    Version:  R1/Development
 Resolution:                         |   Keywords:  gci2013
 Blocked By:                         |   Blocking:
Has a Patch:  1                      |   Platform:  All
-------------------------------------+----------------------------

Comment (by bonefish):

 * Don't use `fprintf()`. Just call `debugger()` in all cases. And `return`
 afterward.
 * String literals should not be wrapped like this. Close the literal at
 the end of the line and start a new on the next line (indent as usual with
 one additional tab), like:
 {{{
 printf("this line is "
         "too long\n");
 }}}
 * Null pointer checks should be spelled explicitly: {{{foo == NULL}}}.

--
Ticket URL: <http://dev.haiku-os.org/ticket/10224#comment:9>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: