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

Leading and trailing spaces in language resource file were removed #124

Closed
iBotPeaches opened this Issue Mar 18, 2015 · 3 comments

Comments

Projects
None yet
1 participant
@iBotPeaches
Owner

iBotPeaches commented Mar 18, 2015

Original issue 14 created by mkzhang1979 on 2010-04-16T19:23:39.000Z:

  1. Decompile Mms.apk.
  2. Open Mms\res\values\strings.xml, you will see some entries with
    trailing space. For example, <string name="forward_prefix">Fwd: </string>
  3. Build Mms foler to APK without any change.
  4. Decompile the new out.apk.
  5. Open out\res\values\strings.xml, check if the trailing spaces are
    preserved.

[Expected]
Spaces are preserved.

[Actual]
Spaces are removed.

I'm using apktool-1.0.0.tar.bz2 and apktool-install-windows-2.1_r01-1.zip
released on April 2. My PC is running Windows 7.

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Mar 18, 2015

Owner

Comment #1 originally posted by Brut.alll on 2010-04-17T00:03:01.000Z:

Yes, aapt ignores whitespaces in XML resources, I tried to escape them somehow, but
failed. Thanks to you, now I know, how to do it properly :-)

Fwd:\u0020

It's from Android sources.

Owner

iBotPeaches commented Mar 18, 2015

Comment #1 originally posted by Brut.alll on 2010-04-17T00:03:01.000Z:

Yes, aapt ignores whitespaces in XML resources, I tried to escape them somehow, but
failed. Thanks to you, now I know, how to do it properly :-)

Fwd:\u0020

It's from Android sources.

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Mar 18, 2015

Owner

Comment #2 originally posted by mkzhang1979 on 2010-04-17T05:42:55.000Z:

Thanks for the workaround ^_^

Owner

iBotPeaches commented Mar 18, 2015

Comment #2 originally posted by mkzhang1979 on 2010-04-17T05:42:55.000Z:

Thanks for the workaround ^_^

@iBotPeaches

This comment has been minimized.

Show comment
Hide comment
@iBotPeaches

iBotPeaches Mar 18, 2015

Owner

Comment #3 originally posted by Brut.alll on 2010-04-28T20:05:43.000Z:

<empty>

Owner

iBotPeaches commented Mar 18, 2015

Comment #3 originally posted by Brut.alll on 2010-04-28T20:05:43.000Z:

<empty>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment