Memory analyzer void safety updates at github

[Anders Persson]

Hi

I have added my changes to github for review and to make it eassier for me and Victorien to collaborate since memeory analyzer library is dependent on the graph library.

You can find the code at:

https://github.com/andersoxie/memory_analyzer/

and the changes are best viewed at

https://github.com/andersoxie/memory_analyzer/pull/1

Unfortunatelly even if some changes are minor the diffs presented by github is big. It is when I have added if-statements like

if attached...

Regards

Anders

[Emmanuel Stapf]

Even though it is not perfect, we will use https://codereview.appspot.com/ for reviewing the code. Can you submit your patch there?

Thanks,

Manu

[Anders Persson]

Hi

I will probably be able to do it tomorrow.

Anders

--
For more messaging options, visit this group at http://forum.eiffel.com.
Information on the Eiffelstudio project: http://dev.eiffel.com.

[Anders Persson]

Hi

I am not sure, after reading the documentation, which way is best to upload a patch. How have you done it, used upload.py? Could you write an example command row how you do it?

Is it possible to upload anything without installing python?

Regards

Anders

[Emmanuel Stapf]

You do not have much choice on that, but if you send me the svn diff results, I can upload it for you.

 

Regards,

Manu

 

--

[Anders Persson]

Hi

I have sent you a patch.

Anders

[Emmanuel Stapf]

I’ve uploaded the patch. Can you check that you have access to:

 

https://codereview.appspot.com/81910043/

 

Manu

 

From: eiffelst...@googlegroups.com [mailto:eiffelst...@googlegroups.com] On Behalf Of Anders Persson
Sent: Friday, March 28, 2014 09:13
To: eiffelst...@googlegroups.com
Subject: [EiffelStudio Dev] Re: Memory analyzer void safety updates at github

 

Hi

--

[Anders Persson]

I can access it. Thanks

Anders