[haiku-bugs] Re: [Haiku] #13612: BTRFS: Adding write support

  • From: "pulkomandy" <trac@xxxxxxxxxxxx>
  • Date: Mon, 24 Jul 2017 10:03:06 -0000

#13612: BTRFS: Adding write support
----------------------------------+----------------------------
   Reporter:  hyche               |      Owner:  korli
       Type:  enhancement         |     Status:  new
   Priority:  normal              |  Milestone:  Unscheduled
  Component:  File Systems/Btrfs  |    Version:  R1/Development
 Resolution:                      |   Keywords:  gsoc2017
 Blocked By:                      |   Blocking:
Has a Patch:  1                   |   Platform:  All
----------------------------------+----------------------------

Comment (by pulkomandy):

 In the second commit ("add cat command"):

 {{{
 if (argc <= 2 && !strcmp(argv[1], "--help"))
 }}}

 This doesn't work. If argc is 1, argv[1] is not set.

 Try this:

 {{{
     if (argc < 2 || strcmp(argv[1], "--help") == 0)
 }}}

 In this case:
 * If there is only one argument, the second part of the condition isn't
 tested and we enter the if clause,
 * If there are two or more arguments, we test the first one to be equal to
 "--help".

 Side note: I don't know if the coding style mandates it, but I use strcmp
 (and any *cmp function) with == operator. It reads like "string compares
 equal" and can also work with <, >, etc. The use of the ! operator is more
 confusing to me.

 There is the downside that you can't cat a file named --help unless we
 also handle the -- option, but that sounds like an acceptable limitation
 for the btrfs_shell.

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

Other related posts: