bugGNU Octave - Bugs: bug #51273, [octave forge] (sockets) function...

 
 

bug #51273: [octave forge] (sockets) function MSG_DONTWAIT and MSG_WAITALL should have error fallbacks defined

Submitter:  None
Submitted:  Tue 20 Jun 2017 09:43:17 AM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  3 - Low Item Group:  Unexpected Error or Warning
Status:  Fixed Assigned to:  lostbard
Originator Name:  Ilkka Kontola Originator Email:  -email is unavailable-
Open/Closed:  * Closed Release:  * dev
Operating System:  * Any Fixed Release:  None
Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Thu 24 Feb 2022 02:24:33 PM UTC, comment #12: 

Comment #11 again: "Thanks for looking at this" with two smileys... Maybe for Savannah ASCII must suffice =)

Kai Torben Ohlhus <siko1056>
Group Member
Thu 24 Feb 2022 02:22:36 PM UTC, comment #11: 

Thanks for taking a look at this 🙂👍

Kai Torben Ohlhus <siko1056>
Group Member
Thu 24 Feb 2022 02:18:25 PM UTC, comment #10: 
John Donoghue <lostbard>
Group Member
Thu 24 Feb 2022 01:06:56 PM UTC, comment #9: 

I had forgotten all about this

Currently it would do the old behavior of not know that variable rather than give an error saying it is unimplmented

John Donoghue <lostbard>
Group Member
Thu 24 Feb 2022 04:01:40 AM UTC, comment #8: 

@lostbard: Is this still an issue with sockets 1.3.0?

Kai Torben Ohlhus <siko1056>
Group Member
Wed 21 Jun 2017 04:58:37 PM UTC, comment #7: 

Patched added. Its been a while since sockets was last released so also needs updates for base level Makefile etc (not included in patch)

(file #41014)

John Donoghue <lostbard>
Group Member
Tue 20 Jun 2017 06:13:27 PM UTC, comment #6: 

That's a good idea. Both MSG_DONTWAIT and MSG_WAITALL are defined within an ifdef, so both should have these workaround error functions if the constants are not defined. Updating summary

Mike Miller <mtmiller>
Group Member
Tue 20 Jun 2017 06:07:58 PM UTC, comment #5: 

The documentation doesn't mention anything about constants available or not available depending on OS, so I guess either document that, or do the same as AF_LOCAL, where it is changed to a function on non-supported systems so it can throw back an error message that it isn't supported?

John Donoghue <lostbard>
Group Member
Tue 20 Jun 2017 05:28:59 PM UTC, comment #4: 

So the bug is that the package is behaving as designed. It intentionally does not define wrappers for constants that are not declared on a given operating system. Is this unclear to the user? Does someone have a suggestion to improve this for the user experience?

Mike Miller <mtmiller>
Group Member
Tue 20 Jun 2017 12:25:27 PM UTC, comment #3: 

Looks like MSG_DONTWAIT is not supported in windows.

https://msdn.microsoft.com/en-us/library/windows/desktop/ms740121(v=vs.85).aspx

John Donoghue <lostbard>
Group Member
Tue 20 Jun 2017 12:20:03 PM UTC, comment #2: 

In windows, sockets 1.2.0, I see:


>> MSG_PEEK
ans =  2
>> MSG_WAITALL
ans =  8
>> MSG_DONTWAIT
error: 'MSG_DONTWAIT' undefined near line 1 column 1
>>


John Donoghue <lostbard>
Group Member
Tue 20 Jun 2017 11:02:23 AM UTC, comment #1: 
Andreas Weber <andy1978>
Group Member
Tue 20 Jun 2017 09:43:17 AM UTC, original submission:  



code:

[raw, len_s] = recv(GPIBSERV,40000, MSG_DONTWAIT);

produces:

error: 'MSG_DONTWAIT' undefined near line ....

using flag MSG_WAITALL the function works as expected



Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #41014:  bug51273.patch added by lostbard (2KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by siko1056 (Posted a comment)
  • -email is unavailable- added by jwe (Updated the item)
  • -email is unavailable- added by lostbard (Adding package maintainer)
  • -email is unavailable- added by lostbard (Posted a comment)
  • -email is unavailable- added by andy1978 (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 17 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2022-02-25 lostbard StatusReady For Test Fixed
        Open/ClosedOpen Closed
    2022-02-24 lostbard StatusPatch Submitted Ready For Test
        Assigned toNone lostbard
    2019-02-26 mtmiller Carbon-CopyRemoved 80942 -
    2019-02-26 mtmiller Release4.0.3 dev
        Operating SystemMicrosoft Windows Any
    2018-07-02 jwe Summary[octave-forge] (sockets) function MSG_DONTWAIT and MSG_WAITALL should have error fallbacks defined [octave forge] (sockets) function MSG_DONTWAIT and MSG_WAITALL should have error fallbacks defined
    2017-08-11 jwe Summary[octave-forge] sockets function MSG_DONTWAIT and MSG_WAITALL should have error fallbacks defined [octave-forge] (sockets) function MSG_DONTWAIT and MSG_WAITALL should have error fallbacks defined
    2017-08-11 jwe Summarysockets package: function MSG_DONTWAIT and MSG_WAITALL should have error fallbacks defined [octave-forge] sockets function MSG_DONTWAIT and MSG_WAITALL should have error fallbacks defined
    2017-06-21 lostbard Attached File- Added bug51273.patch, #41014
        StatusConfirmed Patch Submitted
        Carbon-Copy- Added pauldreik
    2017-06-20 mtmiller Priority5 - Normal 3 - Low
        StatusNeed Info Confirmed
        Summarysockets package function recv flag MSG_DONTWAIT undefined sockets package: function MSG_DONTWAIT and MSG_WAITALL should have error fallbacks defined
    2017-06-20 andy1978 StatusNone Need Info

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code