Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update the default runtime behavior #21

Closed
dharple opened this issue Feb 3, 2021 · 2 comments
Closed

Update the default runtime behavior #21

dharple opened this issue Feb 3, 2021 · 2 comments
Assignees
@dharple
Projects
detox v2.0

Comments

@dharple
Copy link
Owner

dharple commented Feb 3, 2021

Update the default behavior of detox (with or without a config file) so that the only tables run are safe and wipeup. Add notes to the config file about the iso8859_1 and utf_8 sequences being transliteration.
@dharple dharple changed the title Update the default behavior Update the default runtime behavior Feb 3, 2021
@dharple dharple added this to To do in detox v2.0 via automation Feb 3, 2021
@dharple dharple moved this from To Do to In Progress in detox v2.0 Feb 13, 2021
dharple added a commit that referenced this issue Feb 13, 2021
@dharple
Transliteration no longer happens by default (#21)
80dc98f
@dharple
Copy link
Owner Author

dharple commented Feb 13, 2021

Round 1 is done.

I still want to:

  • Move table2header from the devdetox repo to the main repo
  • Rewrite clean_string_safe to use a translation table based on safe.tbl

dharple added a commit that referenced this issue Feb 14, 2021
@dharple
Migrate table2header.c from devdetox repo (#21)
ffc2968
dharple added a commit that referenced this issue Feb 14, 2021
@dharple
Migrate check_table.c and dump_table.c from devdetox repo (#21)
00877da
dharple added a commit that referenced this issue Feb 14, 2021
@dharple
Migrate builtin tables to generated builtin tables (#21, #29)
1de725b
dharple added a commit that referenced this issue Feb 14, 2021
@dharple
Switch from ceil() to round() because it isn't universal (#21)
1516166
@dharple dharple self-assigned this Feb 14, 2021
dharple added a commit that referenced this issue Feb 14, 2021
@dharple
Revert "Switch from ceil() to round() because it isn't universal (#21)"
ef90c24 
This reverts commit 1516166.
dharple added a commit that referenced this issue Feb 14, 2021
@dharple
Totally forgot about -lm (#21)
a086af1
@dharple
Copy link
Owner Author

dharple commented Feb 14, 2021

This is done.

@dharple dharple closed this as completed Feb 14, 2021
detox v2.0 automation moved this from In Progress to Done Feb 14, 2021
dharple added a commit that referenced this issue Feb 14, 2021
@dharple
Minor formatting updates (#21)
b376b05
@dharple dharple mentioned this issue Feb 15, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dharple dharple

Labels
None yet
Projects
No open projects
detox v2.0
Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@dharple