develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (650 of 650 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-5.34%25-green.svg)


Why did you go back to the old controller texture? What was all that fuss about changing it to the other one about?


I prefer the camelCase as i thinks improves the readability of the key.
asaFileCreationResult <--> asafilecreationresult


Peter there is need for updating also [MainActivity](https://github.com/graphhopper/graphhopper/blob/master/android/app/src/main/java/com/graphhopper/android/MainActivity.java#L252) in Android Samples.


I really like the idea. i'm not sure if this may be useful but sure looks nice.
Will do.


@atilaneves: Can you take another look at this?


Try to get in the habit of making more frequent commits. Your commit messages describes at least four different things, that should each be a separate commit.


where is the code ?


a missing require throws LoadError, maybe we should only rescue that


Is this for an event/demo? Please ping me if there are upcoming Cloud Platform stuff, man!


We spoke that `onUnexpectedErrorInStore` has changed to `onUnexpectedStoreError` with error code. Maybe push both errors?


This causes issue #560


Sure, I already added it in later commits :) and removed true here because require return it


Yes, sure. (.oO 'cause they don't know what they're doing...)
Fixup commit is online on my staging branch.


you did see btw my code in the which_dpi function in the updatebinary  I assume? so it is also density-selecting mechanism, but is not as perfect as I would want (it defaults back to highest dpi if there is no matching dpi or universal dpi)


I think there is a reason as to why we don't override `InvalidateOptionsMenu`.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 331](http://192.168.30.225:8111/viewLog.html?buildId=1479&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:26:47


实现的结构和css


Looks great!

There's a gap between the footer and the bottom of the page though:

![](http://i.imgur.com/hZlWajN.png)


Yup I agree it does improve the readability but my suggestion was about making the usage a bit easier for the user.

Right now a user needs to now the case of the keys (which arguably is something internal to the code) to make it work:

```javascript
headers.contentType('followRequest'); // works
headers.contentType('FollowRequest'); // doesn't work
```

If you allow people to make mistakes (or be lazy) you can cover these cases and worse (us updating camel case to upper case for example).

I don't mind anyway so leave it if you don't think it's useful.


```SelectionListener``` - отдельный класс. Тут надо смотреть на метод ```setSliders()```, мы же в его контексте создаем ```new SelectionListener```. А на уровне ```setSliders()``` нам доступны его параметры и атрибуты класса ```ChartFrame```.


Would you like to send a pull request with this commit to my original repository? :)


nice :+1:


hmm seems like a wrong commit ;)


Fixed.


@rldhont Hi, see this two services

http://193.136.216.94/lm211_tests/lizmap/www/index.php/view/

http://193.136.217.85/lm211_tests_master/lizmap/www/index.php/view/

the first uses qgis server 2.8.2 the second master. Both contain two copies of the same project, one has a layout with a legend set to map=map0 and the other set to map=none. On qgis server master when printing the project that has the layout with the legend set to map=none the pdf creation hangs if you don't have all the layers active (so if you don't pass in the url LAYER parameter the complete list of layers that are in the composer legend). Try:

http://193.136.217.85/lm211_tests_master/lizmap/www/index.php/lizmap/service/?repository=montpellier&project=prj290&SERVICE=WMS&VERSION=1.3&REQUEST=GetPrint&FORMAT=pdf&EXCEPTIONS=application/vnd.ogc.se_inimage&TRANSPARENT=true&SRS=EPSG:3763&DPI=300&TEMPLATE=test_layout_legend_map_none&map0:extent=-239795.308000505,-389632.89329657354,132713.64979690503,-9623.755140826455&map0:scale=2500000&map0:grid_interval_x=50000&map0:grid_interval_y=50000&map0:LAYERS=layer1&LAYERS=layer1

compared to:

http://193.136.217.85/lm211_tests_master/lizmap/www/index.php/lizmap/service/?repository=montpellier&project=prj290&SERVICE=WMS&VERSION=1.3&REQUEST=GetPrint&FORMAT=pdf&EXCEPTIONS=application/vnd.ogc.se_inimage&TRANSPARENT=true&SRS=EPSG:3763&DPI=300&TEMPLATE=test_layout_legend_map_none&map0:extent=-239795.308000505,-389632.89329657354,132713.64979690503,-9623.755140826455&map0:scale=2500000&map0:grid_interval_x=50000&map0:grid_interval_y=50000&map0:LAYERS=layer1,layer2&LAYERS=layer1,layer2


