All of them seem to invert based on Screen index 0, which makes it sound as if it will fail on a dual monitor system, but it worked fine on my dual monitor system. We should probably look to centralize this somewhere for one thing (there is an existing flipFrame method in one of the files, but it isn't clear if it can be used in these cases). It isn't used now that I'm done debugging, but I thought I might leave it in there, commented out, in case we need to debug more stuff in the future.?Īn oddity about the code used in the fix is that there are a few places in the code that do this inversion and: wrote some code to be able to inject debugging output into the System.out stream so that it could be seen in context in the Java plugin logs. fixed some typos in the _getEmbeddedY() native method that I saw as I was adding debug print statements to it Note that I left a couple of other fixes in the webrev, but they can be deleted to reduce the scope if needed. ![]() There were a lot of really confusing symptoms here and it was hard to figure out how they all could have boiled down to one failure to invert the Y coordinate system, but fixing this one piece of code fixed all of the popups:
0 Comments
Leave a Reply. |
AuthorWrite something about yourself. No need to be fancy, just an overview. ArchivesCategories |