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.

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

Modify parse_affix_string() in affixmisc.repy to reject certain AFFIX strings. #21

Open
monzum opened this issue Nov 20, 2013 · 5 comments
Labels

Comments

@monzum
Copy link
Contributor

monzum commented Nov 20, 2013

parse_affix_string() needs to be changed to reject various AFFIX strings that were acceptable before. For example, previously an AFFIX stack was allowed to have other AFFIX components underneath branching/splitter AFFIXs. This is no longer allowed in our new design.

@ghost ghost assigned monzum Nov 20, 2013
@JustinCappos
Copy link
Contributor

How does this function know what is classified as a "branching Affix",
etc.?

On Wed, Nov 20, 2013 at 2:35 PM, monzum notifications@github.com wrote:

parse_affix_string() needs to be changed to reject various AFFIX strings
that were acceptable before. For example, previously an AFFIX stack was
allowed to have other AFFIX components underneath branching/splitter
AFFIXs. This is no longer allowed in our new design.


Reply to this email directly or view it on GitHubhttps://github.com//issues/21
.

@monzum
Copy link
Contributor Author

monzum commented Nov 20, 2013

Unfortunately there is no way to identify a branching AFFIX. I will have to give it some thought and come up with a good solution. My initial though was to check if the arguments of the AFFIX are AFFIX stacks, however I don't believe this is a viable solution.

@JustinCappos
Copy link
Contributor

Right. So I think part of being an Affix is correctly serializing /
deserializing.

Justin

On Wed, Nov 20, 2013 at 3:44 PM, monzum notifications@github.com wrote:

Unfortunately there is no way to identify a branching AFFIX. I will have
to give it some thought and come up with a good solution. My initial though
was to check if the arguments of the AFFIX are AFFIX stacks, however I
don't believe this is a viable solution.


Reply to this email directly or view it on GitHubhttps://github.com//issues/21#issuecomment-28929213
.

@monzum
Copy link
Contributor Author

monzum commented Nov 20, 2013

AFFIXs themselves should correctly serialize/deserialize themselves. Users can still provide a bad AFFIX string when initially building the AFFIX stack. I propose that we have a wiki page or README file that educates users/developers of proper AFFIX stacks and what to do or not to do.

@JustinCappos
Copy link
Contributor

They "should" serialize and deserialize, but we don't need to (and most
likely can't) enforce this. It's much like other semantic properties.

On Wed, Nov 20, 2013 at 5:24 PM, monzum notifications@github.com wrote:

AFFIXs themselves should correctly serialize/deserialize themselves. Users
can still provide a bad AFFIX string when initially building the AFFIX
stack. I propose that we have a wiki page or README file that educates
users/developers of proper AFFIX stacks and what to do or not to do.


Reply to this email directly or view it on GitHubhttps://github.com//issues/21#issuecomment-28937753
.

@monzum monzum removed their assignment Jun 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants