[haiku-bugs] Re: [Haiku] #12710: BPackageInfo; URL Validation Checks

  • From: "axeld" <trac@xxxxxxxxxxxx>
  • Date: Fri, 22 Apr 2016 15:12:35 -0000

#12710: BPackageInfo; URL Validation Checks
--------------------------------+----------------------------
   Reporter:  apl-haiku         |      Owner:  bonefish
       Type:  enhancement       |     Status:  new
   Priority:  normal            |  Milestone:  Unscheduled
  Component:  Kits/Package Kit  |    Version:  R1/Development
 Resolution:                    |   Keywords:
 Blocked By:                    |   Blocking:
Has a Patch:  1                 |   Platform:  All
--------------------------------+----------------------------

Comment (by axeld):

 Thanks Andrew. I'm afraid there are more issues that I missed so far,
 though.

 * Still coding style issues: 80 column line limit is violated at several
 places, and stray white space is added to a few lines. I had fixed those
 issues, as I wanted to get over with it, but...
 * It doesn't compile using GCC 5 on Ubuntu (x86_64). Among the errors:
   * 1078/1082: jump to case label
   * 959: enumation value ... not handled
 * The usual format of our git patch line would read like this:
 `BPackageInfo::Parser: Validate URL strings.`

 Also, please don't zip the patch next time for easy online viewing :-)

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

Other related posts: