Hi All,

With Derek's patch, the documentation for destiny branch is more
useful for those using Python.  Consequently, I have pushed the
doxygen documentation for destiny (which include Derek's patch) to
replace the one previously generated from zaku.  FYI.

Regards
KK

On 11 January 2011 11:43, kk yap <yap...@stanford.edu> wrote:
> Hi Derek,
>
> Thanks.  Great patches as usual.  I have pushed them to the destiny branch.
>
> Martin and Murphy, I believe I have pushed most (if not all) of
> Derek's patches at this moment.  If anything else is missing, let me
> know.
>
> Thanks.
>
> Regards
> KK
>
> On 11 January 2011 07:23, Martin Casado <cas...@nicira.com> wrote:
>> 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<derek.corm...@lab.ntt.co.jp>  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<derek.corm...@lab.ntt.co.jp>
>>   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
>> nox-dev@noxrepo.org
>> http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org
>>
>>
>>
>>
>>
>>
>> _______________________________________________
>> nox-dev mailing list
>> nox-dev@noxrepo.org
>> 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
nox-dev@noxrepo.org
http://noxrepo.org/mailman/listinfo/nox-dev_noxrepo.org

Reply via email to