subutux / json-rpc2php

A json-rpc 2.0 server for php and some clients for php,js,python and vala.
GNU General Public License v2.0
46 stars 26 forks source link

I don't use classes in my JSON-RPC calls, so I made them optional #8

Closed scottchiefbaker closed 10 years ago

scottchiefbaker commented 10 years ago

Couple other minor changes

1) <? to <?php

short tags are off in most environments

2) There are a lot of print/echo/print_r in the client code? It makes the output really noisy so I removed them. They should probably be behind a $debug option.

subutux commented 10 years ago

Thanks for the cleanup!

scottchiefbaker commented 10 years ago

Happy to help. Anything else I can help with? Any cleanup that needs doing?

subutux commented 10 years ago

Not that I can think of.Mvg,

Stijn

On Fri, Feb 7, 2014 at 12:16 AM, Scott Baker notifications@github.com wrote:

Happy to help. Anything else I can help with? Any cleanup that needs doing?

Reply to this email directly or view it on GitHub: https://github.com/subutux/json-rpc2php/pull/8#issuecomment-34336639