+ String requestURI = request.getRequestURI();// 访问Servlet的URI
+ String method = request.getMethod();// 访问Servlet的方式Get或Post
+ String param = request.getParameter("param");// 客户端提交的参数param值


This was not intentional. I refactored using idea and I had not noticed this had been removed. I will reinstate.


Unit tests?


Unit tests


@karlnapf @lisitsyn @lambday @sonney2k @vigsterkr @besser82

I am not sure whether this is the right convention of updating news. Just for once, could you please double-check? Also, not sure if we should always use Soeren's name in the first line.


Unit Tests?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1159 of 1161 tests passed)

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


String requestURI = request.getRequestURI();// 访问Servlet的URI
String method = request.getMethod();// 访问Servlet的方式Get或Post
String param = request.getParameter("param");// 客户端提交的参数param值


IMO no need to add `should` just for that


I think you mean `_commentLine` here.


Looks good to me. Thanks. Ship PR once we pass CATs with new CC changes.


- [ ] The logic of getting scheduled before getting all the tags back is still not solved


Detail: you may want to be consistent about the quotes, `"` vs `'`.

In Ruby, you can do interpolation within double quotes only (`"#{a_variable} here"`). The Ruby engine has been designed so it has no noticeable impact on performance though, so it's just a matter of being consistent so it "looks" good.


Hi @sybond, so I can explain a bit: the poll implementations are now a bit more complicated and for some implementations need to be able to store additional state, not just the descriptor.  This code is out-of-date because I have not had a lot of call from users to be able to use WANProxy on Solaris systems with port(3C).  These changes are not complete enough to implement the needed behaviour, and just mask a compile issue -- the comment near the top referring to event_poll_kqueue.c is right, that's where you can look for a better starting point.

Probably it would be easiest (really quite easy) to simply copy event_poll_kqueue.c to event_poll_port.c and re-add the port(3C)-specific code.  It really is a lot like kqueue(2), but I don't know off-hand whether the synchronization cann all be relied upon in the same way, or whether you can trigger user events or need to provide something like event_poll_epoll.c uses.

Good luck!


