Hugh Gibson wrote: >> The implementation of having the contained block be aligned with the >> curlies wouldn't be too difficult. But I would probably suggest an >> option that reflects more on the block being indented or not (e.g. >> something like "--pretty-print-indent-block-..."), rather than on the >> closing curly. >> > > OK. Is that going to be feasible before 0.7 release? If not, I'll implement > something like it locally anyway as I don't want to have to manually reformat > 300k of code... > > >> That's definitely how I think about it (I would pass in the complete >> 'options' object, though). But as I said, the write() function might >> go away altogether in the re-write. But I'm open - if everybody agrees >> to have this feature in 0.7 I'm more than happy to join forces with you >> to implement it in the current version. - Andreas, Sebastian? >> > > Again, I'll be writing this into the pretty-printer anyway locally if you > decide not to support it - better half a day spent developing code to do > things correctly, than a few days correcting code afterwards (though we have > a macro to fix up the column in our preferred editor). I like the list of > positions - that could make our code even neater! > Ok, let's split the tasks. I'll take care of the block indenting. - Can you commit to SVN? Then you could do the column handling of comments, and I'll help you with testing and stuff. Otherwise you can send me a patch or just your version. How does that sound?
=Thomas ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ qooxdoo-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel
