Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-08-01 Thread Nicolas Goaziou
Hello, Jay Kamat writes: > Sorry for bumping this, but could you apply the patch I made to update > worg for documentation as well (or provide feedback)? I'd like that to > stay updated as well if possible. Applied. Thank you. Regards, -- Nicolas Goaziou

Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-08-01 Thread Jay Kamat
Hi Nicolas, Sorry for bumping this, but could you apply the patch I made to update worg for documentation as well (or provide feedback)? I'd like that to stay updated as well if possible. If there's another mailing list for worg let me know and I'll post it there. The patch is re-attached for co

Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-08-01 Thread Nicolas Goaziou
Hello, Jay Kamat writes: > From 65c8add92842c2e547b13d5c6fb2485e7fdbf2db Mon Sep 17 00:00:00 2001 > From: Jay Kamat > Date: Mon, 31 Jul 2017 22:47:33 -0700 > Subject: [PATCH] ORG-NEWS: Fix a small typo in new org babel header example > > * etc/ORG-NEWS (Using): Fix typo in babel :header-args: c

Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-08-01 Thread Nicolas Goaziou
Jay Kamat writes: > Hi Chuck, > > Thanks for pointing that out, this all makes a lot more sense now. > Prior to this patch ':includes' and ':defines' seem to work in the > property list (as described in the old version of the org manual), so > (I think) the updated version of this patch also remo

Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-08-01 Thread Nicolas Goaziou
Hello, Jay Kamat writes: > Let me know if you spot anything fishy or wrong. It's good, thank you. I applied your patch. > > Also, RE: Copyright, this is the form I need to send, correct? > http://orgmode.org/request-assign-future.txt It is. Please let me know when the process is done. Regards

Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-07-31 Thread Jay Kamat
Hi Chuck, Thanks for pointing that out, this all makes a lot more sense now. Prior to this patch ':includes' and ':defines' seem to work in the property list (as described in the old version of the org manual), so (I think) the updated version of this patch also removes those usages (for :defines,

Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-07-31 Thread Charles C. Berry
On Mon, 31 Jul 2017, Jay Kamat wrote: Hello! Sorry for the late reply, I was pretty busy last week. An updated patch is attached! [snip] Using `org-entry-get' is no longer supported. You can replace the `or' with (cdr (assq :namespaces params)) Done, I also replaced the other uses o

Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-07-31 Thread Jay Kamat
Hello! Sorry for the late reply, I was pretty busy last week. An updated patch is attached! > I suggest to add the following to "ob-C.el" so that `org-lint' can issue > a warning whenever :namespaces is used in a C block. Done. I needed to tweak the code a bit though to get it to work and to pre

Re: [O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-07-23 Thread Nicolas Goaziou
Hello, Jay Kamat writes: > However, it would be nice to add a "using namespace std" to this > source code block, so it can become: > > #+BEGIN_SRC C++ :includes :namespaces std > cout << "Hello world\n"; > #+END_SRC > > > Which makes it cleaner and easier to read, especially for very short >

[O] [PATCH] Support for 'using namespace *' in ob-C.el

2017-07-16 Thread Jay Kamat
Hi! A problem that I came across recently was the difficulty of writing C++ tutorials for beginners to programming in org mode. In such tutorials, it would be ideal to keep as much boilerplate away from the examples so they don't ask questions (until we get to those topics). The shortest ob-C++ e