This is an automatically generated e-mail. To reply, visit: http://reviewboard-openlmi.rhcloud.com/r/1295/

doc/src/indications.rst (Diff revision 1)
87
If you want to delete the subscriptions sooner, you can use following methods:
138
If you want to delete the subscriptions sooner, you can use the following method:
88
139
89
.. code-block:: python
140
.. code-block:: python
90
141
91
    > c.unsubscribe_indication(indication_name)
142
    > c.unsubscribe_indication(unique_name)
92
    LMIReturnValue(rval=True, rparams={}, errorstr="")
143
    LMIReturnValue(rval=True, rparams={}, errorstr="")
93
    > c.unsubscribe_all_indications()
144
    > c.unsubscribe_all_indications()
I think that code actually shows two different method*s*, not just one. But sadly they are just in one code snippet without any comment making them hard to discern. I'd suggest splitting the example into two small ones with titles prepended.

- Michal Minar


On November 26th, 2013, 7:08 p.m. UTC, Stephen Gallagher wrote:

Review request for OpenLMI Developers.
By Stephen Gallagher.

Updated Nov. 26, 2013, 7:08 p.m.

Repository: openlmi-tools

Description

Improve lmishell indication documentation

Diffs

  • cli/lmi/shell/LMIIndicationListener.py (1ec2f21c21291153289e8184026be7c69b27d504)
  • doc/src/indications.rst (d29a77d96f834cb8e9ba2654ba8e8db016608d48)

View Diff