Bug 491451 - Add syntax details for opaque structures
Summary: Add syntax details for opaque structures
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Directory Server
Classification: Red Hat
Component: Doc-plugin-programming-guide
Version: 8.1
Hardware: All
OS: Linux
high
medium
Target Milestone: ---
: ---
Assignee: Marc Muehlfeld
QA Contact: Viktor Ashirov
URL:
Whiteboard:
Depends On:
Blocks: 512820
TreeView+ depends on / blocked
 
Reported: 2009-03-21 05:04 UTC by Deon Ballard
Modified: 2017-02-09 00:52 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-02-09 00:14:37 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Deon Ballard 2009-03-21 05:04:47 UTC
According to Nathan's comments for the Slapi_Counter edits in bug 458190, the new write-up for Slapi_Counter includes the private definition. The public header (slapi-plugin.h) doesn't have any of this detail in it. For example:

#include "slapi-plugin.h" 
typedef struct Slapi_Counter {
    PRUint64 value;
#ifndef ATOMIC_64BIT_OPERATIONS
    Slapi_Mutex *mutex;
#endif
} Slapi_Counter;

We should add something similar to the Syntax sections for other opaque structures, such as the Slapi_PBlock.

Comment 1 Deon Ballard 2009-09-02 14:22:33 UTC
Changing the status from "new" to "assigned."

Comment 3 Deon Ballard 2009-09-22 17:36:44 UTC
Setting to block the DS9 docs tracking bug.

Comment 9 Deon Ballard 2014-07-10 16:49:03 UTC
Reassigning to Tomas.

Comment 10 Noriko Hosoi 2017-02-09 00:14:37 UTC
It is not worth showing all the fields in the slapi structures.

Plus, there are some reasons we'd better not to do so.
1) The fields are not meant to access directly. Only the API info to access them should be provided.
2) The fields could be updated frequently by debugging or implementing new features.  Having a snapshot of the structure syntax should be rather harmful.

Hence, closing this bug with WONTFIX. (Acked by Nathan.  Thanks!!)

Comment 11 wibrown@redhat.com 2017-02-09 00:52:00 UTC
Another part of this, is that in the future, I'm going to hide and mask these. You should never access struct internals, only the functions related to them.


Note You need to log in before you can comment on or make changes to this bug.