{"action":"opened","issue":{"url":"https://api.github.com/repos/marshallward/f90nml/issues/12","labels_url":"https://api.github.com/repos/marshallward/f90nml/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/marshallward/f90nml/issues/12/comments","events_url":"https://api.github.com/repos/marshallward/f90nml/issues/12/events","html_url":"https://github.com/marshallward/f90nml/issues/12","id":56429366,"number":12,"title":"Cosmetic Ideas","user":{"login":"jacobwilliams","id":6483379,"avatar_url":"https://avatars.githubusercontent.com/u/6483379?v=3","gravatar_id":"","url":"https://api.github.com/users/jacobwilliams","html_url":"https://github.com/jacobwilliams","followers_url":"https://api.github.com/users/jacobwilliams/followers","following_url":"https://api.github.com/users/jacobwilliams/following{/other_user}","gists_url":"https://api.github.com/users/jacobwilliams/gists{/gist_id}","starred_url":"https://api.github.com/users/jacobwilliams/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jacobwilliams/subscriptions","organizations_url":"https://api.github.com/users/jacobwilliams/orgs","repos_url":"https://api.github.com/users/jacobwilliams/repos","events_url":"https://api.github.com/users/jacobwilliams/events{/privacy}","received_events_url":"https://api.github.com/users/jacobwilliams/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-02-03T20:06:36Z","updated_at":"2015-02-03T20:06:36Z","closed_at":null,"body":"Everything seems to work great now.  So, now all I have are some possible cosmetic suggestions.  These are somewhat minor, so feel free to ignore them.\r\n* Maybe have the ability to specify for the ```write``` method the string to use for the True and False values.  Currently it always uses ```.true.``` and ```.false.```.  Intel (and probably others), also accepts ```T``` and ```F```.  \r\n* Maybe also have the option to specify if we want to force a comma to be written after each variable.  It looks like currently it only does this for strings that contain commas and to separate array elements?  The compiler seems to be OK with that.  However, somebody out there may be parsing namelists and expecting the ending commas (I think I had something like that at one point).\r\n* Maybe allow caller to specify the number of spaces to be used for indenting?\r\n\r\nPerhaps these could be optional arguments?  Something like:\r\n````Python\r\nn = f90nml.read('test.nml')\r\nn.write( 'test2.nml', force=True, indent=2, true_str='T', false_str='F', force_commas=True )\r\n````\r\nAgain: minor stuff, but maybe nice to have (there could be other options as well)."}}