[haiku-bugs] Re: [Haiku] #13513: [BTRFS] Code refactoring and integrate with fs_shell

  • From: "pulkomandy" <trac@xxxxxxxxxxxx>
  • Date: Fri, 19 May 2017 05:42:35 -0000

#13513: [BTRFS] Code refactoring and integrate with fs_shell
---------------------------+----------------------------
   Reporter:  hyche        |      Owner:  nobody
       Type:  enhancement  |     Status:  new
   Priority:  normal       |  Milestone:  Unscheduled
  Component:  - General    |    Version:  R1/Development
 Resolution:               |   Keywords:
 Blocked By:               |   Blocking:
Has a Patch:  1            |   Platform:  All
---------------------------+----------------------------

Comment (by pulkomandy):

 - The asterisk style is not fixed (next to the type or next to the
 variable). As long as it is consistent within one file
 - The indentation in class declarations was correct before your changes.
 The idea is to have one column for modifiers (virtual, const, static,
 etc), one column for types, and one column for variables and function
 names.
 - There are always two blank lines between blocks (for example
 AttributeIterator.h line 12, between the includes and the class
 declaration - also in the other files)

 If possible separate the style changes and fs_shell wrpaping in two
 commits. Also, you don't need to move <new> and <lib.h> if they are the
 same in both cases.

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

Other related posts: