FAQ promotes extremely bad backward-compatibility practice
GoogleCodeExporter opened this issue · comments
Google Code Exporter commented
What steps will reproduce the problem?
https://code.google.com/p/androidsvg/wiki/FAQ#My_SVG_doesn't_render_if_I_try_to_
do_Canvas.drawPicture()_o
What is the expected output? What do you see instead?
Instead of the horrible reflection, please replace it with one of the following
solutions (or both).
Note how the two code blocks are simpler and shorter (even together) than the
current one.
If the project is using support-v4:
import android.support.v4.view.ViewCompat;
ViewCompat.setLayerType(view, ViewCompat.LAYER_TYPE_SOFTWARE, null);
If it's a plain android project targeting API 11+ (which is where the method
was added, not API 13):
import android.os.Build.*;
@TargetApi(VERSION_CODES.HONEYCOMB)
public static void setSoftwareLayerType(View view) {
if (VERSION_CODES.HONEYCOMB <= VERSION.SDK_INT) {
view.setLayerType(View.LAYER_TYPE_SOFTWARE, null);
} // else: no layer support, so no hardware support to disable
}
Original issue reported on code.google.com by papp.robert.s
on 7 Dec 2014 at 9:28
Google Code Exporter commented
I just noticed another point against the current solution: you can simply use
getField(...).getInt(null) to get a static field's value, no need to create a
new View(getContext()).
Original comment by papp.robert.s
on 7 Dec 2014 at 9:31
Google Code Exporter commented
Thanks for that. I didn't know that ViewCompat had that method. However I
should have updated that answer with the other new way of doing it.
I've updated the answer now.
I've kept the old-style way in the FAQ just in case anyone is still using an
old build environment.
Thanks again Robert.
Original comment by paul.leb...@gmail.com
on 8 Dec 2014 at 11:35
- Changed state: Fixed
Google Code Exporter commented
Ahaa, I didn't know the @TargetApi was added in 16 :)
Still the SDK_INT was added in API 4 and I don't think anyone is targeting less
than that... so even if @TargetApi is not available because of old target the
Build.VERSION method works.
@TargetApi is only used to silence lint, like @SuppressLint("NewApi") which is
API 16 too. I guess before API 16 the lint.xml was used to make it shut up when
SDK_INT is used to make stuff compatible.
Original comment by papp.robert.s
on 8 Dec 2014 at 11:55
Google Code Exporter commented
Yeah you are right. That section made no sense. I think I have fixed it now.
:)
Original comment by paul.leb...@gmail.com
on 8 Dec 2014 at 12:26
Google Code Exporter commented
Some of the numbers and terminology is still strange, here're the paragraphs
fixed to my best knowledge:
If you are using the v4 support library in your project (also included with
latest v7 appcompat library), compatibility will be taken care of for you:
If you are targeting API 4 or later, you can use the constants in the
android.os.Build class to prevent setLayerType() being invoked when it is not
available on the user's device.
Please note that the @TargetApi annotation was added in API 16, so if your
target is between API 11 and API 15 and your minSdkVersion is less than API 11
you'll have to find another way to suppress the lint warning about
setLayerType() not being available.
In almost all cases you will be able to use one of the first two approaches. We
recommend you use one of them if you can. However some people may be forced for
some reason to use Java reflection to call the setLayerType() method if it is
available.
private void setSoftwareLayerType() {
try {
Method setLayerTypeMethod = View.class.getMethod("setLayerType", Integer.TYPE, Paint.class);
int LAYER_TYPE_SOFTWARE = View.class.getField("LAYER_TYPE_SOFTWARE").getInt(null);
setLayerTypeMethod.invoke(this, LAYER_TYPE_SOFTWARE, null);
} catch (NoSuchMethodException e) {
// Android platform too old, no setLayerType
} catch (NoSuchFieldException e) {
// Android platform too old, no LAYER_TYPE_SOFTWARE
} catch (Exception e) {
Log.w("MySVGView", "Unexpected failure calling setLayerType", e);
}
}
Original comment by papp.robert.s
on 8 Dec 2014 at 12:50
Google Code Exporter commented
Thanks. I have incorporated your suggestions.
> If you are targeting API 4 or later, you can use the constants...
Yes, the constants are available in API 4, but you can't use
View.setLayerType() till API 11.
Original comment by paul.leb...@gmail.com
on 8 Dec 2014 at 1:29