Menu

#169 [ oorexx-Patches-2954418 ] Add gethostname to rxsock

5.0.0
closed
Erich
rxSock (2)
5
2023-01-01
2010-02-19
No

Just a heads up about this patch, there exists a potential name collision between sockgethostname and sockgethostbyname. I am not sure if my fix will work.

Discussion

  • Brandon Cherry

    Brandon Cherry - 2010-02-19
     
  • Rick McGuire

    Rick McGuire - 2010-02-19

    I'm not sure I understand why there is a problem.

     
  • Brandon Cherry

    Brandon Cherry - 2010-02-19

    Maybe this will help:

    SockGetHostByName <indexterm><primary>SockGetHostName</primary></indexterm> <indexterm> <primary>function</primary> <secondary>SockGetHostByName</secondary> </indexterm>
    SockGetHostByName <indexterm><primary>SockGetHostByName</primary></indexterm> <indexterm> <primary>function</primary> <secondary>SockGetHostByName</secondary> </indexterm> The primary and secondary indexterms are now the same (with my change). Not sure if this is ok.
     
  • Rick McGuire

    Rick McGuire - 2010-02-19

    Ah, sorry, I thought you were talking about the code patch...That seems strange usage there. I think maybe both should use GetHostName as the primary, then have GetHostName and GetHostByName as the appropriate secondaries so both variants are shown. Note also you've duplicated the section ids, which will be a small problem.

     
  • Brandon Cherry

    Brandon Cherry - 2010-02-19
     
  • Brandon Cherry

    Brandon Cherry - 2010-02-19

    "rxsock.sgml.2.patch" should work considering what you said.

     
  • Erich

    Erich - 2016-08-22
    • status: open --> pending
    • assigned_to: Erich
    • Group: --> 5.0.0
     
  • Erich

    Erich - 2016-08-22

    Added getHostByName method with revision [r11151]

     

    Related

    Commit: [r11151]

  • Rony G. Flatscher

    • Status: pending --> closed
     

Anonymous
Anonymous

Add attachments
Cancel