Понятно. Тогда с `ChartFrame.this` не слишком красиво выходит. Ок.


KernelNodes should not be used directly, in this case `RespondToNode` (or `DoesRespondDispatchHeadNode` if the method name could change) is likely better.


Instead of:

```
int shiftAmt;
switch (inbyte) {
case 'a':
    shiftAmt =_a;
    break;
}
return shiftAmt;
```

...this:

```
switch (inbyte) {
case 'a':
    return _a;
    }

```

In a massive `switch`/`case` like this, it can really be easier to read!


bugs fixed.


there is HM2014811 to HM2014818


Just a note: phpinfo() does work here with php.ini-production. And it avoids the 'It is not safe to rely on the system's timezone' warning.


@kgalli should fix asset pre-compilation in production.

@floriank @ulferts this might be dead-code (?) I have no idea if the details pane even uses this code.


I have always doing this ... But sometimes its difficult to track all people that contribute to patches. It will be fixed ... No problem.


Does it make sense to test all comm interfaces here?


ERB in JS is generally a horrible idea.


You should open a jira issue.


some specs here : http://www.mi.com/hongmi2/specs/
check network portion


I have no other variants :-D


@wkerzendorf, the recursion error appears to be because the code tries to set `self.outputs_dict` using the `_setattr_` function itself.


Hmm, what's the error message?


cool!


I can investigate if this is indeed dead code, but on first look it might be needed for the quotation in the WP legacy form (when quoting a comment).