HVACSizingSim-unit-test-deallocation-fixes (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (650 of 652 tests passed)

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


LGTM.


- [ ] unit test?


- [ ] Unit test?


HotFixUnitTests (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 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)


HotFixUnitTests (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Can't this be included as part of a `ChannelQuery` rather than a new method?


Thanks so much for taking this on!  I'd be interested in hearing (privately, if you like) about your interest in WANProxy on Solaris.  I may be able to provide useful suggestions, depending on your circumstances and needs.


HotFixUnitTests (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


HotFixUnitTests (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Hi, Shake0.

Thanks for giving me information about how to compile and build them. I have another question I want to ask you. I have tried to compile your MetaWrite-Master by using: g++ -I include -c src/*.cpp

and I got error like this:

In file included from include/MetaWriter.h:4:0,
                 from src/MetaWriter.cpp:1:
include/ExifTool.h:44:39: error: ‘NULL’ was not declared in this scope
include/ExifTool.h:44:62: error: ‘NULL’ was not declared in this scope
include/ExifTool.h:47:59: error: ‘NULL’ was not declared in this scope
include/ExifTool.h:49:60: error: ‘NULL’ was not declared in this scope
include/ExifTool.h:52:41: error: ‘NULL’ was not declared in this scope
include/ExifTool.h:52:65: error: ‘NULL’ was not declared in this scope
include/ExifTool.h:53:58: error: ‘NULL’ was not declared in this scope
include/ExifTool.h:53:78: error: ‘NULL’ was not declared in this scope
include/ExifTool.h:55:37: error: ‘NULL’ was not declared in this scope
include/ExifTool.h: In member function ‘char* ExifTool::GetOutput()’:
include/ExifTool.h:61:80: error: ‘NULL’ was not declared in this scope
include/ExifTool.h: In member function ‘char* ExifTool::GetError()’:
include/ExifTool.h:62:80: error: ‘NULL’ was not declared in this scope
src/MetaWriter.cpp: In constructor ‘MetaWriter::MetaWriter()’:
src/MetaWriter.cpp:18:26: error: the default argument for parameter 0 of ‘ExifTool::ExifTool(const char*, const char*)’ has not yet been parsed
src/MetaWriter.cpp:18:26: error: the default argument for parameter 1 of ‘ExifTool::ExifTool(const char*, const char*)’ has not yet been parsed
src/MetaWriter.cpp: In member function ‘bool MetaWriter::saveMeta(std::string)’:
src/MetaWriter.cpp:48:32: error: the default argument for parameter 1 of ‘int ExifTool::WriteInfo(const char*, const char*, TagInfo*)’ has not yet been parsed
src/MetaWriter.cpp:48:32: error: the default argument for parameter 2 of ‘int ExifTool::WriteInfo(const char*, const char*, TagInfo*)’ has not yet been parsed

Would you give me suggestion how to solve it? Thank you in advance.

Regards,
Intan


@zhang-hua does not seem you included/merged this commit in your latest? Please confirm. Thanks.


MUCH TO DO


@itstep2013autumnEvening/students

@AndrewVerbilo
@ZuiKost
@Minedy
@SpirkinIlya


I think you missed the thrust of [my comment](https://github.com/lsst/metaserv/pull/3#discussion_r26642654).  With the proper regexp, you don't need the if statement and can always return `result.group(1)`, which will be the empty string if appropriate.


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


Faker faker faker. Looks good, theres some whitespace hound is probably complaining about.


@zhang-hua same for this one. Please confirm that these translations are part of your latest commit. Thanks.


My bad, problem not here, the problem happened with this https://github.com/openfl/lime/commit/bb46ef9b5da474a8578230781fda715210f1b49c


>>тут проблема в другом, то что в таблице торрентов хранится только одна - последняя серия, соответственно если ТМ и утащил две подряд, то информации в БД о предпоследней просто нет
Ну я смотрел на эту проблему, как пользователь transmission.
У нас есть RSS, в котором есть 2 эпизода. в самой ленте они будут идти в обратном порядке:
s01e10
s01e09
Если идти с конца, то в transmission будет добавлен сначала s01e09, затем s01e10 и никаких проблем с тем, что в таблице будет храниться последний эпизод.
Вот как быть с RSS... Возвращаюсь к моему предложению хранить RSS ленту в xml файле. При добавлении новой раздачи просто дописывать в xml файл новый блок item с данными текущей раздачи. Ну и попутно очищать из RSS ленты записи, которые старше, например. 3 дней.


merge


What was the bug here?


Inverse of seems cool. Not sure if it's neccessary, but it's good to have for validation.


Did this pass for you on Windows? I am seeing that running on Windows, node receives a `EPERM` instead of eof as the last character, and the test prints `error` instead of `eof` here.


Ahhh... :relaxed: anytime you manage to write a fully lazy function and get to put ``Yields`` instead of ``Returns`` (ugh...) in the docstring I get a sense of satisfaction for a job well done :smile:


Again, you slightly missed the point of [the comment](https://github.com/lsst/metaserv/pull/3#discussion_r26643122).  That regexp is supposed to allow you to return result.group(1) if the pattern matches, without any of the looping or stripping below.


`pwd` is a command for shell.


fixes #7

This hopefully closes the pull request.  This and several prior commits reproduce the changes from the request.


Can you clean up the code to be shorter?


I was expecting you to simply test `_retrIdxColumns()` with a few `self.assertEqual()` lines, not test it by calling `parse()`.


Note that these `assert`s don't do what you really want -- they're the Python built-in assert exception-raiser.  You want to use the `TestCase` assert functions which, among other things, print out the correct and incorrect values when a failure occurs.


@KorvinSzanto then use `preg_replace('[/+]', '/', $badString)` which will do the tricky for 10 consequent forward slashes too..?


uncommenting the whole user spec seems silly, but whatever the merge wanted from you i guess.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1159 of 1161 tests passed)

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


68304232-1e)-Move-parent-object-sizing-calcs-to-a-common-routine (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (0 of 0 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)


Таразов Айнур Фанисович.
Лабораторная работа №2.
Создал формы для регистрации.
Использовал элементы html5.
Бох ввода e-mail сделан стандартными средствами.
Сделал ещё шаблоны ввода для текст типов.
...


68304232-1e)-Move-parent-object-sizing-calcs-to-a-common-routine (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 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)


82706250-AdaptiveConvectionNew (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (0 of 0 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)


I'd write that regex as `preg_replace('_/+_', '/', $badString)`...


Another example might be that a new product is actually labelled with a number at the end, for example "Firetruck 2.0 v1.2.3"


Thanks a lot. I really need to find a better diff tool.


I don't think this is necessary since you are using roles, but it won't hurt anything, so I will merge anyway.


correction


SQLSTATE[42S22]: Column not found: 1054 Unknown column 'product_sku' in 'field list'


what's the benefit?


82706250-AdaptiveConvectionNew (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 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)


实现了类似黑客帝国字符逐个显示的效果


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.45](http://teamcity.monogame.net/viewLog.html?buildId=5788&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 460, ignored: 6 Build time: 00:14:23


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


try to resolve travis error for this


We should definitely centralize. new parser is a bit weird, it's a later
rfc but is soooo forgiving

On Thursday, March 19, 2015, Robin Ward <notifications@github.com> wrote:

> We actually use URI.parse in many places (including onebox) - should all
> those be updated too?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/discourse/discourse/commit/948617cb0b2e67b3186e0525c652402aa60298fa#commitcomment-10273102>
> .
>


additionalSlinkyGHXWork (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (0 of 0 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)


Do you target areas for refactoring, or do you just fix things as you see them?


additionalSlinkyGHXWork (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 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)


Could you please explain in detail in the commit message what this bug is about ?


I think you forgot to wrap this in a
`#if !(NET20 || NET35 || PORTABLE || PORTABLE40)`


Well, I always write regular expressions as `delimiter`+`regular expression`+`delimiter`+`flags`


is still type is needed? If needed are you planing to use exiting  SAI_ROUTER_INTERFACE_ATTR_PORT_ID for lag also or planning to add new type SAI_ROUTER_INTERFACE_TYPE_LAG.


@maniac103 - what would "perl -v" give you?


No scaladoc? - you made me cry a little ;)


I just browse through them and noticed that they can be simplified.


Исправлено в 59620bb84cef4493dca3ca26e8fc267e822eb532


@max, great!


Opps, true. I'm creating a new state every time a change arrives, plainly wrong.


This filter is runnig ok. But will be better if we can use with formts like dd/mm/yyyy, mm/dd/yyy, etc..

In my case this filter obly works with date format yyyy-m-d (image). new Date parse this kind of format but not the others above.

Why not include moment.js in this project to support more formats ?

I think that we can use cellFilter (that is angular filter used to format the column date data) with moment.js ?

I was playing with it and I think that could be a good solution.

This is my code :

```
if (column.cellFilter) {
	term = moment(term.replace(/\\/g, ''), column.cellFilter.split(':')[1].toUpperCase()).toDate();
} else {
	term = new Date(term.replace(/\\/g, ''));
}
```


The variable `taskflag` is annoying. Please avoid using such global variable to implement simple state-specific operations.


The size of buffer is bit dangerous.


is isEmpty actually used?


TestWithoutLTO (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1655 of 1668 tests passed)

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


q""" val x = "$$" """


Could you create one commit per revision. I see r45-r51 which suggests this single commit is the aggregation of 6 revisions. It would help a lot if you include detailed commit messages as well.


Is it really necessary?


this doesn exist in portable. perhaps a custom delegate instead?


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.46](http://teamcity.monogame.net/viewLog.html?buildId=5790&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 460, ignored: 6 Build time: 00:15:56


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Don't you actually have to store the one time pad somewhere and check not only that you can unmask the token with the pad, but also that the pad sent is the last pad generated? Currently the pad varies with every request, but sending a token with one of the previous pads seems to work just as well as using the new pad. Am I missing something?


TestWithoutLTO (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Please avoid using too generic naming for functions like `_command`.


This breaks image.


http://en.wikipedia.org/wiki/One-time_pad#Key_distribution

*Because the pad, like all shared secrets, must be passed and kept secure, ...*


Why did you commit such code ?
You should resolve conflicted code.


@ElizarovEugene, только добавил функцию:
```php
	//проверяем авторизован пользователь или нет (если авторизация включена)
	public static function checkAuth()
	{
		if (session_id() == '')
			session_start();
		include_once "Database.class.php";
    	$auth = Database::getSetting('auth');

		if ($auth)
		{
			if (empty($_SESSION['TM']))
				return FALSE;

			if ( ! empty($_SESSION['TM']))
			{
				$hash_pass = Database::getSetting('password');
				if ($_SESSION['TM'] != $hash_pass)
					return FALSE;
				else
					return TRUE;
			}
		}
		if ( ! $auth)
			return TRUE;
	}
```


89224666-FixNANProblems-Issue4557 (rraustad) - x86_64-MacOS-10.9-clang: OK (1659 of 1664 tests passed)

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


I'm thinking like 183 should also check for the `-shells` option, so it can bail out if more than one image is supplied - it already does if any gradient import or export options are specified. That should avoid any issue like this - if there is no gradient information, the `-shells` option should fail so it can be detected in a script through the exit code. And we shouldn't allow these options with multiple input files anyway, it's would be too difficult to sanely parse the output in a script if we allowed it...


Is there a specific reason you're changing this? I did a lot of tweaking to these in the past to make it look good at different resolutions.


I guess /dev/null was meant here.


note that `which` is not available by default on all distributions. type git would be more portable.


개인적인 생각으로 캐쉬를 이용할 것이라면
public List<ApiMetaDataBo> getApiMetaData(String agentId, long time, int apiId)  메소드 형태를
public List<ApiMetaDataBo> getApiMetaData(ApiMetaDataBo apiMetaDataBo) 변경하는 것이 더 좋을 것으로 보입니다. (저의 말대로 한다면 ApiMetaDataBo의 hashCode와 toString은 포함이 필요할 것으로 보입니다.)

이전의 경우 static final String SPEL_KEY = "#agentId.toString() + '.' + #time.toString() + '.' + #apiId.toString()";
매우 적은 리소스지만 불필요한 키를 계속해서 생성하고 있어 보이기 때문입니다.

참고만 하셔요 ~
감사합니다.
구태진 드림


 :+1:
Yes, those options are only there to let the people know that we can change them without looking at the code.
I have added the possible options in the comments. And I plan to try them as soon as I have deployed a redis server


Where did these values come from? This is almost identical to http://tests.stockfishchess.org/tests/view/54da3bfd0ebc59381bbfb537


so much....


fake is the new black.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1159 of 1161 tests passed)

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


Won't this line flood the outpout if a few iterations are needed?


in
<pre>
remove src/check_version and replace with a extended version
of src/make_check
</pre>
in the commit message : src/make_version instead.


yeah - that inverse relationship validation looks cool.


ㅇㅇㅇ


nice work, DJ!


Are the new dependencies automatically added to the META stuff before release?
FWIW, I really thing supporting en/decryption is useful


Actually, I had something to do other than this and procrastinated spending time on browsing my code. So this change is not for any particular goal. :/


For clarity, perhaps a comment showing some example gcc output wouldn't hurt here?


sorry for the whitespaces


Backwards migration? Is that a thing? I've been writing everything to support forward version changes and backwards compatibility, but not changing the written version backwards.

> On Mar 18, 2015, at 6:36 PM, Neil Layne <notifications@github.com> wrote:
>
> Oh, and last I checked, backwards migration was still broken. :)
>
> \
> Reply to this email directly or view it on GitHub.
>


WHAT DOES TD STAND FOR!?!??!


Hello,

Why does this file have a `.rs` file extension?  It's JavaScript, right?

Thanks!


I know the feeling, I browse 8cc for the same reason.


Pretty sure an exact location lookup was something explicitly undesirable (private NF's, etc.).

Why not ditch the exact coords and just maintain distance? Keeps the mystery,  but solves the core question -- how far away.


Shouldn't there be an error message / warning when a .h file was not found anywhere? Right now, it completely halts further dependency processing (as it should, the alternative is an infinite loop, since the rest of the code always returns the first failed .h filename), but informing the user of that would be useful?


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


create --> creates


There was no point in making this "tiny refactoring" commit as there were no functional changes.

I'm fine with 'for (int i'-style declarations. Programming in C99 is much more fun. However, in the future you'll have to restrain yourself from pushing changes as random as found in this commit.


It's https://github.com/AdamBrodzinski/RedScript which is translated on fly to js
Here I register loader https://github.com/Paxa/postbird/blob/master/sugar/redscript-loader.js#L46


Thanks!
Why not call the env variable AGDA_BIN like in the Makefile?


I figure if we want to create drama based on factories being too close we need exact coords. Let's say the nearest factory is 4k blocks (28 of each aspect i believe), I would have to search a 4k radius circle for a nether factory that is likely hidden underground, so much for drama if i cant even find the damn thing right?


sure we can change it, this is just the only thing we have for now. We can change everything if its better as it goes for me. This is more of an initial proposal of how it could look like.


I think this as a matter of principle a good idea.
But we must have in mind that the local release branches have at the end a release version instead o a SNAPSHOT. When merging the branches after this wouldn't be the the SNAPSHOTS overwritten?


The main problem is that it's used in all RHEL kickstarts and our tests require EPEL. Anyway I was very surprised seeing internal address in upstream and yes, I'd like to see upstream version there. (and preferably as additional install url than using %post)


looks good!


status in effect!


My Bad ;)

On Wednesday, March 18, 2015, lejoe <notifications@github.com> wrote:

> doch: http://www.fontsquirrel.com/fonts/roboto
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/swissjeese/logo.js/commit/3cd57c90d91b8144dfa10a858e7fbba2bf35f0aa#commitcomment-10274089>
> .
>

--
http://nelm.io/pierre


admin index lookin good!


It's amazing how quick these patches are once you have a working product!!


Nevermind, my collegeaues explained me it's just to randomize this part of the response :)


+1  I really didn't expect that one to be fixed :-)


LGTM


Thank you for pointing this out. I have taken the test down to look at it more closely, and if it is a duplicate, I'll abandon it.


I think we should move towards CSS animations and avoid slide effects. They are indeed unreliable.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1159 of 1161 tests passed)

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


For the record: this is *not* what I argued about.


hello


i test it later


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

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


True... missing brackets... uggg :100:  points

Not really important, but, Zarafa developers always put the "else" in a new line:
```
}
else {
```


I pretty like your implementation and just want to know about your intention with the code above. If you want to prevent developers to accidently standard-construct `Card_info` it suggest to `delete` the standard constructor like follows
```c++
Card_info() = delete;
```


Yes, set the locale to de and have a look at it. It doesn't look good with the current settings.


Phew... good thing it is definitely cool!


nice catch!


No, it is not. It is just left over from my first idea to overload the AxisTags::contains member.


My dreams about multiple-apps with laravel was killed.
Really, What is it?
You make incredible arch of services and controllers! I was able to write app/Office and app/Api (and any other logical application part) with different services and controllers, just extend Application class to return different storage paths for each of them and have cool app. And what now? My different apps should have one compiled file and one services file for all of them, although they have completely different ServiceProvider sets.
Wtf?
In addition to all these paths are not configurable in any way!!!
In Vendor?
Really?

It was the worst thing what can happen with L5...


A thing of beauty!


this exits the the do-while loop, this doesn't look correct


OK, after discussion with @areis I (hopefully) finally understand the idea:

1. test developer uses absolute and relative paths to obtain params. For normal test params it's recommended to use relative path to allow easy portability/reusability
1. test-executioner uses combination of "-m" to compose his desired tree with multiple paths. Then __he__ specifies an entry point or multiple paths to search for relative paths, which effectively allows completely different trees to be used with the same test based on the entry point (or multiple paths). The resolution order (or simple entry point) can be either hardcoded, specified in config file or overriden on runner command line.

I'll modify this pull request to reflect this knowledge to demonstrate the usage and then slowly work on the final version with clash detection and multiple files support.

Thanks again, Ademar, and I'm sorry for the confusion


Hi, @syohex
The conflicted code has been solved.
Thanks!


Ah enfin merci ! :)


This kind of refactoring makes me recall of pruning Bonsai trees for aesthetics. Trimming excess bit by bit. It might not be productive, but it's fun.


Yes it is. You'll notice that batch restore doesn't have `actual_extended_cases`: https://github.com/dimagi/commcare-hq/blob/master/corehq/ex-submodules/casexml/apps/phone/caselogic.py#L461-461

Since I was expecting batch restore to replace this I didn't put any effort into improving this.


Init makes performs inserting groups into database, so it should be invoked inside transaction.


Where is the class `GenericRestResource`?


:sunglasses:


Please refactor this line:
* give the "magic" numbers 35 and 25 a semantic variable names.
* Please don't do all the work in one line.


~~problems~~ problem.


@SubJunk it is not an integer so user can use real numbers as well. The value between -2 and 2 is only my guess and user can experiment with other values. I don't have a 3D TV so I don't know how it looks.


Вот эта конструкция появляется из раза в раз:

Session session = getSessionFactory().openSession();
Transaction tx = null;
try {
tx = session.beginTransaction();
//Something useful...
tx.commit();
} catch (HibernateException he) {
if (tx != null) {
tx.rollback();
}
throw he;
} finally {
session.close();
}

Это скрытый копипаст. Совсем от него избавиться можно только используя аспекты/интерсепторы, сильно сократить его можно, используя паттерн Command:

public abstract class TransactedAction<T> implements Callable<T>
{

    @Override
    public T call() throws Exception
    {
        Session session = getSessionFactory().openSession();
        Transaction tx = null;
        T t = null;
        try
        {
            tx = session.beginTransaction();
            //Something useful...
            T t = act();
            tx.commit();
        }
        catch (HibernateException he)
        {
            if (tx != null)
            {
                tx.rollback();
            }
            throw he;
        }
        finally
        {
            session.close();
        }
        return t;
    }

    protected abstract T act();
}

После чего можно делать такие вещи:

(new TransactedAction<String>()
        {
            @Override
            protected String act()
            {
                //Something useful
                return string;
            }
        }).call();

Не намного лаконичнее, но в этом исполнении правильность всех конструкций контролируется компилятором, а не разработчиком, если когда-нибудь Вы захотите перейти на другой способ управления транзакциями, Вам нужно будет поправить только в одном месте, ну и т.д.


EL programa recibe la orden de Rekor para leer el RFID y regresa el SN por medio de USB


Nice pun.


А rss у вас у всех нормально грузится? У меня все поломалось примерно месяц назад :( Сначала перешел на https, но потом перестал парсится rss и все накрылось. Когда пытаюсь подгрузить вручную - получаю отлуп от curl'а: "curl: (35) error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure". Чтобы понять на какой стороне затык буду рад увидеть вывод `openssl version` и соответствующий запрос rss через curl с флагом -v с вашей машины. Заранее спасибо :)


This commit misses `git add executables.uroboro.hs`. I'll update the pull request.


@nobnago
Going back over this code now in light of #760... I don't think the lock is doing anything here as the method is synchronized. Did you add that for a reason, or can we just make `openCollection` `closeCollection` both synchronized methods and get rid of the lock? Any background on this commit that you remember would be appreciated.


Manda la orden al Arduino para leer el RFID y espera recibir "$" para guardar el número de serie del RFID, hay que quitar los DEBUG y remover comentarios de las funciones normales.


:D


> My dreams about multiple-apps with laravel was killed.
> Really, What is it?

Really? You mean to say you can extends `Application` class but can't extends some commands to use back the old structure if you wish to?


HVACSizingSim-unit-test-deallocation-fixes (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1159 of 1161 tests passed)

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


HotFixPurchasedAirUnitTest (mjwitte) - x86_64-MacOS-10.9-clang: OK (1661 of 1663 tests passed)

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


Bug étrange regarde si ca marche pour toi


@kodeine I would suggest that you revert the changes to the migration and create a new one where you alter the table schema. This should fix some problems for guys who are updating....if the migration is published after ```composer update```


Haha. Misschien in de main repo zetten: “tips, tricks en +1's zijn welkom” ;-P


![device-2015-03-19-124210](https://cloud.githubusercontent.com/assets/2087781/6726394/2bfdcfc2-ce49-11e4-9c51-9a1fd921c0a1.png)
when using latest snapshot (most likely including this commit) my game Label, TextButton, ImageTextButton, and anything that use BitmapFont, acting strange, the text is offseted like in the picture, I use scaled font


damn! :-)


Haha. Misschien in de main repo zetten: “tips, tricks en +1's zijn welkom” ;-P

Bedoelde daarmee eigenlijk alleen comments vanuit internen. E.g., nu wij, straks een stagiair en nog wat later medewerkers.


What I meant is that we should ensure that all the changes in the release branch also end-up in the main branch. If I understand correctly this is now handled by cherry-pick, but that can be error prone.

In the end the main branches need to use SNAPSHOTs, but we should still be able to do that. Basically after merging the release branch in, the version gets changed again to next SNAPSHOT.


thx


develop (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1159 of 1161 tests passed)

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


Ja to wziąłem stąd:
http://www.smarty.net/docsv2/en/language.modifier.string.format.tpl
Tam nie ma żadnej informacji, że to może działać różnie - zależnie od platformy etc.

@chilek próbowałeś robić podsumowanie na kalkulatorze i porównać z tym co wypluwa Ci LMS ?? U nas były różnice a po tej poprawce, którą wycofałeś wszystko się zgadza i nie musimy drukować paragonów każdego miesiąca, bo wystarczy ten raport.


@TAPL-93 You must be fun at parties


Se añadieron los avances del proyecto al diagrama de Gantt.


@m-stein The commit looks okay to me. Once you confirm I'll merge the changes to staging.


О такая же ошибка у Евгения, но он ее получает со сквида провайдера почему то.
SSLv2/3 никто в здравом уме уже не будет использовать из-за дырявости https://www.openssl.org/~bodo/ssl-poodle.pdf
У меня подозрение что Вы ходите именно этим способом, но не c TLS, потому как со стороны lostfilm(cloud-flare), у них только TLS https://www.ssllabs.com/ssltest/analyze.html?d=lostfilm.tv&s=104.28.16.79&latest
```
Sergeys-MacBook-Air:~ psv$ openssl version
OpenSSL 0.9.8zc 15 Oct 2014
Sergeys-MacBook-Air:~ psv$ curl https://www.lostfilm.tv/rssdd.xml -v
* Hostname was NOT found in DNS cache
*   Trying 104.28.17.79...
* Connected to www.lostfilm.tv (104.28.17.79) port 443 (#0)
* TLS 1.2 connection using TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA256
* Server certificate: sni48626.cloudflaressl.com
* Server certificate: COMODO ECC Domain Validation Secure Server CA 2
* Server certificate: COMODO ECC Certification Authority
* Server certificate: AddTrust External CA Root
> GET /rssdd.xml HTTP/1.1
> User-Agent: curl/7.37.1
> Host: www.lostfilm.tv
> Accept: */*
>
< HTTP/1.1 200 OK
* Server cloudflare-nginx is not blacklisted
< Server: cloudflare-nginx
< Date: Thu, 19 Mar 2015 08:09:14 GMT
< Content-Type: text/xml
< Transfer-Encoding: chunked
< Connection: keep-alive
< Set-Cookie: __cfduid=d5d93e9346466a01af5fc2ad85399e5c31426752553; expires=Fri, 18-Mar-16 08:09:13 GMT; path=/; domain=.lostfilm.tv; HttpOnly
< Last-Modified: Wed, 18 Mar 2015 19:10:08 GMT
< Vary: Accept-Encoding
< CF-RAY: 1c97b1a67db200dd-AMS
<
<?xml version="1.0" encoding="windows-1251" ?>

.... bla bla bla
```


@crynobone Sry, maybe I was very rough, but $this->laravel->storagePath() - deps on my extension of Application. I have working app in production now. And $this->basePath() in commands - force me to extend two commands, update Application, composer.json (or deployment script) to use new commands instead of default L commands.
I guess - it's to hard for minor update.


@kosmikus Turns out that GHC HEAD (and maybe v7.10) warns on some pattern matches with involved GADTs. Those (`extractd` and `extracti`) seem unreachable. This is not a pull request to revert changes!

Here is a mail to George Karachalias, that I sent him a few days ago.
```
Hi George,

Two of those warnings have been fixed with a recent commit:
  https://github.com/eelco/gdiff/commit/885d3a0fa0b1be8b0d95d12b4a5084a5e2da180f
Do your observations mean that this was unreachable code and that the
patch can be reverted? (I just tried 'git revert
885d3a0fa0b1be8b0d95d12b4a5084a5e2da180f' and indeed there are no
warnings from GHC HEAD). Considering that this was originally code
ported from an Agda code base, I would be surprised if the totality
checker would have missed those cases...

The `extractd` and `extracti` warnings do not appear in GHC HEAD, I
can confirm that too.

Overall it is nice to see the GADT warnings situation to improve :-)

I do not think you should cite gdiff, unless you want to mention it in
the "dead code added because the GHC false positive warnings tricked
us" category.

Cheers,

    Gabor

On 3/4/15, George Karachalias <george.karachalias@gmail.com> wrote:
> Hello everyone!
>
> Gabor, thank you very much for all the feedback :)
>
> |  I may suggest looking at the "gdiff" library, as it puts GADTs nicely
>> |  into scene. (I was not aware of you looking for libraries for
>> |  research, until now.)
>>
>
> I have compiled gdiff with 3 versions of GHC (like we did in the paper) and
> I have the following results:
>
>    - These are the warnings the original GHC (without patches, like
>    function `dataConCannotMatch') issues:
>
> src/Data/Generic/Diff.hs:490:1: Warning:
>     Pattern match(es) are non-exhaustive
>     In an equation for ‘extractd’:
>         Patterns not matched:
>             (NC _ _ _) _
>             (NN _) _
>
> src/Data/Generic/Diff.hs:494:1: Warning:
>     Pattern match(es) are non-exhaustive
>     In an equation for ‘sourceTail’: Patterns not matched: End
>
> src/Data/Generic/Diff.hs:502:1: Warning:
>     Pattern match(es) are non-exhaustive
>     In an equation for ‘extracti’:
>         Patterns not matched:
>             (CN _ _ _) _
>             (NN _) _
>
> src/Data/Generic/Diff.hs:506:1: Warning:
>     Pattern match(es) are non-exhaustive
>     In an equation for ‘targetTail’: Patterns not matched: End
>
>    - All of them are treated correctly (no warning issued at all) by
>    GHC with the patch (current HEAD).
>
>
>    - They are treated as they should by our algorithm as well.
```


TestWithoutLTO (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1651 of 1668 tests passed)

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


Done


@crynobone Also it's a stable release, so why I should rewrite core commands and foundations to get to work my app if L get updates from 5.0.17 to 5.0.18?


I'm agree with the previous post ...


It's a little trick to trigger the rendering of the cells. If you remove that animation, the cells are not visible. You cna read more [here](http://andreamazz.github.io/blog/2014/04/19/transitioning/)


you are absolutely right. It's not needed as you can identify the object type using the object id. Remove this SAI_ROUTER_INTERFACE_ATTR_TYPE, also rename SAI_ROUTER_INTERFACE_ATTR_PORT_ID to SAI_ROUTER_INTERFACE_ATTR_PORT_LAG_VLAN_ID. Not sure if it is a good in case we cloud associate other objects to a router interface.


Why do you need this?


@gaya- What is the purpose of this extension? I read this as: You want to get callbacks whenever you published a transform. But I do not understand the purpose.

Two remarks:
- Making the topic latched is not correct. TF2 comes with an extra latched topic over which static transforms shall be published.
- I think the logic of ```with-transforms-changed-callbacks``` is flawed: Imagine the callbacks have been enabled before calling it. Then your macro will disable the callbacks upon finishing. Is that the desired behavior?


yes it was


Hi Juan,

thank you for the suggested change. Two comments from my side:
1) You should not throw an exception, when there is no yUnit, just a warning, I would say. The files that the toolbox writes (in the end using bbciutil_recordSignals.m) does not specify yUnit (because this information is not provided by the bbci_acquire_* functions).
2) The 'unitOfClab' is a cell array of string, right? I am afraid that many plotting functions (such as plot_channel, grid_plot) expect that .yUnit is a string and will result in an error otherwise. How do you make use of the channelwise information in the .yUnit field, with your own functions?

Benjamin


Hi, by adding this parameter i would primary start a discussion about the blocking transmit function of the radio driver. Thinking of TI's sniff mode in combination with long wake-up preambles or low data rates i think the concept has to be reviewed.


Actually the check is wrong. Stupid logic of the seekable fields.


Hello, thanks for your work on SNMP, but I have some remarks :
 * the engine SNMP should be distributed with the ``canopsis.engines`` package, your file would be in : ``sources/python/snmp/canopsis/engines/snmp.py`` ;
 * make sure the file ``sources/python/snmp/canopsis/engines/__init__.py`` have the same content as ``sources/python/snmp/canopsis/__init__.py`` (this allows you to distribute files in a Python package already installed by another project) ;
 * you should then cancel your modifications on the ``engine-launcher`` script.


That should have been removed. Updated .gitignore to ignore *.swp files.


I ran a local test with 20% verification time and it was not better.
W 5873 - L 5907 - D 14415  [0.499] 26195


from collections import Counter
return [k for k,v in Counter(A) if v == 1][0]
This is the nice-looking solution, though not fast.


For this application, use dic.setdefault(0). Then you won't have to check existence.
Or, use collections.defaultdict, and dic=defaultdict(int).


*number


*means


Laravel never advertise/suggest/promise that multi-app installation would work. Nor it would consider that any changes it make going break these hacks. If you decide based on your best interest to go with this approach it something that you have to deal with.


Maybe we can add, that the vault recovery possibility **always** breaks or at least weakens the security of the application.


Isn't. the return statement. redundant ?


Okay. So this isn't a change and go conf  modification? We would need to modify the mobmod settings directly to stop drops?


Restricting the -shells option to single-file input kind of goes against your mantra of 'allow the user to do anything that has sane & well-defined behaviour', doesn't it? :-P
I'm actually all for that solution.


Can you also make that wiki page for logilink?


@hughsando: You should add `@:coreApi` to your Int64 and try to compile:

```
C:\GitHub\haxe\std/cpp/_std/haxe/Int64.hx:37: characters 31-64 : Field copy has different type than in core type
C:\GitHub\haxe\std/cpp/_std/haxe/Int64.hx:37: characters 31-64 : this : haxe._Int64.__Int64 -> haxe.Int64 should be this : haxe.__Int64 -> haxe.Int64
C:\GitHub\haxe\std/cpp/_std/haxe/Int64.hx:37: characters 31-64 : haxe._Int64.__Int64 should be haxe.__Int64
C:\GitHub\haxe\std/cpp/_std/haxe/Int64.hx:40: lines 40-42 : Field make has different type than in core type
C:\GitHub\haxe\std/cpp/_std/haxe/Int64.hx:40: lines 40-42 : high : haxe.Int32 -> low : haxe.Int32 -> haxe.Int64 should be high : Int -> low : Int -> haxe.Int64
C:\GitHub\haxe\std/cpp/_std/haxe/Int64.hx:40: lines 40-42 : haxe.Int32 should be Int
```


Emphasis on 'sane and well defined'...


missing space


No, that was me.


Uh, sure.


linux convention is --foo --bar (two '-')


And I'm pretty sure that was _your_ solution in the first place - adding the `-shells` option really should fit in with the existing framework...


grml. thanks.


hednfndnd


I don't understand this remark. This change wasn't about `extractd` and `extracti`.


__API Blueprint syntax error on line 1__: Expected "---", "HOST:" or empty line but "T" found. (Autogenerated message from apiary.io)


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 172](http://build.osmsharp.com:8080/viewLog.html?buildId=4644&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) is now running


heihei


What's this answer? This doesn't work?


there is such a block a few lines up already, please add the DEPS there.


嗯，那个应该更高级，直接用Baidu自己的api，用用户登录的cookie之类来传输的。有时间我也考虑用这个“完整”的api来实现。


NOTE: Обычно методы фабрики называются Create(), CreateXXX(). А вот в паттерне Builder - Build()


k. style disagreements, one block per logic is my guide. willco


This revision is clean.


Альбус Персиваль Вульфрик Брайан Дамблдор
Tarja Soile Susanna Turunen
Хорхе Франсиско Исидоро Луис Борхес Асеведо
Хм...


Laravel is amazing framework to build anything. With namespaced app arch in L5 You can write multi-app layout in 5 minutes. When minor changes broke it - it's unacceptable.
This story look like update of Filesystem class in L4 which provide fatal errors...
It's undermines trust in stability of framework...
What if tomorrow Taylor decide to store views in /vendor/views coz it's "has mach more sense"? - it's unimaginable example, but...


done. gotta run lecture now, so don't expect a response for 2-3h.


1. Minification.
2. Consistency across methods.


What do you think of providing the Getters in the default driver already, so if nothing was overridden, the default caching-approach is used, in other cases a custom variant __could__ be used ?

This would be some kind of "best of both worlds" approach.


Hi fess89,
Please can you tell me how to add a drawable shape, for example the circle on the area??
Thank you


坎坎坷坷


You referenced the wrong guy :)


OPENGEX_IMPORTER? (N vs M)


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 4.3.0.172](http://build.osmsharp.com:8080/viewLog.html?buildId=4644&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) outcome was **SUCCESS**
Summary: Tests passed: 469 Build time: 0:0:0


NOTE: Тогда лучше бы в конструкторе и создал :-)


What is the reason you don't use `return server.start()` here?


Очень хотелось бы увидеть log4j+slf4j или хотя бы java.util.logging


Where is the related change to documentation? It really breaks after upgrading from 1.5.1 now.


Why do we need this?


use symlink() here -- it's added to the environment in build_environment.py, and it comes from llnl.util.filesystem -- clearly I should document this better.


Fixed in https://github.com/HaxeFoundation/haxe/commit/9eb1e35af1042dfec841191147b415b38437f2fe


Sorry, github automatically converted a comment @Flavio as a link to you...


whit


Correct. A new issue would have been a better place. This commit is
totally (sic) fine.

On 3/19/15, Andres Löh <notifications@github.com> wrote:
> I don't understand this remark. This change wasn't about `extractd` and
> `extracti`.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/eelco/gdiff/commit/885d3a0fa0b1be8b0d95d12b4a5084a5e2da180f#commitcomment-10280750


Why not to use loop here?

    for (int i = 1; i < 11; ++i)
        Assert.False(array.All(x => x > i), string.Format("Failure on i == {0}", i));


