[haiku-bugs] Re: [Haiku] #12388: Missing support for TLS SNI (easy)

  • From: "markh" <trac@xxxxxxxxxxxx>
  • Date: Tue, 21 Jun 2016 18:10:20 -0000

#12388: Missing support for TLS SNI (easy)
--------------------------------+----------------------------
   Reporter:  FreeFull          |      Owner:  axeld
       Type:  bug               |     Status:  closed
   Priority:  normal            |  Milestone:  Unscheduled
  Component:  Kits/Network Kit  |    Version:  R1/Development
 Resolution:  fixed             |   Keywords:
 Blocked By:                    |   Blocking:
Has a Patch:  1                 |   Platform:  All
--------------------------------+----------------------------

Comment (by markh):

 That's not what the code was doing. It was returning early on success,
 which meant that any value that needed to be done on success needed to be
 done in multiple places. This was already happening with fStatus and
 adding fHostName multiple times seemed to me to make it worse.

 Also, in general I dislike multiple exit points in a function, but if the
 coding style is to prefer this then I will keep it in mind for other
 patches.

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

Other related posts: