Fix Bug 1057: Emacs output does not escape special characters.
This commit is contained in:
parent
3a2eb94bef
commit
db73e7af9e
2 changed files with 69 additions and 62 deletions
22
src/emacs.cc
22
src/emacs.cc
|
|
@ -30,7 +30,7 @@
|
||||||
*/
|
*/
|
||||||
|
|
||||||
#include <system.hh>
|
#include <system.hh>
|
||||||
|
#include <boost/algorithm/string.hpp>
|
||||||
#include "emacs.h"
|
#include "emacs.h"
|
||||||
#include "xact.h"
|
#include "xact.h"
|
||||||
#include "post.h"
|
#include "post.h"
|
||||||
|
|
@ -38,8 +38,8 @@
|
||||||
|
|
||||||
namespace ledger {
|
namespace ledger {
|
||||||
|
|
||||||
void format_emacs_posts::write_xact(xact_t& xact)
|
void format_emacs_posts::write_xact(xact_t& xact)
|
||||||
{
|
{
|
||||||
if (xact.pos)
|
if (xact.pos)
|
||||||
out << "\"" << xact.pos->pathname.string() << "\" "
|
out << "\"" << xact.pos->pathname.string() << "\" "
|
||||||
<< xact.pos->beg_line << " ";
|
<< xact.pos->beg_line << " ";
|
||||||
|
|
@ -62,10 +62,10 @@ void format_emacs_posts::write_xact(xact_t& xact)
|
||||||
out << "\"" << xact.payee << "\"";
|
out << "\"" << xact.payee << "\"";
|
||||||
|
|
||||||
out << "\n";
|
out << "\n";
|
||||||
}
|
}
|
||||||
|
|
||||||
void format_emacs_posts::operator()(post_t& post)
|
void format_emacs_posts::operator()(post_t& post)
|
||||||
{
|
{
|
||||||
if (! post.has_xdata() ||
|
if (! post.has_xdata() ||
|
||||||
! post.xdata().has_flags(POST_EXT_DISPLAYED)) {
|
! post.xdata().has_flags(POST_EXT_DISPLAYED)) {
|
||||||
if (! last_xact) {
|
if (! last_xact) {
|
||||||
|
|
@ -103,13 +103,19 @@ void format_emacs_posts::operator()(post_t& post)
|
||||||
if (post.cost)
|
if (post.cost)
|
||||||
out << " \"" << *post.cost << "\"";
|
out << " \"" << *post.cost << "\"";
|
||||||
if (post.note)
|
if (post.note)
|
||||||
out << " \"" << *post.note << "\"";
|
out << " \"" << escape_string(*post.note) << "\"";
|
||||||
out << ")";
|
out << ")";
|
||||||
|
|
||||||
last_xact = post.xact;
|
last_xact = post.xact;
|
||||||
|
|
||||||
post.xdata().add_flags(POST_EXT_DISPLAYED);
|
post.xdata().add_flags(POST_EXT_DISPLAYED);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
string format_emacs_posts::escape_string(string raw){
|
||||||
|
replace_all(raw, "\\", "\\\\");
|
||||||
|
replace_all(raw, "\"", "\\\"");
|
||||||
|
return raw;
|
||||||
|
}
|
||||||
|
|
||||||
} // namespace ledger
|
} // namespace ledger
|
||||||
|
|
|
||||||
|
|
@ -72,6 +72,7 @@ public:
|
||||||
out.flush();
|
out.flush();
|
||||||
}
|
}
|
||||||
virtual void operator()(post_t& post);
|
virtual void operator()(post_t& post);
|
||||||
|
virtual string escape_string(string raw);
|
||||||
};
|
};
|
||||||
|
|
||||||
} // namespace ledger
|
} // namespace ledger
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue