[haiku-bugs] Re: [Haiku] #11934: [PATCH] Solve memory leaks in Deskbar and some replicants.

  • From: "pdziepak" <trac@xxxxxxxxxxxx>
  • Date: Mon, 30 Mar 2015 16:46:21 -0000

#11934: [PATCH] Solve memory leaks in Deskbar and some replicants.
------------------------------+----------------------------
   Reporter:  Barrett         |      Owner:  stippi
       Type:  enhancement     |     Status:  new
   Priority:  normal          |  Milestone:  R1
  Component:  User Interface  |    Version:  R1/Development
 Resolution:                  |   Keywords:  Deskbar
 Blocked By:                  |   Blocking:
Has a Patch:  1               |   Platform:  All
------------------------------+----------------------------

Comment (by pdziepak):

 VLAs are a C99 feature so I would expect most compilers to support them
 even when compiling C++ code (well, at least gcc and clang do), although a
 warning may be emitted. Moreover, there have been some attempts to add
 VLAs to C++, but, unfortunately, they didn't made it into C++14. Maybe
 things will change with C++17.

 Anyway, raw VLAs are tricky and I don't think that the code from the first
 patch is performance critical, so it probably is better to leave it as it
 was. (And in case it actually is performance critical, some variation of
 small_vector<> may be useful).

--
Ticket URL: <https://dev.haiku-os.org/ticket/11934#comment:3>
Haiku <https://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: