[haiku-bugs] Re: [Haiku] #12334: MediaConvert does not check for read only target directory (easy)

  • From: "Barrett" <trac@xxxxxxxxxxxx>
  • Date: Sat, 04 Mar 2017 14:27:00 -0000

#12334: MediaConvert does not check for read only target directory (easy)
-------------------------------------------+----------------------------
   Reporter:  pulkomandy                   |      Owner:  Barrett
       Type:  bug                          |     Status:  assigned
   Priority:  normal                       |  Milestone:  R1
  Component:  Applications/MediaConverter  |    Version:  R1/Development
 Resolution:                               |   Keywords:
 Blocked By:                               |   Blocking:
Has a Patch:  0                            |   Platform:  All
-------------------------------------------+----------------------------

Comment (by Barrett):

 Some comments :

 * Fix else brackets
 * Use the BAlert constructor directly instead to create a BString
 * Use the BPath directly no need to use another string to Append (it
 handles slash automatically), see BPath::Append and BPath::Path which
 returns a string.
 * Use tempFile instead of fp as var name.

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

Other related posts: