Eiffel Graph complete void-safe

[Victorien Elvinger]

Hi,

The complete void-safe version of Eiffel Graph is available at:

https://github.com/Conaclos/EG-complete-safe/

A branch with more advanced redesign is available at:

https://github.com/Conaclos/EG-complete-safe/tree/redesign_1

Note: The Memory Analyzer void-safe version of Anders, compiles with both versions.

Best regards,

Victorien ELVINGER

[Emmanuel Stapf]

Could you send me a patch for both changes so that we can review it as well?

Thanks,

Manu

[Victorien Elvinger]

Hi,

The two patches are in the attched zip.

Note: First commits don't follow the local style guideline of Eiffel Software (al_ used instead of l_). A style update commit change it.

Best regards and good reviews.

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

--

Victorien ELVINGER
Responsable du F'ESSTIN des jeux 2011 ... 2013
Élève en master 2 Services et Sécurité des Systèmes et des Réseaux
Élève ingénieur ESSTIN en 5e année - Systèmes d'Information et Réseaux

[Emmanuel Stapf]

Thanks for the zip however that’s a lot of small patches. Could you instead send me the combined patch?

 

Thanks,

Manu

[Victorien Elvinger]

Hi again,

The patches in the attached Zip are unified commits, as requested ;)

Best regards,

[Emmanuel Stapf]

Thanks. I’ve only put the first patch for review at https://codereview.appspot.com/83240043/ because the second one has too many changes with respects to style and we have changed our internal rules so we should first review the important stuff before reviewing the styles anyway.

 

To clarify the new style guidelines, we have decided that all comments should be terminated by a punctuation mark. This is why we do not remove the final . anymore.

[Emmanuel Stapf]

Hi Victorien and Anders,

It has been a while and although we could not integrate your void-safety improvements for the 14.05 release, it is time to do this for next release. Could you update against the latest source code (most likely no major changes) and submit a final patch?

I'll take it from there and commit this in the next few weeks.

Thanks,

Manu

[Anders Persson]

Hi

I am not able to do  anything for some weeks due to that I am not at home.

Anders

--

[Emmanuel Stapf]

That’s ok. Do you remember if you have changed anything since last time you put your patch? If not I can take it from there. I just don’t want to lose any changes you may have done in between.

 

Thanks,

Manu

 

From: eiffelst...@googlegroups.com [mailto:eiffelst...@googlegroups.com] On Behalf Of Anders Persson
Sent: Friday, July 25, 2014 16:08
To: eiffelst...@googlegroups.com
Subject: Re: [EiffelStudio Dev] Re: Eiffel Graph complete void-safe

 

Hi

[Victorien Elvinger]

Hi,

I take a look for the next week...

I must check if the last changes are compatible with Anders 's work...

Best regards,

Victorien ELVINGER
Master Thesis at Chair of Software Engineering, ETH Zürich
Master Student in Services et Sécurité des Systèmes et des Réseaux, Université de Lorraine
Engineer Student at ESSTIN - Systèmes d'Information et Réseaux, 

Responsible for F'ESSTIN des jeux 2011 ... 2013

[Anders Persson]

Hi, I have not made any changes since last patch.

Anders

2014-07-26 1:29 GMT+02:00 Emmanuel Stapf <ma...@eiffel.com>:

[Victorien Elvinger]

Hi,

I didn't send the last pull-request to Anders on github to ensure compatibility with the last void-safty redesigns.

Now the pull-request is open at https://github.com/andersoxie/memory_analyzer/pull/3

I attached:

- Eiffel Graph patch for complete void-safety

- Memory ANalyzer patch for complete void-safety, including Anders's changes and my pull-request for compatibilty purpose

If Anders is ok with the last changes, you can use these patches...

Best regards,