I should add a copy() command for this type of thing.  Would symlink() not suffice though?


I'm quite sure we have a `broadcast` function


should use symlink -- i'm trying to avoid relying on too many external commands


The "NULL not declared" is very surprising, I have been cheap and I had the flag `-DNULL=0` which is the same of writing "#define NULL 0" in the source code. I just try it now by running `make` and it compiled on ArchLinux.  A more CS explanation is found here: http://stackoverflow.com/questions/462165/error-null-was-not-declared-in-this-scope
So .. they should include the cstddef header where NULL is defined.

I believe that fix everything cause "the default argument for parameter 0 of ‘ExifTool::ExifTool(const char, const char)’ has not yet been parsed" is due to all the arguments that have a default value of NULL.

Assuming you are in the root folder of MetaWriter project, run `make` which is actually executing these lines in the Makefile:
* `g++ -I include/ -DNULL=0 src/* ExifTester.cpp -o read`
* `g++ -I include/ -DNULL=0 src/* example1.cpp -o write`

you shouldn't have any compile error (but possibly you can get a runtime error if you didn't install ExifTool !).
Good luck! :smile:


This would serve `n_min` from the beginning, not the end of stream data, right? (Essentially, `head`, not `tail`).

If yes, let me rework it to do `tail`.


AFAIK expected value should be passed to `Assert.Equal` as first argument


Because /sudo console /<cmd> threw an error for the following: console.chat(/<cmd>)
and I cleaned up the code a little. I think it's much easier to read now


Ок, ваш ответ немного прояснил ситуацию :)
```
root@multimedia:/ # openssl version
OpenSSL 0.9.8y 5 Feb 2013
root@multimedia:/ # curl -v https://www.lostfilm.tv
* Rebuilt URL to: https://www.lostfilm.tv/
*   Trying 104.28.17.79...
* Connected to www.lostfilm.tv (104.28.17.79) port 443 (#0)
* successfully set certificate verify locations:
*   CAfile: /usr/local/share/certs/ca-root-nss.crt
  CApath: none
* TLSv1.0, TLS handshake, Client hello (1):
* error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure
* Closing connection 0
curl: (35) error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure
```
https://www.ssllabs.com/ssltest/viewClient.html?name=OpenSSL&version=0.9.8y

В общем, побежал обновлять OpenSSL, а вместе с ним и Nas4Free.


Oh yeah right. I'll fix that later.


dependent: destroy


Мне как раз удалось воспроизвести проблему, и я оказался прав:
```
Sergeys-MacBook-Air:~ psv$ openssl s_client -ssl3 -connect lostfilm.tv:443
CONNECTED(00000003)
39205:error:14094410:SSL routines:SSL3_READ_BYTES:sslv3 alert handshake failure:/SourceCache/OpenSSL098/OpenSSL098-52.10.1/src/ssl/s3_pkt.c:1143:SSL alert number 40
39205:error:1409E0E5:SSL routines:SSL3_WRITE_BYTES:ssl handshake failure:/SourceCache/OpenSSL098/OpenSSL098-52.10.1/src/ssl/s3_pkt.c:564:

Sergeys-MacBook-Air:~ psv$ openssl s_client -tls1 -connect lostfilm.tv:443
CONNECTED(00000003)
depth=1 /C=BE/O=GlobalSign nv-sa/CN=GlobalSign Organization Validation CA - G2
verify error:num=20:unable to get local issuer certificate
verify return:0
... bla bla bla, all is ok
```


Esta funcionalidad ha sido probada usando un curso dentro de una sesión?


required: true
http://api.rubyonrails.org/classes/ActiveRecord/Associations/ClassMethods.html#method-i-belongs_to


I'm on my way home, I'll refuse the edits when I am home.For testing, I actually fork this repo, and add a gh-pages branch. (for non-user repository, getting a github page to work needs the gh-pages branch.)Then I accessDoplgang.github.io/(repo name)Merge any new stuff to the branch, and it will show up in the github page.When I am done testing, I would remember deleting the remote branch so that the page disappears. It is best done so that the same website does not appear in two different domains with different versions and contents, especially when Google robot may crawl the local copy.


Usar api_get_path() para obtener la URL.


По аналогии и дальше


dependent: destroy и по аналогии дальше


Ciao Simone,
A me sono rimasti dei fle relativi alle API offerte, tipo Get.php, Post.php, Get.php .... , solo che sul repo non li vedo più, sono stati per caso cancellati.


It's been such a long time now...
I just remember we had some really severe problems with deadlocks when closing and quickly reopening AnkiDroid


User el ícono de survey y no de copy.gif


TestWithoutLTO (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1159 of 1161 tests passed)

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


good


xamarin doesn't like "and"?


Так как ты обращаешься к методу `hasNext()` в новом итераторе, то дополнительная проверка `i < n` здесь не нужна - она уже делается в `hasNext()`. На корректность в целом это не влияет, конечно.


Имеется в виду, что внешнему коду незачем знать посредством этого свойства, что мы храним наши коэффициенты в массиве даблов? То есть открыты детали реализации.


why not use the sai_next_hop_list_t structure itself.


Это поле проставляется автоматически после создания, его объявлять не нужно


Plutôt: var url = "{% url 'liste_subcategories' %}" + "?cate_id=" + $('select#id_cate_livre').val();


zgadza się;) Category -> User Many to One a później
```
{{ Category.createdBy.name }}
```
:+1:


Проблема с сортировкой скорее всего тут. Ты не сохраняешь тред после присвоения значения полю bump_time


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Sounds great. That way I could also review the created page. You might add a link to the comparison page to the main page.


You can speed it up a bit by using `-XX:+TieredCompilation` or even `-XX:+TieredCompilation -XX:TieredStopAtLevel=1`.

    java -XX:+TieredCompilation -XX:TieredStopAtLevel=1 -jar target/uberjar/perfect-timer-*-standalone.jar

See also https://github.com/technomancy/leiningen/wiki/Faster#tiered-compilation


great


OK, I will change that.


Looks good


You are right, I just wrote it quickly. I will change it.


Got the new file but the error is still there.
Pastebin of the error: http://pastebin.com/wpDzFR7P


Line 183 was needed because there was no sane way to provide output paths for gradient table text files if there was more than one input file. `-shells` is just writing to std::cout, no reason why it couldn't theoretically be used with multiple input files, and in a script just exploit the fact that the shells are sorted in ascending b-value to separate the individual outputs. Of course looping over images and calling mrinfo for each would make a hell of a lot more sense; I just thought you were against placing unnecessary restrictions on capabilities.


`params[:board].permit(:name, :pages_limit, :bumplimit, :description, :terms)` лучше вынести в приватный метод контроллера. Вот пример http://edgeapi.rubyonrails.org/classes/ActionController/StrongParameters.html Это позволит не поторять эту строку


No, I didn't :) It is generally more readable if forward references are not used. Kind of a typical C or C++ thing.
https://en.wikipedia.org/wiki/Forward_declaration

> The ```and``` keyword can be used in some cases to allow mutual recursion, but is discouraged.

http://fsharpforfunandprofit.com/posts/recipe-part3/


Разделяй экшны пустой строкой


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1159 of 1161 tests passed)

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


Sure. I could not come up with how to make it do tail without knowing at
which entry the stream drains and starts to wait for the upcoming entries.


Это нужно вынести в before_action


What about StringUtils.substringBeforeLast(templateName, FILE_SEPARATOR)?

See http://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html#substringBeforeLast(java.lang.String,%20java.lang.String)


This seems to test Min, not Average.


It is deprecated in rails 4.2 :)


Non capisco cosa intendi sono rimasti, a sqeuito di quale operazione? probabilmente erano file che tu avevi modificato localmente. Per ora lascia tutto quello che ha mandato Simone continuiamo a lavorare sulla struttura che ha organizzato lui che abbiamo concordato assieme non modificarla.


Yes, that's the tricky part. The original design was not supporting this usecase, in fact :) Hence I need to sleep over the idea, as it makes too much sense now.


Il y a plusieurs façons de sérialiser des instances de modèles en JSON. Pour ma part, je ferais:
`json.dumps(list(subcategories.values('pk', 'nom_sous_cate')))`
Si tu utilises serializers, ça irait aussi, mais ça donne une structure un peu plus compliquée à gérer du côté JavaScript, avec des champs dont tu n'as pas besoin.


Your error message shows the **old** version of the file.

```
(83, 'LIBRARY_MEDIA_DELETEOLDVER_CHECKB','Unchecked','dropdown','Default the checkbox for Deleting Old Version of media when a new file is being uploaded to the library.','Checked|Unchecked','defaults',1,'Default for "Delete old version of Media" checkbox. Showen when Editing Library Media.','dropdown','',50,'Unchecked',1)
```


Для подобных вещей обчно используют scope в модели. После этого это будет выглядеть примерно как `@posts = @thr.simple_posts` и `@posts = @thr.op_posts`


How about injecting entropy with timestamp while open session and close session is invoked?  This is also un-predictable?


I agree doing md5 hash everytime is not ideal solution but I don't think there is a better one .


I did leave those clocks here because those are "pure"  8226 clocks, not bound to any specific Sony device.


Why is there no colon `'http://{0}:{1}'.format(...)`? It seems that user needs to set `PORT = ':80'` to make it works correctly.

Another way is to use [urlparse.urlunparse](https://docs.python.org/2/library/urlparse.html#urlparse.urlunparse)


この意味は？


ファイル名とクラス名が不一致


+1

On 19 March 2015 at 10:05, Sheng Yu Chiu <notifications@github.com> wrote:

> How about injecting entropy with timestamp while open session and close
> session is invoked? This is also un-predictable?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/m943040028/optee_os/commit/b85cae75153b5bcea598a78dadef2eaedd88132e#commitcomment-10281113>
> .
>


С этим экшном надо что-то придумать. Может быть ситуция, когда сохранится тред, но не сохранится пост. Таким образом, нужен метод в модели, который бы сохранял их одновременно. При ошибке любого не сохранится ни один. См. метод ActiveRecord::Base.transaction


Good


Все логические расчеты лучше вынести в методы модели


In my situation, I'm unable to run the application under the same user as Apache and this is a real headache of a change that doesn't make a ton of sense considering that those files are gitignored in the first place.


Great.


Тестов можно было больше написать. Особенно на всякие многоблочные ситуации с известными клетками.
Например, известная клеточка в начале и блок (должны остаться клетки в конце). Также многоблоков, когда они не заполняют всю линию.


править старые миграции - плохая идея


Could you gist me the log of the warning without both changes?


Нет все гораздо хуже, поскольку код-клиент может создать новую ссылку на Ваш внутренний(!) массив и изменить его умышленно или нет.


I have done as you said and it works. Thank you! There is something I want
to ask, when i tried to run ExifTester.cpp using command line in terminal
like this:
intannf@intanlaptop:~/MetaWriter-master$ ./ExifTester
i got error:
bash: ./ExifTester: No such file or directory

even when i tried to run example1.cpp using command line in terminal like
this:
intannf@intanlaptop:~/MetaWriter-master$ ./example1 IMG_3511.JPG
i got error:
bash: ./example1: No such file or directory

But when i tried to run example of cpp_exiftool, they worked successfully.
Do you have any idea why i get those error?

2015-03-19 15:49 GMT+07:00 Rodrigo Coniglione <notifications@github.com>:

> The "NULL not declared" is very surprising, I have been cheap and I had
> the flag -DNULL=0 which is the same of writing "#define NULL 0" in the
> source code. I just try it now by running make and it compiled on
> ArchLinux. A more CS explanation is found here:
> http://stackoverflow.com/questions/462165/error-null-was-not-declared-in-this-scope
> So .. they should include the cstddef header where NULL is defined.
>
> I believe that fix everything cause "the default argument for parameter 0
> of ‘ExifTool::ExifTool(const char, const char)’ has not yet been parsed" is
> due to all the arguments that have a default value of NULL.
>
> Assuming you are in the root folder of MetaWriter project, run make which
> is actually executing these lines in the Makefile:
>
>    - g++ -I include/ -DNULL=0 src/* ExifTester.cpp -o read
>    - g++ -I include/ -DNULL=0 src/* example1.cpp -o write
>
> you shouldn't have any compile error (but possibly you can get a runtime
> error if you didn't install ExifTool !).
> Good luck! [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/UAV-Concordia/MetaWriter/commit/d0866c4900af19bf29514b92629cc6045da94ff5#commitcomment-10280869>
> .
>


__apiary.manifest contains reference to non-existent file in repository.__ (Autogenerated message from apiary.io)
+ docs/contrib/recaptch.apib


ok, I guess it makes the eval method easier to read.


                                                                                I am actually considering editing the templates so that the users can easily find the pages they look for, in case our project site grows, so that it can be much more user friendly. This, if brought to realty, would reside in a separate pr.                                                                                From: L-HenkeSent: Thursday, 19 March, 2015 4:58 PMTo: SecrecySupportTeam/secrecysupportteam.github.ioReply To: SecrecySupportTeam/secrecysupportteam.github.ioCc: DoplgangrSubject: Re: [secrecysupportteam.github.io] add comparison page (4a9bea4)Sounds great. That way I could also review the created page. You might add a link to the comparison page to the main page.

—Reply to this email directly or view it on GitHub.


Right, it is entirely possible at this line that mItemEntries.size() is 0, which would lead to index -1 being requested. Thank you, will fix.


Merging pull request #1


In model header files, this method generate the following declaration:

when property is primitive type, output `NSNumber*`
when property is model type, output `SWGCategory*`
when property is array of primitive type, output `NSArray*`
when property is array of model type, output `NSArray<SWGTag>*`

The above declarations used by JSONModel


Add `JSONModel` to Podfile


@jmazzitelli from what I understand, static imports should follow the same order as usual imports, right? By the way, should imports categories be separated by blank lines? I'm asking because @lucasponce maintains an IJ settings file and this might be of interest.


Вероятно предполагается, что кроссворд будет скопирован.
На самом же деле создается новый кроссворд с ссылками на исходные линии.


A quanto pare, non li avevo nella cartella corretta, non ho idea per quale motivo. Poco male, ora mi fixo.


Ciao Marco,
confermo la cancellazione di tali files.

Il 19/03/2015 09:54, Marco ha scritto:
>
> Ciao Simone,
> A me sono rimasti dei fle relativi alle API offerte, tipo Get.php,
> Post.php, Get.php .... , solo che sul repo non li vedo più, sono stati
> per caso cancellati.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coder-molok/foowd_alpha2/commit/6a4e90197c5273c5cbe5673cc26a8f3da2c332f9#commitcomment-10280948>.
>


Hey @daigo75 , your solution should be changed , this is because the recursion starts within the
  $this->get_cart_from_session();
thus it will never reach the $this->cart_loaded_from_session = true;
this is also the reason why    _doing_it_wrong( __FUNCTION__, __( 'Get cart should not be called before the wp_loaded action.', 'woocommerce' ), '2.3' );
Is actually never being printed.

public function get_cart() {
  if ( ! did_action( 'wp_loaded' ) && !$this->cart_loaded_from_session ) {
    $this->cart_loaded_from_session = true;
    $this->get_cart_from_session();
    _doing_it_wrong( __FUNCTION__, __( 'Get cart should not be called before the wp_loaded action.', 'woocommerce' ), '2.3' );
  }
  return array_filter( (array) $this->cart_contents );
}

and the rest of the code needed for this to work , at the start of the Cart Class

  /** @var boolean */
    public  $cart_loaded_from_session;

and on the __construct()

public function __construct() {
        $this->cart_loaded_from_session =false;

These changes solved the recursion  issue for me.


update base class of `SWGObject` to `JSONModel`, and remove all method declarations here,
 because JSONModel has method `initWithDicionary` and `toDictionary`, do the same work and better!


~~this~~


Написано верно, но похоже нигде не используется.


Wow... Thanks for reporting this. I don't know why it even worked in the first place then.


add `keyMapper` method to support `JSONModel key mapping`,
map json key to property name

ref: https://github.com/icanzilb/JSONModel#key-mapping


Vlan is not part of your Unified object id list. So we can not find it in SAI is id is vlan or port/lag. So type might be needed for this case to identify vlan.


Длинный тернарный оператор лучше оформлять так:
`condition
    ? a
    :  b;`


I changed enum here to state factory, because checkstyle was showing cyclomatic complexity in switch statement below. I could fix this also by making method, but this problem would come back in the future if more actions would be added.


peut-être lui donner un nom plus générique ?


the `-o` flag tells the compile the name of the final executable (default in linux is "a.out").

So briefly you start with .cpp and .h files, the compiler call the preprocessor (which basically 90% do copy-pasting). Then it compiles it into .o files and finally links them together by making the well known .exe in windows.

Linux don't really use file extension for mimetypes, so you can call it whatever you want it.
Then in my case the two executable are called 'read' and 'write'


I would suggest to make a proper JavaDoc comment if you change it anyway.


why just remove this test case?
can not put it on @ignore and file a jira? (that seems better than removing)?
Or is there a good reason for removingi t


because `JSONModel` can convert `date` and `datetime` string to `NSDate`,
and convert `NSDate` to string automaticly,
there no need `SWGDate` class.


Длинная строка. Лучше записать тернарный оператор в 3 строки (условие, ? вариант 1, : вариант 2;


88239574_sql_annualTotalCost (asparke2) - i386-Windows-7-VisualStudio-12: OK (2176 of 2195 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.13%25-yellow.svg)


Ok, so Let can be understood like "Let name refer to (eval e1) in eval e2" and Where is like "eval e1 where name refers to (eval e2)" ? I am a little confused. When do you need Where ?


Indeed Thanks a lot ;D


Currently we don't even need boost, but I'm not sure which libraries we'll use later.


30414/Kari Z. Tamas Zsolt/Sentinel/main.cpp

```p->nr++``` <-- shorter


this condition, p is array of primitive, then return type of p, e.g. `NSArray*`


I was just digging around for the docs on using Browser and discovered it's gone. Any insight you can share on why?


Good


Классика копирования :-)


What else would you write?


this is a test


fixed


The advantage of my current implementation is that it doesn't need to allocate the memory twice nor to copy the memory contents around (from the cstring to the other). It also never re-allocates, which is good.

JSON is a good idea.


@Robinlovelace you've removed working code and replaced it with something that is broken!  I  don't really understand your logic so maybe I'm being stupid but the slider works (and you can put the max range to 1000) and this gives:
`ERROR: missing value where TRUE/FALSE needed` when I change the input if it is empty (like when I select and delete).


thie condition, p is array of model, then return type of p combine type of inner, e.g. `NSArray<SWGTag>`


why not


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

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


Good catch! I'll send another PR if you don't mind.


界面说明

1.导航菜单
![efdcda5f28e3a7d62a9a182d7331e239](https://cloud.githubusercontent.com/assets/3400907/6727417/bfa4c614-ce5c-11e4-8093-686326eb3942.png)


Как в IMDB


@nikolai-b Looking into it - worked last night


The message is correct but it is not a proper JavaDoc comment: f.e. parameter ProzessID is not described:

```
/**
 * Remove lock for process
 *
 * @param ProzessID Id of process to unlock
*/
```


В метод


как в IMDB


hey. I started  a ROM project, and remembered about this hack you used. But i see you didnt update anymore hehe. I'll check the code to see if i can do something :) Thanks for this share!


private ArrayList<PartSale> partsaleList;


@nikolai-b cannot reproduce


Thanks dude -- I added that line just recently, so this was my bad. :-)


@cec If we switch got img-thumbnail, we would have to bump the dependency of rails_admin to 0.6.7
Since this is a minor difference. What do you think about using this as a solution?


вернее использовать crosswordCopy


TeamCity OsmSharp :: GitHub - OsmSharp [Build 996](http://build.osmsharp.com:8080/viewLog.html?buildId=4645&buildTypeId=bt16) is now running


After thinking about it, I find ```Let```much less appealing then ```Where```, seen from a notational point of view.

Use case for ```Let```:

```i = 0, f(x) = \sum_i^x```

Use case for ```Where```:

```f(x) = \sum_i^x, i = 0```

Using ```Where``` you can forward-reference a variable, which is nice in math, I guess.

> On 19 Mar 2015, at 10:22, Johan von Tangen Sivertsen <notifications@github.com> wrote:
>
> Ok, so Let can be understood like "Let name refer to (eval e1) in eval e2" and Where is like "eval e1 where name refers to (eval e2)" ? I am a little confused. When do you need Where ?
>
> —
> Reply to this email directly or view it on GitHub.
>


The logic is much better in this commit but please refer to issue #207 for how I believe it could be improved on.

In the future if we had a settings page for IOMAD this would be a good place to put some configurable settings in.


This is what fixes the missing copy_file method for boost filesystem. Removing the flag breaks compilation on current version of Linux boostlib.


That looks good Doug.  We do have jshint and jscs rules defined, that run over all of the core skulpt files.  We have yet to automate running them on all of the libraries.

Do you want me to accept this now, or would you rather continue to add to this PR with additional additions?


removed until it works perfectly on lollipoop


ButtonIcon-help => ButtonIconInactive or similar - as it the same as button icon accept no rollover/inactive. The name gives better indication about what it's about and increases reusability - as can be used by anyone wanting this rather than only on help views.


fix #176


:smiley:


Yes, that might be indeed the case... . Proper fix would probably require much more delicate work with the TCP connections, so this is really just working around the problem.


Is ``initialized`` in line 55 not always be null? Or could it happened that first ``config`` is null and until ``synchronized`` block is reached that ``config`` change to a non-null value?


OK I can - but you just enter a new number to fix. Easy!


![Uploading EB02C3039C3D070B20ADDA89AE51BB8A.jpg . . .]()


Ok, agora testes 100% :+1:


@tommysqueak `>= 1.7.0`  has an issue on Windows with JRuby https://github.com/rest-client/rest-client/issues/302


Thanks! Sure, go ahead and accept this one, and we'll make more PRs.

Question: do you want minimal changes, or could we, say, rearrange the code to alphabetize the function names?

BTW, what part of the world are you in that you are awake at this time of day? :)


removed not work


Internal opimizations that the JVM is allowed to do may cause these things to happen. To quote from [the link](http://antrix.net/posts/2012/java-lazy-initialization/): “This is how our intuition fails us and this is one of the reasons why folks keep saying that *multi-threaded programming is hard!*”


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.996](http://build.osmsharp.com:8080/viewLog.html?buildId=4645&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 519 Build time: 0:0:0


We should be able to handle both in the parser, but I think both can be parsed to Let.


Where are the tests?


What is the purpose of this commit?


I am a little bit uncertain if I should do this or not?
By default the ffms2 dll is installed into bin, which seems wrong to me.


Did it fix the Error?


Most important is the last edit, on the "Extending divided powers" section. I don't think that proof is correct :/


Hi !

i see the pb.

for me, it will be better to add this protection in the  _setPageHeader and _setPageFooter methods, by :
- setting the default font before $this->_makeHTMLcode();
- restoring the default font after $this->_makeHTMLcode();

because if we put this protection in the _tag_open_WRITE method, the test will always be executed for each writed text => lower performance.

What do you think ?


ok for me, i merge


Reach beyond the pull


83782874_LBNL_ThermostatOffsetFault (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1668 of 1670 tests passed)

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


If it did not fix the error then why commit it?


Hello @christocracy I have seen that on your plugin page, and was discussing about it with my team. Mind if I get to you via email? (eilin.chang@gophr.it)


It looks quite ugly. Can you use any effective magic number?


Shouldn't there be some comment?

This is very opaque. Especially given that most of the options go into the first `options` parameter, and only that one stands apart for whatever reason.


Why not make this the default scope? In fact, isn't line 43 making card_classes sorted by cost anyway - so why is this any different to card_classes at all?


Added it, and (the period)[http://www-inf.int-evry.fr/cours/java/JAVADOC/writingdoccomments.html#descriptions] was missing as well…


Para quem estiver vendo este commit, esse `while` com `cut` para definir variáveis foi otimizado usando a variável `$IFS` e o próprio `read`. Veja os commits 8d1476c65e13b09c97abf6e31bcc3d70ea4b653e e 87db37268e858798cd9d2324cf92f2f646074b12


I thought it wouldn't hurt :=)


Thanks, will do that now, but 2 comments:

1) Looks like failing on Travis too.  This suggests that the process I'm
using is wrong?
Namely:
https://secure.mysociety.org/intranet/ProductionSites/FixMyStreet/AddAPerlModule
(I blew away local/ on my dev vm and re-ran but gave same result)

I guess the manual munging will fix Travis fail too (as it won't pull in a
line about Catalyst::Test), but:

2) The manual adding of stuff in cpanfile.snapshot is problematic, and is
going to get ever more so (e.g. when a module gets moved across trees etc.)
The faff of adding a new module is a discouragement against doing so, which
I don't think is helpful.

The process is also arguably *wrong* -- Carton is meant for reproducible
installs, and installing on another OS/perl version isn't really a good
basis for that.

Could we think about something like:

 - use cpanfile.snapshot just on mySociety managed infrastructure, and fall
back to cpanm --installdeps or similar for reusers?
 - yes, that'd mean we may get more version clashes, but we'd trap those by
either:
    - testing on multiple infrastructures?
    - user reports, and fix

(Not ideal either...)

Hakim
--
*Developer*, mySociety Services
http://www.mysociety.co.uk

On 18 March 2015 at 18:14, Matthew Somerville <notifications@github.com>
wrote:

> Just to note this commit will need to be altered so that it contains only
> the new modules (Email-MIME, Email-MIME-ContentType-1.017,
> Email-MIME-Encodings, Email-MessageID, and Mime-LITE), no removals, so that
> this still works okay on older OS versions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mysociety/fixmystreet/commit/c84c9d4f0dc07d0616f765451e958ec77ff916c3#commitcomment-10270578>
> .
>


html_options ||= {}


Its still works, no need to do any thing else.
Just use latest sql code.


Very good! Thanks! But do we have the underscore function already?


Please revert the change. It goes not in line with the documentation. There it was advised to use it like:
icon('flag')
which is impossible with your commit. Or take a look at my comment in the code.


I see and try to understand this.

Just a question: it is possible to move the inner part of ``if (initialized == null)`` into its own small private / protected maybe synchronized method to lower level of indent and make it easier to read later?


If you want this gem in both development and production, you may as well not limit it to a group, so `gem 'puma'` will do.


Clean Project, should then start in supervisor mode!
(see in CCS in the right bottom, 'USR' or 'SPV')


@mnapoli  thanks


@squiddle this breaks a number of clients (for instance s3cmd)


I'm a new programmer in here, please help me!


Mais uma que teve que ser reescrita :(
:+1:


Thank you my friend. I'll give proper credits when uploading to git hub. Just edited the files on my PC. :)

hope that you are well! miss your development!


TeamCity OsmSharp :: GitHub - OsmSharp [Build 997](http://build.osmsharp.com:8080/viewLog.html?buildId=4647&buildTypeId=bt16) is now running


Can you provide UNIX-style command to print Fibonacci series?


Good point. Willfix.


small space added for demonstration


Both can be interpreted as Let("i",Literal 0.0,Sum(Var "i", x,Func (fun (a:value) -> a))) without changing meaning, right?


thx


According to this, the issue was fixed in boost 1.57 https://svn.boost.org/trac/boost/ticket/10038. Should we do this?

https://github.com/anttiviljami/PotreeConverter/commit/3fe7e27a555ef377d34d0f981477f55cf6d23cb9


Maybe it breaks other stuff


We're not validating that the :keys are valid keys, aren't we? That means that I can send you a random hash with 10,000 key/value pairs via ConfigurationsController and they will be saved to the database, no?


I don't understand this ... why did you need to add "object" to the userInfo dictionary if you could just add it when when you post the notification?


This should be the version of the Desktop Entry Specification and not the version of your software.
See http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html#recognized-keys


Can you implement a generic and effective worker task to accept the requests of computing oriented input?


Hello man, do u know navbar stuff coding? Do u use g+ or freenode?


You can actually concatenate constant strings like this in C:

```c
#define MTK_INIT \
    "$PMTK314,0,1,0,1,5,0,0,0,0,0,0,0,0,0,0,0,0,0,0*2D\r\n" \
    "$PMTK220,1000*1F\r\n" \
    "$PMTK220,500*2B\r\n"
```

May make it easier to read? It'd look more like your set of commented-out defines at the top of the file.


your improvement is a further consequence of:
https://cognifide.atlassian.net/browse/SLICE-107?filter=-2

I will create a PR for it today.


Note to self: apparently I included `invenio/modules/relationships/testsuite/test_nodes.py` in the implementation commit, while it should only be in the tests commit. Fix this.


I don’t like the idea of separating that code. My feeling is that it belongs together in one method. But I agree that tabbing indent causing 8 en indents is just *too* much. I prefer four spaces indent—but that’s a different discussion.


True, and that’s what in my pull request, the eval does on encountering ```Were``` when traversing the AST.

> On 19 Mar 2015, at 10:49, Johan von Tangen Sivertsen <notifications@github.com> wrote:
>
> Both can be interpreted as Let("i",Literal 0.0,Sum(Var "i", x,Func (fun (a:value) -> a))) without changing meaning, right?
>
> —
> Reply to this email directly or view it on GitHub.
>


Yeah, good idea. Implemented it this morning. Works.


never a read / write hdd going to be faster than the speed of RAM.
Having a terminal with 1 gb of RAM is unneeded zram only generates lag for its low bandwidth, the zram is long gone with 1GB of RAM terminals....default=y


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.997](http://build.osmsharp.com:8080/viewLog.html?buildId=4647&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 519 Build time: 0:0:0


It was a general remark of indent level not how an indent is used (TABS vs SPACE). But you are correct: it should be long together.


@skalk: \<nitpicking\>Bad indentation.\</nitpicking\>


This is the error, if we that define

```
CMakeFiles/PotreeConverter.dir/src/stuff.cpp.o: In function `boost::filesystem::copy_file(boost::filesystem::path const&, boost::filesystem::path const&, boost::filesystem::copy_option)':
stuff.cpp:(.text._ZN5boost10filesystem9copy_fileERKNS0_4pathES3_NS0_11copy_optionE[_ZN5boost10filesystem9copy_fileERKNS0_4pathES3_NS0_11copy_optionE]+0x27): undefined reference to `boost::filesystem::detail::copy_file(boost::filesystem::path const&, boost::filesystem::path const&, boost::filesystem::copy_option, boost::system::error_code*)'
collect2: error: ld returned 1 exit status
make[2]: *** [PotreeConverter/PotreeConverter] Error 1
make[1]: *** [PotreeConverter/CMakeFiles/PotreeConverter.dir/all] Error 2
make: *** [all] Error 2
```


Do you test if it works?
It's impossible that it works, because |textureID| is not texture id. Real texture id belong to GPU process. How can WebCL context in render process shares GL context in GPU process?


More on this:
http://stackoverflow.com/questions/18783790/boostfilesystemcopy-file-missing-symbol-in-c11


v0.2.1 update pushed.


Fixed in b6c9294793904dca536c646e52e7467e6c7c9932.


Wrong repo?

S

On Thu, Mar 19, 2015 at 4:49 PM, Zach Loafman <notifications@github.com>
wrote:

> Both @satnam6502 and E2E tests confirm: This code is no longer needed
>
> Deletion is wonderful.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/zmerlynn/kubernetes/commit/6e35611a488e639726207b699aa4ed53d62ce2ca
>


@marcind This branch and commit is where I have broken out the links model for your review. Definitely open to adding more detail or tweaking content on https://github.com/CenturyLinkCloud/APIDocs/blob/links-revamp/Getting%20Started/api-v20-links-framework.md based on your feedback.


last loose end - closing #4


Thanks!


ICSSolarCollector_#4723 (Nigusse) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Yes. Yes it is. I must have somehow missed that foreach.


I am getting a build warning which fails build here
```
gcc -g -O2 -fstack-protector-strong -Wformat -Werror=format-security -I. -g -Wall -Wextra -Werror -pie -fPIC `pkg-config --cflags vchan-xen` -D_FORTIFY_SOURCE=2  -c -o qrexec-agent.o qrexec-agent.c
qrexec-agent.c: In function 'handle_server_exec_request':
qrexec-agent.c:254:9: error: ignoring return value of 'write', declared with attribute warn_unused_result [-Werror=unused-result]
         write(client_fd, &params, sizeof(params));
         ^
cc1: all warnings being treated as errors
<builtin>: recipe for target 'qrexec-agent.o' failed
```


This looks fairly solid. Haven't tested yet but the structure looks cleaner. Unfortunately because the other methods were public we will have to retain them (documented or not) so we'll need to use the new methods to handle the old methods.

For instance
```
_setDisabledClass: function() {
  this.checkbox('setDisabledState', true);
}
```


i have the same problem with your update
https://github.com/odoo/odoo/issues/5820


is the `e = e = ` intentional?


This is the problem I'm having with gerrit
https://plus.google.com/+PaulBeeler/posts/P8BcmxwcKxA


with zip function, you should check for same sized elements. I seem to recall the results of different sized lists was funky


Any chance you might spin a release with this in it soon?  Would love to not have to run off a fork.


live version


TeamCity xBIM :: XbimExchange :: Build [Build 129](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1182&buildTypeId=XBIM_XbimExchange_Build) is now running


Good, this always bothered me as it wasn't very flexible


TeamCity xBIM :: XbimExchange :: Build [Build 129](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1182&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Compilation error: Xbim.COBieLite.SerialisationHelper\SerialisationHelper.csproj Build time: 00:00:16


Unnecessary import


Does the intent system require this? ie, could I do a multi-point to point, but the source is really just a point?

This may be useful if you were modifying the group of senders


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1065
success: http://build.servo.org/builders/linux2/builds/1065
success: http://build.servo.org/builders/android/builds/1099
success: http://build.servo.org/builders/mac1/builds/1281
success: http://build.servo.org/builders/mac2/builds/1281
success: http://build.servo.org/builders/gonk/builds/252


ResultsSchema (sanyalj) - i386-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-69.31%25-red.svg)


we should get rid of this.


same here.


lel


I would consider getting rid of the `motd` variable here.

In fact, line 24 will actually work exactly as it is now if we remove line 22. `motd` on line 24 actually calles the `getMotd()` method.


Better than straight commenting it out :laughing:


fast-forwarding master to auto = 794d9f69


Just wondering, is there some reason we cannot use @FragmentById injection of vms and event fragments?


But if it's an NPE then people don't get information :crying_cat_face:


meh


We should try put put `@Nullable` on methods which we know may return nulls.


This (and similar bits of code in this commit) would be easier to read if the two elements were on separate lines.


@seanzxx Is ```os.remove(pf_config_file)``` needed? When you ```open(pf_config_file, 'w+')``` doesn't ```w+``` overwrite the file if it exists? If so why not simply leave it on the system and overwrite it each time?


Why not?

    try:
        self.sequence_number = int(cmd)
        cmd, arg, line = self.parseline(arg)
    except (ValueError, TypeError):
        pass

This avoids increasing the indentation and (depending on who you talk to) is "more pythonic."


Why do we need to check for both undefined and null on firstChild?


90693826-Central-cooling-coils-not-autosizing-to-correct-airflow-in-some-cases-#4810 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1162 of 1164 tests passed)

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


afinal o truque para o facebook era ter o url com / final!!!
funciona:
http://www.facebook.com/sharer/sharer.php?u=http://osgeopt.pt/blog/site/osgeo/novo-site/

não funciona:
http://www.facebook.com/sharer/sharer.php?u=http://osgeopt.pt/blog/site/osgeo/novo-site


We do not need to sort here. The data in Cassandra is already (and always) sorted. It is worth noting though that we're storing the data in descending order since we are usually interested in recent values.


rax retry tests


ah, line 154 is the remove but i just noticed it's calling the incorrect property.

the observer should be added to layer not playerLayer, but it actually does the same thing. can update for consistency. good eye. :eyes:


No need for this `tearDown()` method as it does nothing.


I was getting some KVO crashes and it made me do a double take. Not related, but might as well pair them up anyways


Hi, thank you for this excellent patch! I tried it and it works really well. Just some minor comment about the fragments: was it necessary to split them out into separate files? Also is it any beneficial to wrap framgment into 2 layers of noop Layouts and then again into one fragment? I can merge this patch as is (the tabs look really nice) but I wonder if there are some deeper reasons for this organization. Thanks


ここで定義した boolean型 の変数 b はどこかで使ってますか？


Sadly it only allows 4.2.0, not 4.2.1, which is a bug. See #92


That didn't work ?


i think this still needs a

if (pid && getpid() == pid)
   _set_affinity(id);

?  so that it will work after the thread is created


At least one of these two layouts (relative or frame) is unnecessary.


There is a .make in the timepicker module that should take care of this library. It does not need to be included here.


Thank you for this. Save me loads of time.


Nice! I'd have preferred an OCaml script, but that's probably too much of a burden to write :)


load_candidate is no longer required since we've updated gdal.


first


Was this a back merge?
Sorting out the history could get exciting.


Oops, sorry. I don't see that effect, but I can imagine it can happen in some themes. Another reason to avoid symbols for snippets. I'm reverting my patch.


i think it was python stdin forwarding not working with node. using the shell to forward stdin was better.


@foo86: One more little thingy. This should be striped.

> libdcadec.a: error adding symbols: Archive has no index; run ranlib to add one


rdaf


Yeah you are probably right.. I think by itself it wouldn't be a big deal, but linked to a person's username and in a public database, not good.  I'll take it out.


Good thing I found your onCommand method... I just switched around all our 25+ cmds! :P
That's some mess but still really awesome plugin <3
(talking about this repository: https://github.com/RedstonerServer/redstoner-utils)


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1162 of 1164 tests passed)

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


bkjhb


created new branch Week2_app and added content


Good catch, I'll fix that.


Gopal
You created text2.txt under the Master , and that is not correct. You should create text2.txt under the branch that you have created.delete text2.txt and recreate it again under the Test_Branch
correct it and let me know when you finish

Thank you..Haifa
Date: Thu, 19 Mar 2015 19:13:07 -0400
Subject: RE: [Team12_Repository] Update names.txt (7de5c56)
From: dr.csc211@hotmail.com
To: reply@reply.github.com

Ok

Sent from my Samsung Galaxy smartphone.

-------- Original message --------

From: gopalmorampudi <notifications@github.com>

Date:19/03/2015 7:08 PM (GMT-05:00)

To: HaifaAlsalem/Team12_Repository <Team12_Repository@noreply.github.com>

Cc: HaifaAlsalem <dr.csc211@hotmail.com>

Subject: Re: [Team12_Repository] Update names.txt (7de5c56)

Yeah I am done with that. Please proceed to your next action and notify me when you complete.

—

Reply to this email directly or
view it on GitHub.


I see it now. The patch has been reverted. I'll try to limit my activity to real issues before 2.18.1.


Thanks for letting me know, fixed it


I sure hope this fixes it! :D


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1162 of 1164 tests passed)

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


Oh! Sorry I didn’t know that lol

Sent from Windows Mail

From: Gordon Cassie
Sent: Thursday, March 19, 2015 18:41
To: saytay/emotions
Cc: Knight Lin

Haha, I'm sarah's brother so not a total random just creeping on github repositories.

—
Reply to this email directly or view it on GitHub.


if namespaceHomeLocation is null, the exception is not being thrown.


ConstructionsGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2175 of 2195 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.09%25-yellow.svg)


This obviously isn't the final solution; I'm just trying to figure out if that particular git call is actually the problem, or what.  I just did this on my home Windows box and it completes successfully.


this does not do what you think it does


Should this line remove the "#" character and not the "?" character?


`assert` is not a function... hopefully no one ever adds a message like `assert(self.get_db() != CommCareCase.get_db(), "message")` ... it will never fail.


Minor fixes such as re-wording.


Heh. You weren't supposed to see this commit yet. As I mentioned, I
realized before I made a PR of it that is accidentally deleting the echo.
On Mar 19, 2015 5:02 PM, "Satnam Singh" <notifications@github.com> wrote:

> Wrong repo?
>
> S
>
> On Thu, Mar 19, 2015 at 4:49 PM, Zach Loafman <notifications@github.com>
> wrote:
>
> > Both @satnam6502 and E2E tests confirm: This code is no longer needed
> >
> > Deletion is wonderful.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/zmerlynn/kubernetes/commit/6e35611a488e639726207b699aa4ed53d62ce2ca
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zmerlynn/kubernetes/commit/6e35611a488e639726207b699aa4ed53d62ce2ca#commitcomment-10297208>
> .
>


Whoops, looks like the merge went the wrong way


@sindresorhus can you explain why did you remove this file?


"use strict"
Its better if we use strict mode ( https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode ) inside all our javascript code to minimize errors.


we should free contigMask and maskPrefixSum before the error here


You all were doing compat in preinit, the blocks don't exist yet. Should have done it in Compatibility. See https://github.com/TheCricket/Chisel-2/commit/a65331cbab22fef498c726ecc120675a817a74cf


This is certainly much better.  Will Remo accept the term "interference" on
Line 121?  Might we insert "local" or "electronic" or "non-atmospheric" before
the offending word to be clearer?


ICSSolarCollector_#4723 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


merge issues here, since we maybe didn't have the latest? i didn't change this code.


omg haha.. i just spent 20 minutes trying to figure out where and when i checked this into my swift1.2 branch lol   I actually added this so i thought i accidently committed it


same here, etc.


var spanDom = $('span', this);

Variables should be declared according the "strict mode" spec https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode#Simplifying_variable_uses


omg haha ... i just spent 20 minutes trying to figure out how i accidentally committed this to my swift1.2 branch...because i also add this method on my fork


Looks like a merge issue with the reduction stuff. I only replaced the basic reduction by dimension, not the reduction with index etc. code.


ICSSolarCollector_#4723 (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


Which test does this break?


@leahpadgett looks like this variable `cheeseString` is never used. You might see a compiler warning on that line - the yellow triangle.


We should better use a button here instead  an anchor and then avoid the "javascript:void(0)" call.


ohh yea, and I also updated the tabs! forgot about that.


HotFixPurchasedAirUnitTest (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1671 of 1672 tests passed)

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


Thanks switch statements


HotFixPurchasedAirUnitTest (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


@leahpadgett two points here.

1) it's best to not use a variable of the same name twice in one method (even though they are different types - NSstring and NSUinteger). The first variable is the name of the argument of the method ` - (NSString *) numberOfCheesesStringWithCheeseCount:(NSUInteger)cheeseCount {`

2) the value you're adding to the `[NSString stringWithFormat:@"%d cheese", 1]` will never change (think of the if test), so why not jsut make it a static string like `NSString *cheeseCount = @"1 cheese";`?