Yes, with the current code without a 1-1 mapping between Rule/Search Type and Timer Type, changing channel, filter and search filter is very likely to create breakage, or at least recordings being recognized in 'unexpected' categories. Over the past few days attempting to get any channel 'working' each change I made broke something else :(.

Also, to get Any Chanel selection in the Helper75 case working fully for 'Record all', you would need to add 'handle.SetType(newrule.Type()) to change between an Any and a Channel record type.
A better solution might be to allow, search and type changes only in specific cases - but that would mean that UpdateRecordingRule would need to know the PVR_TIMER type, which it currently doesn't :(.

I still think ultimately this will be necessary to make the user interface make sense, but while the all, once, daily and weekly Timer Types have three modes (EPG/Search/Time+Channel) it seems very complex to try and find all the combinations. I would much rather change philosophy to use EPG info only to set up a new timer, then only modify rule parameters which are required to change things adjustable using the Timer Settings dialog.


The whole dispatch chain accepts Java or Ruby Strings as well as Ruby Symbols.


it's where kai.h and kai.lib are expected to be found. So, yes, you may give a path to your git repo there.


fix #224


apply media file --> load  media file


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.313](http://teamcity.monogame.net/viewLog.html?buildId=8503&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 501, ignored: 6 Build time: 00:26:11


great work man

Have Good Luck


@floriank right.. I thought that form was still mostly using Prototype.


apply --> load


I love your coding and implementation. I done almost everything which you have mention. Now i would like to render view from controller with a template engine. Could you please add support for twig or blade template engine inside the mvc implementation. This would be my humble  request to you. Please share your video link here for the same.


Maybe use [`@Ignore`](http://junit.sourceforge.net/javadoc/org/junit/Ignore.html) instead? :wink:


Yes, I saw it and it's in fact shorter and more clever than mine, but if you can't get unzip return non-zero in TWRP it's unfortunately useless. In my code (that's longer and uglier) I focused on selecting the nearest density available. I didn't have time to test it, but it should work.


Or maybe I should have explained more.
The field was missing here and that's why in the flavor module I had a dependency to an other module (xwiki-commons-extension-server-api), where "averagevote" was defined. But this module should not be bundled in XE.
See https://github.com/xwiki/xwiki-platform/commit/7c81d62a949fea1aa93955be016135a95ba5cabb to get the whole picture.


> I think drag and drop fails the 'could my mom do it?' test, there should definitely be some buttons.

Yes, but improvements over this part will come later, i'm not really happy with this either. Do your mom really use Picard ? :wink:


seems does not use red-100x100.png


Nah, there is some dark magic needed when re-initializing the Textarea AtWho Functionality.


this is the lib flags you should do it only on tests


Sorry to disturb you，could tell me why remove the trash command from plugins？Thx~


No semicolons is the way forward. You need to understand asi coding js anyways. Unneeded noise, the language is designed to be used without or with.

Sent from my Windows Phone
________________________________
From: Keith Woods<mailto:notifications@github.com>
Sent: 30/06/2015 17:00
To: JakeGinnivan/react-adaptive-trader<mailto:react-adaptive-trader@noreply.github.com>
Cc: Jake Ginnivan<mailto:jake@ginnivan.net>
Subject: Re: [react-adaptive-trader] No semi-colons. Just to annoy ray (ccb6b34)

Just no

—
Reply to this email directly or view it on GitHub<https://github.com/JakeGinnivan/react-adaptive-trader/commit/ccb6b3465d663a283a13ecb139347ed12015aaaa#commitcomment-11927004>.


Yes, well a possible solution for my would be, to instead of relying on the returncode, to get the output in a string, and then test whether this string would be empty or not (e.g. with test -z).

Btw the returncode issue is only a suspicion till now, I just see that it always enters the 'if' clausule as if the unzip statement would be true, but something else could be happening.

Does your code btw also account for the folders with multiple dpi specification (like Maps)?


@floriank oh my :worried:


@sarkiroka you have a bit more digging to do, because `new TypeError() instanceof Error` is indeed `true`.


In your example, you assign the results of the include statement to a variable; I don't see such assignment of the include of "parse_analyze.inc.php" in your pull request.


Thing is that I think/thought it would be better to have a directory containing only the target `.cmake` files (currently `cmake/target`). And another one to contain what is factorised (currently `cmake/toolchain`).
The names I picked are probably not good at all but I think this structure is good on the other hand.
And I find this structure to be quite similar to what was already done in the `mk` directory in the `modules-cmake` branch.


Surely this should go in /usr/share/pixmaps or something?


apply --> load


@since 7.1.1 is wrong


Same as above, comments before the function


No need for whitespace control here


You are welcome, and thanks!

Regards


Can be done with a single query:

```python
self.folder = attachment.folder
if self.folder.id != request.view_args['folder_id']:
    raise NotFound
```


Hi, I have solved my problem, because I did not installed cyrus-sasl-gssapi lib, so client got an error with sasl mechanism. Your lib works well with cyrus-sasl-gssapi :D


Oh, that reminds me that we need to fix matches that contain newlines.


Will follow later


why not `/event/<confId>/manage/attachments/<folder_id>/<attachment_id>/`?


Why don't you take an universal reference as parameter? As defaultBuilder ownership is given to DefaultElementLibrary it can be good.


Can I ask why the Options have been deleted? Does it work with Options as well?


Perhaps a rebase would have been more appropriate here. Let's do that next time.

https://www.atlassian.com/git/tutorials/merging-vs-rebasing/conceptual-overview


jkhjkh


why call it? this should be a property.

actually, it could use `@generated_data` so you can simply use `form.protection_mode.data` like you'd expect for real form fields


As for me, the use of the three places the language setting a bad option. Or we use language to all or use the default language. This is usually in other applications. In tvheadend we have three options to choose the language and you can get lost.


That may come from missing parenthesis, try with

($start and @name=$start) or (not($start) and parent::xs:schema)


If you were to give up trying to re-attach Rule timers to an EPG event after initial creation, and only use information available in the mythtv rule itself to drive edits to an existing timer, this problem would disappear.
As long as 'upcoming' recordings get attached to EPG entries, the icons will still appear in the EPG view.

Regarding using just selected channel and start time rather than ProgramID to find a matching EPG entry, I think this is a bad idea. If the user changes channel they would expect it to record instances of the same program (maybe at a different time), not a different program at the same time.

On a loosely related topic, it should be possible to create a mapping between Rules and expired EPG entries using the 'RecordID' parameter returned in the Recording section of the GetProgramDetails API method for EPG entries. These could be used to generate pseudo 'expired upcoming' records for EPG entries which could be added as 'completed' timers. Not sure what the performance of this would be, but it might be worth attempting once the basics are stabilized.


because there will be one for `/link` too


you can simply pass the attachment itself as a single positional argument here. that should give you everything you need to build the url


this is a property, not an instance variable - not calling it won't cause problems later

I plan to mark those properties (`self.tackExt`) as deprecated, the first step is not to call them if they are unset by user when `create()` is called


Although I'm not sure how useful it will be in the future, it'd be good to also update the comment at the top of this file which shows which context variables are expected...


I would use the same url/controller for both... You know the type from the attachment anyway.


Hi emanuele,
sorry for bothering, but I'm always a bit concerned with hardcoded numbers, I mean the 89 for the trigger type. Could you include the TriggerTypeDefs.h and make it trigger::TriggerTHT? I think that would be saver for the future.
Cheers, -H


should also be disabled by default


For opsworks I had error: `Could not satisfy version constraints for: mysql`, because opsworks still use old version.


Just raise an exception here. BadRequest i would say.


Just use `file = request.files['file']`. If more than one is uploaded you simply ignore it.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.314](http://teamcity.monogame.net/viewLog.html?buildId=8507&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


missing i18n


あれ、ほんとだ。
戻しておきます！


Commit bug !ATTENTION! le 'underscore' dans les noms de bundle ne marche pas.


we don't support removing a file from the edit dialog. dropping a new file on it should automatically replace the old one


There is a trick for `all`:

    template <bool...> struct Bools {};

    template <typename Predicate, typename ... Ts>
    using all = std::is_same<Bools<true, Predicate<Ts>::value...>, Bools<Predicate<Ts>::value...>, true>;


`existingFile`


Why add a "/get/"? HTTP method and REST standard implicitly defined this information


Unfortunately texinfo doesn't really do links. It puts the focus on printed manuals and cross references. So this would look like this in an info reader:

```
Magit is an interface to the version control system Git
(https://git-scm.com), implemented as an Emacs package.  Magit aspires
to be a complete Git porcelain.  While we cannot (yet) claim that Magit
wraps and improves upon each and every Git command, it is complete
enough to allow even experienced Git users to perform almost all of
their daily version control tasks directly from within Emacs.  While
many fine Git clients exist, only Magit and Git itself deserve to be
called porcelains.

...

   While one has to know the basic features of Emacs to be able to make
full use of Magit, acquiring just enough Emacs skills doesn’t take long
and is worth it, even for users who prefer other editors.  Vim users are
advised to give Evil (https://bitbucket.org/lyro/evil/wiki/Home), the
"Extensible VI Layer for Emacs", and Spacemacs
(https://github.com/syl20bnr/spacemacs), an "Emacs starter-kit focused
on Evil" a try.
```

I don't want the first paragraph to look like this, so I am dropping the first link.


Should probably be => `/v1/album/:page`
ref (https://github.com/miniwaveme/api/commit/b90f79249de64471d53fbf983a7ee05e17192807#commitcomment-11927798)


The regex should be like this?

```javascript
/\s*(\S+)\s*(?:,\s*(\S+))?\s+in\s+(.+)/
```


does not compile!


この一文はここではなくてリストの後に置くのが良いと思います．
その後の`… high 2-adicity).`の文の次に，`Note that the old parameters presented in \[BDMOHT\] are not used now.`と入れるのはいかがでしょうか？


	if(!mIsCardNameEditable) {
		cardName.setEnabled(false);
		... // false
	} else {
		cardName.setEnabled(true);
		... // true
	}

should be replaced by.

	if(mIsCardNameEditable) {
		... // true
	} else {
		... // false
	}
	cardName.setEnabled(mIsCardNameEditable);


It might be more clear just to use self.user here, since the ability to merge doesn't seem to affect this test?


dropping


There exists no official Gentoo ebuild for it. But even if it seems to be abandoned by the original author I want to use it, since it is easy to use and stable. [My fork](https://github.com/velnias75/libnotify-qt).


What about to use `self::DEFAULT_NAMESPACE => array()` instead of hardcoded `default` value?


As part of the migration, shouldn't release commits be tagged appropriately?


Probably not the most useful user type I could have picked. Does self.user have edit access?


This seems inside out. It's not really the "static site's data"— that implies that the static site is key, and this is derived from that. But the data is the core. *All* the various sites are then built using the data from it.


can you add some extra revision space for tex backports? like bump to 30 or so?


Merged both (3ce2c65 and 26024a5).


I've extracted that from from the `games-board/netmaumau` code and want to develop it further as independent project. Thus  there exists no official Gentoo ebuild (*yet*). [My repo](https://github.com/velnias75/QGitHubReleaseAPI).


Since the preceding sentence said that "there are" (as in "exist") buffers, it seems we should assume the existence here too.


I used toggleStar as a reference when building these functions, and it behaves the same way. In fact most of the message controller functions do. If you know how to do what you're proposing, I'd like to know how. AFAIK, there is no way to directly edit a message without referencing the mailbox folder.


可以考虑合并get和set方法。


@hasufell How will we handle the new dependencies?
See the *line notes* above for details.


This is fine (and great that you're testing multiple current elections) but it strikes me that if we want to do more generic testing in future it might be worthwhile to add a single decorator `@override_election_app` that will update all the ELECTION_APP-dependent settings. I'll create a ticket for that, though, I don't think it should hold up merging this PR.


I think that the new methods like setUsernameParam and setPasswordParam are bad design. The interface FormLoginHandler should not force the implementation to restrict the form fields to a combination of username and password. Btw, these methods are only used in test-code currently. But if it is the design intention than the interface should be called something like UserLoginHandler.


This will need to be expanded on greatly. This file is basically the machine-readable map to how everything hangs together — if you want to interact with the data in any way, you need to understand this file, and follow what it tells you, rather than relying on things like current directory structures, naming conventions, etc.


This is not used for any purpose other than in toDecimal(). This should be a local variable - not an instance level field. This is what I meant when I said " Be precise in the internal representation of your classes. An instance should only contain the minimum data that it needs to perform its operations and maintain its state. The selection of the internal data structure which could be an array or a map is also important as that would decide how efficient your methods are going to be."


Sweet! Thanks @andrewfb
I kind of hit a wall last time I tried to do this.

I just tried to load a few .dds and .ktx generated with cmft studio and I can't seem to get the mipmap chain working. Not sure if it's a format or loader problem though. I will try to investigate a bit more this weekend.


Most of the sources aren't "Open Data"


Status isn't a boolean, it is an integer. See `CommonParams::journalApplicationStatus()`.


That link doesn't work, but that's a good thing as it lets you see more clearly what happens for a bicameral Parliament :)


sorry


hello~


Then, it's wrong. libkai is distributed with its own directory structures,include and lib. You shoud use $libkaidir/include for headers, and $libkaidir/lib for libs.


I'm not sure if I understand your question correctly. You are asking why I implemented addition synchronisation to "protect" BlockingQueue ? If yes, my answer is because peek() isn't blocking.


I know ABP plans to support `https://subscribe.adblockplus.org/...` for subscription syntax, but I haven't stumbled upon a case yet. I will take it into account when the new syntax is actually in use.


Odroid-C1 initial


Well, what? Want me to change it to `"What the fuck?"` because we're against abbreviations?


isUpperCase is not a valid check. Look at the documentation for isUpperCase. There are characters outside of A-Z which will return true.
http://docs.oracle.com/javase/7/docs/api/java/lang/Character.html#isUpperCase(char)


Typo:"http://morphi.io", though the phrasing here is a little odd as well. Saying that something is scraped from morph is a little ambiguous or misleading — probably worth spelling out in more detail that we run scrapers *on* morph, and then download that data over their API, as this is a process we'd like to encourage other people to do too. (Though that's likely a pointer to a separate article on exactly that)


	if(!mIsCardNameEditable) {
		cardName.setEnabled(false);
		... // false
	} else {
		cardName.setEnabled(true);
		cardName.setHintTextColor(mHintTextColor);
		... // true
	}

should be replaced by.

	if(mIsCardNameEditable) {
		... // true
	} else {
		... // false
	}
	cardName.setEnabled(mIsCardNameEditable);
	cardName.setHintTextColor(mHintTextColor);


Caused regression fixed in https://github.com/maur1th/naxos-project/commit/e5428556350fbb40eab436dfa7e3db7cb434d748


Calculate sum? Why not in another function?


Avoid code duplication. When you are duplicating logic like this you will end up having to fix it everywhere if you need to change the validity criteria. Instead introduce one method that will tell you whether the character is valid or not. That way you would have to change in just one place when needed.


It's not so much that it's updated to include metadata about this change — it's that the map is updated to reflect the new reality.


Looks like you misspelled "canola"


cXZxzzx