Re: [O] babel C, C++, D enhancement

Le 17/06/2014 19:16, Achim Gratz a écrit : > Thierry Banel writes: >> You are right, INT_MAX is the C++ constant to compare to. >> It is defined in limits.h >> >> I'm not sure I want to locate limits.h (where is it ? are there several >> versions ?), >> and parse it, all from within Emacs-lisp... >

Re: [O] babel C, C++, D enhancement

Thierry Banel writes: > You are right, INT_MAX is the C++ constant to compare to. > It is defined in limits.h > > I'm not sure I want to locate limits.h (where is it ? are there several > versions ?), > and parse it, all from within Emacs-lisp... No, this isn't something you should even try. The

Re: [O] babel C, C++, D enhancement

t...@tsdye.com (Thomas S. Dye) writes: > Aloha Thierry, > > How about a user-customizable variable with a default value of > 2147483647? > > All the best, > Tom > > Thierry Banel writes: > >> Hi Achim. >> >> You are right, INT_MAX is the C++ constant to compare to. >> It is defined in limits.h >>

Re: [O] babel C, C++, D enhancement

Aloha Thierry, How about a user-customizable variable with a default value of 2147483647? All the best, Tom Thierry Banel writes: > Hi Achim. > > You are right, INT_MAX is the C++ constant to compare to. > It is defined in limits.h > > I'm not sure I want to locate limits.h (where is it ? are

Re: [O] babel C, C++, D enhancement

Hi Achim. You are right, INT_MAX is the C++ constant to compare to. It is defined in limits.h I'm not sure I want to locate limits.h (where is it ? are there several versions ?), and parse it, all from within Emacs-lisp... And this will have to be done for all languages (Python, Perl, Java, and s

Re: [O] babel C, C++, D enhancement

Thierry Banel writes: > So Babel C++ may cause problem for large integers. > I am not sure how we can fix this in any case. You'd need to know INT_MAX and give an error for larger values or use an integral type that is large enough to handle Emacs' integer (which would most likely be necessary for

Re: [O] babel C, C++, D enhancement

Thanks Achim for finding and fixing this bug. I do agree about fuzziness between int and float. My Emacs considers 2305843009213693951 as an integer, 2305843009213693952 too large to be an integer. My C++ compiler sees 4294967295 as an integer, 4294967296 as too large to fit an int. So Ba

Re: [O] babel C, C++, D enhancement

Bastien writes: > Hi Eric and Thierry, > > Eric Schulte writes: > >> This new patch looks great, and the test suite passes locally. I've >> just applied it. > > Thanks for applying this -- let me just be boring again and insist on > properly rewrite the Changelog when applying patches... in thi

Re: [O] babel C, C++, D enhancement

Achim Gratz writes: > Eric Schulte writes: >> This new patch looks great, and the test suite passes locally. I've >> just applied it. > > You also get a warning from the byte-compiler on something that is > clearly a bug. I think the fix should be: > > --8<---cut here---

Re: [O] babel C, C++, D enhancement

Hi Eric and Thierry, Eric Schulte writes: > This new patch looks great, and the test suite passes locally. I've > just applied it. Thanks for applying this -- let me just be boring again and insist on properly rewrite the Changelog when applying patches... in this case, there are a lot of tiny

Re: [O] babel C, C++, D enhancement

Eric Schulte writes: > This new patch looks great, and the test suite passes locally. I've > just applied it. You also get a warning from the byte-compiler on something that is clearly a bug. I think the fix should be: --8<---cut here---start->8--- diff --git

Re: [O] babel C, C++, D enhancement

Good ! Now I will update the documentation (worg/org-contrib/babel/languages/ob-doc-C.org) and... that's it. Le 09/06/2014 01:22, Eric Schulte a écrit : > Hi Theirry, > > This new patch looks great, and the test suite passes locally. I've > just applied it. > > Thanks! > Eric > > also, thanks for

Re: [O] babel C, C++, D enhancement

Thierry Banel writes: > Hi Eric, hi All > > Here is a new patch (do not apply the previous). > I added a check in the test suite for the D compiler availability > (and for the C++ compiler as well). > It silently returns true when the compiler is not found. > > To run the test suite I did that: >

Re: [O] babel C, C++, D enhancement

Hi Eric, hi All Here is a new patch (do not apply the previous). I added a check in the test suite for the D compiler availability (and for the C++ compiler as well). It silently returns true when the compiler is not found. To run the test suite I did that: - start from an up-to-date org-mode/ GI

Re: [O] babel C, C++, D enhancement

Thanks Eric for your feed-back. Bastien told me the same as you about the commit message. So I will take time to clear all those issues. Le 06/06/2014 18:31, Eric Schulte a écrit : > Hi Thierry, > > This looks wonderful, however after applying this patch I get the > following errors when running

Re: [O] babel C, C++, D enhancement

Hi Thierry, This looks wonderful, however after applying this patch I get the following errors when running "make test-dirty". > 10 unexpected results: >FAILED ob-C/inhomogeneous_table >FAILED ob-D/inhomogeneous_table >FAILED ob-D/integer-var >FAILED ob-D/list-list-var >FA

[O] babel C, C++, D enhancement

Hi Bastien, hi all Here is a Babel enhancement for C, C++ and D languages - support for non-homogeneous tables (mixing numerics and strings) - support for table headers - easier iterating over a table - added some error handling - new D support for simple lists and vectors - new unit tests Curren