@gromakovsky done


@leahpadgett you're hard coding a result here. you need to use the argument that contains the value passed to the method.  `NSString *cheeseCount = [NSString stringWithFormat:@"%d cheeses", cheeseCount];`

It's confusing because the double use of cheeseCount as a variable name.


I imagine you've already fixed the "heterodyn".


lol


okay, good advice thanks


i was following what was in one of the checks.  should i just check for truthy?


that makes more sense, I'll change it to return undefined, thanks again


@birarda if you pull this out cleanly you get to pick a name :)


Math.max（）の引数の型は何？
array[0] の型は何？


@leahpadgett I see you said this was hard. When we struggle we learn. It's perfectly okay maybe even ideal. You're learning to think in a new way and that takes effort (sometimes called struggle).

TIP: don't add superlative type language when you're describing challenges - like 'reallyyyy'. Just say it was a challenge and you'll feel differently about it.

Keep leaning into this. Thank you for the hard work.


I like seeing this go away :)


And this! :+1:


I really like that you tidy up as you fixing bugs :+1:


I've been holding off on the batch updating while investigating into how we can get magiccard.info URL for cards without a collector number. I'll continue on the URL generation then.


creat cache inverse matrix function for coursera


This class is incredible! Class presentations are simple (even for a first timer like me) and easy to follow. The ability to virtually interact with fellow classmates make this truly an awesome experience.


Sigh.


HotFixPurchasedAirUnitTest (mjwitte) - x86_64-MacOS-10.9-clang: OK (1665 of 1666 tests passed)

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


I would add some comments explaining what this is doing to find the "next" node.


what if nr != 0 && err != nil ? we should not forget to handle the error.
I suggest:

```
nr, err := getAvailableInputEvents(...)
if nr > 0 {
// ...
}
if err != nil {
  // return error
}


Overall, this commit looks pretty good to me. Nice clean-ups that make this more readable.


please also check the new version of the script ;D