Derek,

Thanks for the patches, these are very helpful. One of us will take a look shortly.

.martin

Hello,
Here are two patches for destiny for the Python API.

patch 1:
- in pyrt.cc, add datapath id to the dictionaries created by Flow_removed_event and Error_event

patch2:
- add lots of doxygen doc strings to core.api
- add callback generators for flow mod and flow removed events
- change the names of callback generators to make them more readable
- add more constants for dictionary keys

-Derek

On 01/04/2011 10:27 AM, kk yap wrote:
Hi Derek,

Thanks.  Doxygen comments are presented in the code.  E.g.,

/** \brief Event
  * Some event
  *
  * @author ykk
  * @date Jan 2011
  */
struct x_event
{
   ...
}

They are more in-depth example in the components.

Regards
KK

On 3 January 2011 17:19, Derek Cormier<[email protected]> wrote:
Sure thing. Where do the doxygen comments go? I've never used doxygen
before, I'm assuming they go above the struct definition but I don't see any
such comments in the files I edited. Is there another file?

Thanks,
-Derek

On 01/04/2011 06:20 AM, kk yap wrote:
Hi Derek,

Sorry it took me a while to get to this.  Can you put in doxygen
comments for these patches?  The patches includes new structs and so
on from what I can see.  Else, I can do it.  But I doubt I can get to
it fast enough.  Thanks.

Regards
KK

On 27 December 2010 23:23, Derek Cormier<[email protected]>
  wrote:
Here's a couple more patches for destiny.

'barrier-reply.tar.gz'  -  make barrier reply events available to the
Python
API.
'flow-stats-in.tar.gz'  -  add a convenience method in component to
register
for flow stats in events.

-Derek

On 12/28/2010 01:30 AM, Martin Casado wrote:
Thanks KK.  I pushed a few edits as well.

Hi,

I have done that for destiny and pushed the file. Martin, you should probably check the list again. If I have missed you out (since I only
exploited git log), do let me know.

Regards
KK

_______________________________________________
nox-dev mailing list
[email protected]
http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org






_______________________________________________
nox-dev mailing list
[email protected]
http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org


--
~~~~~~~~~~~~~~~~~~~~~~~~~~~
Martin Casado
Nicira Networks, Inc.
www.nicira.com | www.openvswitch.org
cell: 650-776-1457
~~~~~~~~~~~~~~~~~~~~~~~~~~~

_______________________________________________
nox-dev mailing list
[email protected]
http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org

Reply via email to