[haiku-bugs] Re: [Haiku] #10402: [PATCHSET] Port kernel, bootloader and runtime_loader to Clang.

  • From: "js" <trac@xxxxxxxxxxxx>
  • Date: Mon, 13 Jan 2014 17:46:19 -0000

#10402: [PATCHSET] Port kernel, bootloader and runtime_loader to Clang.
---------------------------+----------------------------
   Reporter:  js           |      Owner:  nobody
       Type:  enhancement  |     Status:  new
   Priority:  normal       |  Milestone:  R1
  Component:  - General    |    Version:  R1/Development
 Resolution:               |   Keywords:
 Blocked By:               |   Blocking:
Has a Patch:  1            |   Platform:  All
---------------------------+----------------------------

Comment (by js):

 Hm, to make it more clear to people like me who are more at home with C
 than with C++ and sometimes forget about such things? ;) Nah, I can remove
 it, no problem.

 About the dso handle, I think we can just keep it this way then.

 About 0004: Clang complains about it with [], but accepts [0], which is
 strange. [0] should be equally wrong in C++. Maybe it accepts it as [0] is
 a GNU extension? The correct way would be to use [1], but that would break
 memory management. I'm not sure about what to do with it. Just change it
 to [0] and be done, even though it's still not proper C++? It seems to be
 wrong anyway, because memory allocation will be wrong, even with []. So I
 think this code is wrong by design and needs to be fixed properly some
 day. What do others think?

 Other than that, can the patches be applied after removing the useless
 static?

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

Other related posts:

  1. Home
  2. haiku-bugs
  3. Archive
  4. 01-2014