[xorg-bugzilla-noise] [Bug 1498] New: Wrong description in XtChangeManagedSet manpage

[bugzilla-daemon at freedesktop.org]

Please do not reply to this email: if you want to comment on the bug, go to          
the URL shown below and enter yourcomments there.   
 
https://freedesktop.org/bugzilla/show_bug.cgi?id=1498        
   
           Summary: Wrong description in XtChangeManagedSet manpage
           Product: xorg
           Version: CVS_head
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Lib/Xt
        AssignedTo: xorg-bugzilla-noise at freedesktop.org
        ReportedBy: D.Yu.Bolkhovityanov at inp.nsk.su


The manual page states that XtChangeManagedSet() parameters are (in this order):

unmanage_children, num_unmanage_children, manage_children, num_manage_children,
post_unmanage_pre_manage_hook, client_data

But looking at the sources and Composite.h file reveals that the order is
actually different (taken from cvsweb):

extern void XtChangeManagedSet(
    WidgetList          /* unmanage_children */,
    Cardinal            /* num_unmanage_children */,
    XtDoChangeProc      /* do_change_proc */,
    XtPointer           /* client_data */,
    WidgetList          /* manage_children */,
    Cardinal            /* num_manage_children */
);

I.e. that the do_change_proc+client data pair is BETWEEN the lists, not AFTER them.

And, of course, following the manual page makes program SIGSEGV.

This error seems to be there from the very introduction of XtChangeManagedSet().

I've reported this bug to XFree86 (bugreport #1363) and it was fixed there, but
is still present in X.org tree (since the fix was applied AFTER XFree86/X.org
split).        
   
   
--         
Configure bugmail: https://freedesktop.org/bugzilla/userprefs.cgi?tab=email       
   
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.