patchGNU Octave - Patches: patch #8239, Remove setting value unnecessarily...


patch #8239: Remove setting value unnecessarily to retval in

Submitted by:  PrasannaKumar Muralidharan <prasannatsm>
Submitted on:  Sat 16 Nov 2013 05:02:03 AM UTC  
Category: NonePriority: 5 - Normal
Status: DonePrivacy: Public
Assigned to: NoneOpen/Closed: Closed

Add a New Comment (Rich MarkupRich Markup):

You are not logged in

Please log in, so followups can be emailed to you.


Sun 04 May 2014 01:55:20 PM UTC, comment #3:

The mailing list is a good place to discuss these sorts of things. Sometimes it can be pretty hard to determine what the best thing to do is. In this case, I started to apply your patch and only then noticed that what the original coder had probably intended was for Octave to exit.

Rik <rik5>
Project Administrator
Sun 04 May 2014 04:05:30 AM UTC, comment #2:

I was not aware of how octave should behave when setsid() fails. It will be good if I could understand such things before submitting a patch / bug report.

What is the best way to discuss such details? Can this be discussed in mailing list?

PrasannaKumar Muralidharan <prasannatsm>
Sun 04 May 2014 12:11:04 AM UTC, comment #1:

I took a slightly different approach. It appears that Octave was really meant to abort if setsid() failed so the assignment to retval was accurate. See this changeset ( Thanks for pointing this out.

Rik <rik5>
Project Administrator
Sat 16 Nov 2013 05:02:03 AM UTC, original submission:
PrasannaKumar Muralidharan <prasannatsm>


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

Attach File(s):

Attached Files


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by prasannatsm (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Please enter the title of George Orwell's famous dystopian book (it's a date):



    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 04 May 2014 12:11:04 AM UTCrik5StatusNone=>Done
    Sat 16 Nov 2013 05:02:03 AM UTCprasannatsmAttached File-=>Added remove_unnecessary_setting_value.diff, #29629

    Back to the top

    Powered by Savane 3.1-cleanup