Thanks for letting me know! Have you updated past
https://phabricator-dot-kythe-repo.appspot.com/D435 (committed as
5deadc9d33b369a850f0383604c4b667ab431a6b)?

Our issue tracker is hosted in Phabricator. We'll work on making this more
discoverable.

  Luke

On Wed, Jul 29, 2015 at 3:47 PM, Pierre Bourdon <notifications@github.com>
wrote:

> Could not find an issue tracker, so reporting here...
>
> This commit looks partly broken. Here is a test case where it causes
> indexing errors:
>
> class C {
>  protected:
>   /// \c bar
>   int foo();
>   int bar();
> };
>
> error: 'bar' is a protected member of 'C'
>
> I've also managed to trigger this which I think is related -- but couldn't
> find a simple repro:
>
> indexer: /home/delroth/src/kythe/third_party/llvm/llvm/tools/clang/lib/AST/DeclBase.cpp:1341: DeclContext::lookup_result clang::DeclContext::lookup(clang::DeclarationName) const: Assertion `DeclKind != Decl::LinkageSpec && "Should not perform lookups into linkage specs!"' failed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/kythe/commit/ed65bff28f9500680b06bc0ef0542b97a9b374a0#commitcomment-12432706>
> .
>


@jgrigonis Actually, yes ;)
MS can also affect [emotional and behavioural regions of the brain](http://www.nationalmssociety.org/Symptoms-Diagnosis/MS-Symptoms/Emotional-Changes#section-5), which is quite scary.

It can affect your very human ness, your emotions, your feelings, your body, your mind.


Because the conversation holds ourInstanceTag and theirInstanceTag. Instead of copying it over the place it's easier to just send it in to serialize function. (look at the implementation in otrv3 instead of otrv2)


```
T foo[42];
some_algorithm(begin(move(foo)), end(move(foo)), ...);
```
maybe, but that would require `begin` and `end(T(&&)[N])` to return `move_iterator<T*>` to be useful.

Perhaps `begin(T&& t)` & `end(T&&t)` should be `safe_begin` & `safe_end` in general?


Thanks, Bryan.  I will probably pull this in a day or so, I just need to write a unit test for it.


@rusuandreirobert The second one (`views/embed.js.php`) shouldn't be changed. It's actually a poorly named variable that is from Memberful, not WP (see earlier in the code).


“创建Javascript分析以及开始调试会话。” 我觉得这里可以意译，比如：创建 JavaScript 性能分析器以及进入调试模式。

直译可能会对新手产生疑惑，因为没有调试会话这个概念，而且 创建xxx分析 也不是很通顺。


TeamCity Bender :: Deploy [Build 3.0.16.0](http://build.mikeobrien.net/viewLog.html?buildId=2448&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3242, ignored: 5 Build time: 00:01:15


I have removed the unnecessary len(b64) <= c.FragmentSize in e0c7c2be1e029cf9cdbb343a76ead16fd86c3871

But for b64 length do you mean we can use a []byte without specific length?


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 227](https://teamcity.labkey.org/viewLog.html?buildId=263510&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabkeyTrunk :: build Build time: 00:00:00


I would say you don't need all of these assertions between 263 - 294. Just dispatch the event that exercises the method with your change.


So `raw_input` is a function which returns a value. To remember the value and use it later, you store it in a local variable. For example:

```python
answer = raw_input(questions['strong'])
```

The `answer` variable will be a string value of the text the user typed in. Does that make sense?


If you have looked at the fragmentation.go file, you will see that the whole if statement you have there is completely unnecessary.


MemberArray_Reduce (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (1777 of 1777 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)


It's just a word...


the only place ::xinetd::params::default_user and ::xinetd::params::default_group are referenced is in the xinetd::service defined type where it is used as the default for the 'group' and 'user' class parameters.  You can override it there when you declare the resource.

```
xinetd::service { 'whatever_service':
  group => 'not_root',
  user   => 'not_root',
}
```


`.to_not raise_error` is redundant. The example will fail when an exception is raised.

You included it for documentation proposes?


Frank Zappa's thoughts on the matter:

https://www.youtube.com/watch?v=8ISil7IHzxc


@SimonCropp it may be my non English background but I find it hard to understand what it all means, words like `Transiently` it can confuse non English speaking users.
So we trying to explain to the user that the response may never arrive, as I said I would prefer to convey that in the `remarks` section of the api doco, and explain why and if they need guarantee maybe they should use a proper handler.
Also if we start digging into when a response may never arrive we may find that is not so much of a problem any more. Before we introduced the callback queue, this was problematic when scaling out but now it is not so much, so do we still need to convey that at the method name, I doubt it.

So in summary, I find the new name quite confusing, IMO it also hides the real purpose of the api.

Hope this clarifies my comment above.


It is true that there needs to be control and guarantee over what operations can run trigger GC, and which ones can't.  But right now that's pretty well defined: a call to any of the Do_XXX evaluators (or anything that calls them), a manual call to Recycle(), or some checkpoints in the initialization process.  Hence, any value creator that needs to invoke the evaluator to get its job done should be structured accordingly.

The rule would be something like "Don't create field 1 of a structure visible to the GC, then call Do_Next to produce the thing you're going to put in field 2 of that structure."  You either need to make the Do_Next call up front and *then* make the structure, or initialize everything properly so the GC isn't going to choke on the half-made thing.

I think rather than going around with GC disablement/enablement, what's needed is a warning if something you call is invoking the evaluator without you knowing that's happening.  Anyway, yes, it's another issue of the GC story (along with proper triage/infant handling)


Great suggestion.  I'll incorporate it while I rebase.


rspec warns you now when you use `.to_not raise_error(klass)`. Maybe we should disable that warning.


Oh! Didn't notice that fix commit. Local repo synced to dea222dd90c9c8c8d9411ace7b4f6e3dc8faeb49, so a few revisions early. I guess it explains why I was only seeing that issue locally and not on my "prod" server...

Thanks! Now I feel stupid. :)


@dkubb No I meant we do not need the `expect { something }.to_not raise_error` *at all*. `something` is enough.


> You included it for documentation proposes?

I generally prefer the error that occurs than to allow the `it` block to fail altogether. I think it represents my intentions better.


The funk file controlled text on tiles for a a star algo test which I think is no longer needed


Initially, in the actioncables-example application I did not understand the sparseness of the app/channels/application_cable/channel.rb and app/channels/application_cable/connection.rb files.
* Are these files going to be relatively sparse in most cases?  I say this because they just seem to define a generic connection and a generic connection.
* I noted that these files collectively define the ApplicationCable module across two different files.  In each of the files, connection.rb and channel.rb a class is defined.  Thus the ApplicationCable module contains two different class definitions spread across two different files.
* Class Connection subclasses ActionCable::Base::Connection while Class Channel subclasses ActionCable::Base::Channel.
* Nothing else subclasses Class Connection, so can we assume that any connection is automatically handled by that code?  So in terms of instantiating a connection, i.e., opening up another browser to the application, can we assume a connection is automatically established as long as the user is verified?  Again I am referring to the action cables-example repository.
* I note that app/channels/comments_channel.rb subclasses the Channel class within the ApplicationCable module.
* In the follow method in the comments_channel.rb file, I do not know what the following code is doing: stream_from "messages:#{data['message_id'].to_i}:comments".  Is ActionCable on the server side receiving a stream from a URL in double quotes?


You can use the defaultCopy method now to simplify things


testing testing WHAT A STUPID DIFF


MemberArray_Reduce (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (1777 of 1777 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 would recommend trialling this api in slack with non familiar people and see what they think the api method means/does


@krainboltgreene I hope you're not that horrible of a person in real live.


Work well tested on my Mac / TextMate version 2.0-beta.7.4
Are you backward compatible with RSpec 3.2 ?


I haven't come across this as a problem, but I remember you had some issue at some stage? If a timestamp matches a given regular expression, what is the chance that it isn't a valid time though? It's much harder to write code that would put an invalid timestamp (eg. "2015-15-40 34:61:61") in a response than a valid one.


I hope he gets his brain checked out, could be something really wrong with it.


Was meinst du? Wo ist die 2.?


Please add tests for the numeric case.


Follow naming conventions.


your welcome


Statements vulnerable to exceptions should be kept in Try Catch block. It is possible the conversion may throw an exception.


Put comments on functions for describing what they do.


I did consider that, and I think I like having less state better.


* ~~clicking "RSVP" in the nav should always take you to rsvp-search-view~~
* in rsvp-multiple-results-view and rsvp-search-results-view, the "Search again" link should take you to rsvp-search-view
* we should probably print the response in rsvp-confirmation-view in a more human-readable way than a json blob?


This closes #6


the e.g. was lost; what was it?


assert.h - I think maybe it got interpreted as an html tag?


https://github.com/portertech/kitchen-docker/pull/146 gets kitchen-docker working in the :build_context false case on Ubuntu 14.04 for me.  I'm not planning to dig into the build_context true case at the moment since I'm not using it.


@wsnipex, @montellese: got another question for ya. how can I use this patch file: https://github.com/garbear/protobuf/commit/f7f6552.patch instead of forking and cherry-picking?


Are you sure this compiles? You got rid of the GPIO package.


Did not check, but at this point many games (known to work with Citra, that is) have been tested with the VSJIT, and no reverse jumps have been found. That being said, I feel that it's sufficient to just keep this assertion until we find something that actually does this. It can always be added in a followup PR if needed.


Thanks for this!


I followed the previous code writing style, a look at the function add_custom_templates on line 323 form_filters.php file


Also removes redundant arguments.


Remove commented out lines


which is also the same, I followed the previous code writing style, a look at the file in line custom_class previous row


can you do this as try with resources ? in that way you don't have to close.


使用方法：

[WQAlertView showConnectWWAN];//网络状态为移动网络
[WQAlertview showConnectWifi];//网络状态为wifi
[WQAlertView showNotNetwork];//网络状态为无网络


okay I will update this


@ronkorving This PR does not introduce the text you are commenting on, so this comment seems out of place. Please PR any contributions you would like to make to the node docs, they can surely use improvements.


Also change change platforms/ios/cordova/lib/list_devices line 24 to var Q = require('q'),


I agree that this is more readable, but this register is called SYSCLK in the reference manual. In my experience having names match the device documentation really helps with debugging; can you revert this rename?


That's the gist of it. At the moment the only way to "kill" ABL after `G29` is to do `G28` again or reset the machine. Once the `plan_bed_level_matrix` is set to something other than `identity` it continues to operate the machine with rotation.


lol retards


Delete dead code


Ditto


Missing a "p" in Power_Down_Deepsleep


Clear_Standby_Flag for consistency with Clear_Wakeup_Flag?


Wakeup for consistency


Should be ANDPS.


Seems once applied this patch, still got some permission issue in "Network  Data Usage Stats",
When user tapping on QUERY SUMMARY FOR DEVICE/QUERY SUMMARY FOR USER, java.lang.SecurityException: NetworkStats: Neither user 1110063 nor current process has android.permission.READ_NETWORK_USAGE_HISTORY...
Is there still some changes in permission or app installed as system-priv needed? Thanks. :)

/Joey


Can you give this a more descriptive name, like Interrupt_Index or similar?


Typo?


Extra space before :


asdfasdf


Dead code?


Once Linus gives up control of the kernel we're in for a ride.


This needs to be fixed not removed


Ack


XORPS, please.


This needs to be reverted to `gzip | zcat`. Because it tests piping at the same time.


Sensitive-ass SJWs. Fuck off, retards.


Show Message Box is not that good.
Looks like "/range" could be a way to go.


Done and can be found here:
http://people.redhat.com/~vrozenfe/qxlwddm/qxlwddm-0.12.zip
Best regards,
Vadim.


Very good


You can put this inside the switch, and then use the flags for the jCC, instead of having to recompute the flags.


Yes, they do, unless they need to maintain flags.


it seems this function is not in use, okay I'll remove


This fixes a crash that could occur right after Bronann obtains the wooden sword. If you open the menu, and from the inventory, equip Bronannn with the sword; then without exiting the menu, try to switch to the "equip" section, the game will crash with a segfault. This would happen because MenuMode::_object would dangle, referring to a GlobalObject which was previously freed.

This is what the stack would look like when the object is freed:

#0  _int_free (av=0x7ffff51a8cc0 <main_arena>, p=0x1f64110, have_lock=0) at malloc.c:3996
#1  0x000000000097baf7 in vt_global::GlobalWeapon::~GlobalWeapon (this=0x1f64120, __in_chrg=<optimized out>)
    at /home/endoalir/ValyriaTearE/src/common/global/global_objects.h:342
#2  0x000000000098262c in vt_global::GameGlobal::_RemoveFromInventory<vt_global::GlobalWeapon> (this=0x128e840, obj_id=10001, inv=...)
    at /home/endoalir/ValyriaTearE/src/common/global/global.h:1296
#3  0x000000000096d51f in vt_global::GameGlobal::RemoveFromInventory (this=0x128e840, obj_id=10001)
    at /home/endoalir/ValyriaTearE/src/common/global/global.cpp:638
#4  0x000000000096e736 in vt_global::GameGlobal::DecrementItemCount (this=0x128e840, obj_id=10001, count=1)
    at /home/endoalir/ValyriaTearE/src/common/global/global.cpp:736
#5  0x0000000000c38612 in vt_menu::private_menu::InventoryWindow::Update (this=0x1ee2f98)
    at /home/endoalir/ValyriaTearE/src/modes/menu/menu_views.cpp:611
#6  0x0000000000c2c296 in vt_menu::private_menu::InventoryState::_ActiveWindowUpdate (this=0x1ee1a00)
    at /home/endoalir/ValyriaTearE/src/modes/menu/menu.cpp:391
#7  0x0000000000c29ff8 in vt_menu::private_menu::AbstractMenuState::Update (this=0x1ee1a08)
    at /home/endoalir/ValyriaTearE/src/modes/menu/menu.cpp:77
#8  0x0000000000c30a80 in vt_menu::MenuMode::Update (this=0x1ee0780) at /home/endoalir/ValyriaTearE/src/modes/menu/menu.cpp:980
#9  0x0000000000a83f5b in vt_mode_manager::ModeEngine::Update (this=0x1273690)
    at /home/endoalir/ValyriaTearE/src/engine/mode_manager.cpp:332
#10 0x0000000000d792af in main (argc=1, argv=0x7fffffffd9e8) at /home/endoalir/ValyriaTearE/src/main.cpp:606

And then, the game would crash with a backtrace that looks like this:

#0  vt_menu::MenuMode::DrawEquipmentInfo (this=0x1ee0780) at /home/endoalir/ValyriaTearE/src/modes/menu/menu.cpp:1214
#1  0x0000000000c47ad1 in vt_menu::private_menu::EquipWindow::Draw (this=0x1ee4fe8)
    at /home/endoalir/ValyriaTearE/src/modes/menu/menu_views.cpp:2261
#2  0x0000000000c2c364 in vt_menu::private_menu::InventoryState::_OnDrawMainWindow (this=0x1ee1a00)
    at /home/endoalir/ValyriaTearE/src/modes/menu/menu.cpp:407
#3  0x0000000000c2a593 in vt_menu::private_menu::AbstractMenuState::Draw (this=0x1ee1a08)
    at /home/endoalir/ValyriaTearE/src/modes/menu/menu.cpp:172
#4  0x0000000000c3208a in vt_menu::MenuMode::Draw (this=0x1ee0780) at /home/endoalir/ValyriaTearE/src/modes/menu/menu.cpp:1196
#5  0x0000000000a83fa7 in vt_mode_manager::ModeEngine::Draw (this=0x1273690) at /home/endoalir/ValyriaTearE/src/engine/mode_manager.cpp:341
#6  0x0000000000d79219 in main (argc=1, argv=0x7fffffffd9e8) at /home/endoalir/ValyriaTearE/src/main.cpp:582


This needs a docstring for the auto-generated documentation.

My solution to make `test_safe_pipe_stdout_read` pass would be to have `def start(self, keep_stdin=False)`. And when `keep_stdin` is True you keep it open. The docstring would have a [big red warning](http://sphinx-doc.org/markup/para.html#directive-warning) explaining that you should only use `start(keep_stdin=True)` when you know what your doing.


There's no `'greetings'` here.


Clear high bits:

mov32 addroffs_reg, addroffs_reg

Clear low bits:

mov32 tmp, addroffs_reg
xor64 addroffs_reg, tmp


Not sure I understand the comment here, but the HLSL Decompiler does not need that float management code because it takes the float values directly from the binary, not from the ASM text.

With the addition of the %.9g output format, I think that Decompiled HLSL should now always have correct constants.


master


I wouldn't recommend this; loads are generally cheaper. But why do we need the extra level of indirection?

You can just do MOVAPS(SCRATCH, M(&one)).


fix_intel_compile_linux (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (742 of 742 tests passed)

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


You may run into issues with your approximation differing too much from the GPU's. Lets hope we don't.


Or do cmpps + movmskps


Ah, this is definitely nicer! I didn't realize you could use the notation on the left like that either. That's pretty cool.


Maybe add a TODO to use FMA here (it's supported in the emitter)


Good question. I haven't been able to think of a good way to avoid having one-client-per-service, but it would be great to hear if you have any ideas. My concerns are:

  - I want to avoid a single "god" `Client` that knows about every service and so has a huge API
I think if the client is too large, it will make documentation a nightmare. Also, the number of services will only grow as time goes by.
  - Developers should be able to use as few or as many services as they like
  - Developers should be able to use different credentials/HTTP clients/project IDs for different services (or even for the same service within a single binary)

Having a `Client`-per-service is the best compromise I've been able to come up with. For what I've seen, most binaries/servers tend to use a handful of different services, which would result in only a small number of clients.


See #284


AND(32, R(reg), Imm32(0xFF)) is a longer version of MOVZX(32, 8, R(reg), R(reg)).


Real code ends lines with semicolons;


to my knowledge, because it has no default value in the function with function ($ currency = array ())

so if we add a new filter he already automatically add


Same comment here about the needless extra level of indirection.


I had to do remove the platforms and add them back to existing project.


测试


With this, CrossPlatform no longer builds for non-win32 builds:

src/Engine/CrossPlatform.cpp:79:2: error: ‘cerr’ is not a member of ‘std’

src/Engine/CrossPlatform.cpp:808:21: error: ‘Language’ has not been declared

src/Engine/CrossPlatform.cpp:808:42: error: ‘Language’ has not been declared

src/Engine/CrossPlatform.cpp:809:43: error: ‘IMG_Load’ was not declared in this scope

restoring the relevant includes gets me building again.


I have not tried wearing get_template_part while I followed the previous code,
get_template_part and to my knowledge it is only used in the theme could not for the plugin


As a convention, we should place the parameterized test function directly above the set of parameters passed to it.


These derives can be consolidated into one line: `#[derive(Debug, PartialEq, Eq)]`


@dysinger: I don't think so, if that's in the `inner` (which it appears to be) that will cause it to always fail when Docker is enabled for builds.


Combine these `derive`s into one line.


I'm aware of this. I copied them from the `ai` module.


What about testing an empty string?


Oops! That was actually #4830...


Okay, I added a new feature, multiple bank account, I could not follow the existing code because it will be more difficult. so I made a new option and I save it as a new option that the contents of the array.


This isn't necessary, the compiler can handle ``self.t = xxx`` directly in kernel code.


LGTM


That's a good boundary test. It will execute the same lines as this test, but I'll add it in.


I think line 62 should be updated like below:
if mdata.analysis.find({"target.file_id": ObjectId(analysis["target"]["file_id"])}).count() == 1:


我太聪明了，哈哈哈


这是个苦力活啊


Yeah, had tried that. But, unfortunately you can't do that today because ``ClosableIterator`` is not ``Closable``. It defines its own ``close()`` method.  I've reported https://issues.cask.co/browse/CDAP-3130 to address this later.


#include<string.h>
#include<unistd.h>
#include<sys/types.h>
#include<sys/stat.h>
#include<fcntl.h>
#include<stdio.h>
#define CAPCITY 1.5*1024*1024*1024
#define Location SEEK_CUR
int main(int argc,char *argv[])
{
int fd;
char msg[]="This is a 1.5G file";
fd=open("creat_big_file",O_CREAT|O_RDWR);
if(fd<0)
perror("lseek\n");

write(fd,msg,strlen(msg));
lseek(fd,CAPCITY,Location);
write(fd,msg,strlen(msg));
lseek(fd,0,SEEK_END);
if(close(handle)==0)
printf("close successfully\n");
return 0;
}


Awesome! I love it. I'll pull this change onto jonreeve.com soon, too (since it's not actually a github.io site). Also, I'm planning on getting Annotags its own domain soon, so this hack and other Annotag-related things can get more exposure, and don't have to live at my site. Thanks so much!


will remove bigdecimal next time.


Yeah, I noticed that I did that in a couple of places. I think I fixed it in the `game-logic` branch.


Things like empty strings can often have strange behavior (like you said, boundary cases). It's always good to test them, and shouldn't be too much trouble.


@CrazyWolf23 ikr


thanks. pretty n00bish on my part. land it.


@johnsimons @SimonCropp is this better


Easy peasy: 15cdba5f0a4f4eb6bebbba5334b34288da094e54


Should we add a ``flush`` function to make it clear what this does?


@alexblair1 `self.context` has not been setup at this point so it fails. need to move the code that set's it up from `fetchRequest` to `viewDidLoad` before calling this method.


You can move this before ``with parallel`` and get rid of the ``with sequential``.


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1770 of 1771 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)


IMYIndividuation


@alexblair1 in order to get the app to stop throwing the error at launch I had to reset the simulator. this is normal during development. remember - clean the project and reset the simulator anytime you get odd results.


@alexblair1 resetting ios simulator and fixing the `self.context` issue allows the app to run. but it crashes when attempting to search on the main tableView. Do you need help with that?


saw approval from shino
at https://github.com/basho/riak_cs/commit/9e5d394ad780c597597024e6941733a84add72e8


merging basho/riak_cs/feature/fast-get-user = 9e5d394a into borshop-integration-1191-feature/fast-get-user


basho/riak_cs/feature/fast-get-user = 9e5d394a merged ok, testing candidate = c07db4c6


It seems that radians should not be used and instead a kind of "normalized Nyquist frequency" according to the source. @tap can you take a look at [the varaible "fc" here](http://musicdsp.org/showArchiveComment.php?ArchiveID=26) and confirm that you see what I do?

More stable now, but the cutoff is now slightly out of tune.


:heart: :star: :star: :star: :star: :star:


:scissors:


nice :+1:


Oh, it's kind of weird that `-vV` allows any other valid flags, but kind of useful here. You are confident forcing alternate implementations to maintain this feature of -v?


Pointless blank space


@efreak it isn't broken, it just needs to be `IsUpTrigger` instead of `isupTrigger`.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1420


To elaborate on the bootstrap, the three stages are:

* Stage0: download some [blessed previous build](https://github.com/rust-lang/rust/blob/master/src/snapshots.txt) of the compiler and build the *current source* with that to produce a new compiler. This compiler should have all the latest language and library features, but there are several reasons why it shouldn't be used. At this point the most prominent reasons are:
    * It was built with an old compiler, and therefore its generated code may not have all the sweetest new perf.
    * It was built with tons of weird cfg(stage0) tweaks that are intended to be removed -- we really want to verify the *new* code

* Stage1: use the compiler we just built to build the current source again. This compiler has all the latest features, was built with the "true" source (no cfg(stage0)'s) and its generated binary should be the state-of-the-art. This compiler is in principle adequate except for one key detail: it's standard library and plugins were built with the the cfg(stage0)-y compiler. This may mean that its standard library has a different ABI than the the one it outputs! This would be disastrous to ship, as most code it compiles will want to link with the standard library!

* Stage2: use the previous compiler to built the current souce *again*. This compiler should now be perfect: it was built by a compiler that has been thoroughly cleansed of the stage0 pollution. All of its standard library and plugins should have the same ABI that it generates. Note that it's not a problem that it was built by something with a weird standard library because the Rust source is entirely self hosted. It doesn't assume an external standard library to link against; it provides its own. This doesn't imply that everything is effectively no_std. Much of the standard library itself is -- but rustdoc and rustc in particular get to behave like normal rust programs with std defined.


This test, as currently written, will validate that both `System.register` and `System.import` append the `.js` extension to any module ID provided when `defaultJSExtensions` is `true`.  The bug I am running into is when inside a package that *should not* have a `.js` extension added (`defaultJSExtensions` is overridden by `defaultExtension`).  I can't run the tests (see other issue I created) but I suspect that if you were to change the test to the following it would fail:

```js
System.config({
	"defaultJSExtensions": true,
	"packages": {
		"foo": {
			"defaultExtension": "ext"
		}
	}
});
System.register('foo/custom.ext', [], function(_export) {
	return {
		setters: [],
		execute: function() {
			_export('zip', 'zap');
		}
	};
});
System['import']('foo/custom').then(function(m) {
	ok(m.custom == 'ext');
	start();
}, err);
```

Note: `System['import']('foo/custom.ext')` will also fail since the bug is with `System.register` registering `foo/custom.ext.js`, not with `System.import` failing to utilize the `defaultExtension` configuration.


This is probably OK. Little value in adding an assert if this throws as well.


Also apply to vl and polestar.

On Tue, Jul 28, 2015 at 9:16 PM, Kanit Wongsuphasawat <
notifications@github.com> wrote:

> @domoritz A,B,C origin
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/uwdata/voyager/commit/22f4cae77e2961959a59cc9af116b7941e72c1e2
>


Can you add a -j flag here so that the build is done in parallel. It will save time in testing. Add it as a separate commit.


Can this be replace with goog.asserts.assert in the closure build?


LGTM


辛苦了， 目前这个阶段线上可以实际用吗？


True, but Rails 3.2 is the final minor release of Rails 3, correct? :wink:


This modification seems irrelevant to this pull request?


Ah yes, i find it redundant. It's not related to this PR.


> Just to make sure I understand, you are talking of veth42 being managed by networkd?

Yes.

> I will post a PR.

I'll look at it/test it, thanks!


@yfdyh000 你知道firefox加载修改过的语言包里的searchplugin的机制吗？我记得百度搜索我也是用的改过的，不过是直接复制了一个新文件过来的，这个是修改的但是就不能加载到设置里


https://github.com/jun325/CCA/commit/665286632c2d6b1733039f83ee83c8e4e2adca42

pls try it.


Is there a way to get the first tr without needing to add an id to it?


Documentation is needed here...


Documentation must be provided in these methods, its a major public class...


ReadVarsSmallDiffTest (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1776 of 1777 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)


fix_intel_compile_linux (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1260 of 1260 tests passed)

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


Fixed in 936bf085ea


You didn't unset isFirstRow in activityLogEntry after this?


Fixed in 936bf08


Fixed in 936bf08. Thanks.


maybe we could use `.. versionadded:: 3.1`


Would be great to document some of these API changes. Here are the changes that I picked up over the last few days:

```
aws.Long() -> aws.Int64()
Region: "" -> Region: aws.String("")
aws.StringValue -> aws.Prettify()
```


The reason I have to add an ID to the first row is that the function `extractHtmlPartFromFile()` only accepts By.id #4029
Another way I can think of is using recursion to find the first tr in DOM Document but I think recursion would not be preferred in this case.


fast-forwarding develop to borshop-integration-1191-feature/fast-get-user = c07db4c6


use


The first row should have isFirstRow set while the rest does not, right? It will be used in jsp.


why is laziness being removed?


example:
```
kerl build git ~/repos/klarna/ klarna/OTP_R16B03-1-mnesia_ext klarna-16.3.1-ext
```bash


Yes I had that idea in mind, but using Array#sort! would make it difficult to track indexes. Or maybe the index can be added as another element in the Array of Arrays and sorted alongwith the dataframe.

I'll try that out.


Alright i looked over a bit of it, couldn't really test (Just spammed errors, probably because i used 1.7.10 lol)

I do have some basic questions:

1) Why use "this." when there is no need to define the class owns the field. I'd understand the use of "this." if you had parameters with the same name as a field, but i simply don't understand the point of defining the class owns the field when there is no reason to. Maybe i'm just weird

2) UUID support is still a bit strange atm. I'm not 100% sure if it's even safe to use the UUID as a main lookup tool seeing as how much downtime the UUID servers have. Also, seeing as the way you implemented it, UUID's will have a pretty big effect on the performance of HawkEye and require a lot more CPU cycles. I'm just not sure UUID support is worth the massive performance loss especially since it in your code, you don't use any hash checks, only pure string checks (Hurts  the CPU :( ).

3) What exactly is the "server" data for? If it's for the web panel only, we could use basic MySQL queries to join all the different tables together rather then stuff everything inside the same table. Otherwise we're just adding even MORE bytes per row and allowing server owners to create extremely large tables (Which is something that'll create a lot of issues in the future). I'm also not exactly sure how InnoDB handles tons of Async insert/select queries on the same table, so that's something to look into.

 4) Why not use integers vs material names? If ID's ever become fully removed, we'll switch to our own ID database. Logging each item name is a big performance destroyer and will increase the database size (by A LOT), decrease all our search query efficiency, and will cause a lot more issues with logging that requires serialization (Such as the inventory serializer).

Once again, i wasn't able to test this too much because of it's apparent incompatibility with 1.7.10, but i just feel it'll bog down our performance by a lot. Still really good job though, hopefully we'll find some more performance friendly ways to implement some of this.


Thanks to caf in #epic for the split function - I'd used sar which isn't as nice.


@ercanozkaya Coding style. Else statement should be one line up.


Will basename ensure no directory walking (ie. is it secure). Probably makes sens to also santiize it:
``````
$baseFileName = check_file_dir_name(basename($eachFile));
``````
This function will die if there are any non-filename characters. See here for the function:
https://github.com/openemr/openemr/blob/master/library/sanitize.inc.php#L25

Good to ensure this is secure, otherwise a hacker could use this script to collect files on the server. Let me know your thoughts.


Fixes #22.


@ercanozkaya Coding style. I would use a $stream->isReadable() move the logic inside if and the exception below. Method docblocks need to be updated to include the exception being thrown.


ReadVarsSmallDiffTest (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1776 of 1777 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)


@ercanozkaya Would use a break statement here, and return at the end of the method. default $mimetype = null;


Tests look good, should you include a test that tests for any array, not just empty?


Tracking tickets and notifications across forks is confusing. But https://github.com/ansistrano/deploy/pull/57 is in place now, anyway.


I can create a general assert function like goog.asserts.assert. It should be possible to use a different lib/asserts in the closure build and instead use/proxy to the closure asserts, but I am pretty sure such a bridge would need to be built manually.


When you create a new directory to store files, need to create the new directory in the $GLOBALS['OE_SITE_DIR']/documents/ directory. Then you will be leveraging the security, upload privileges, and backup mechanism that already exists for the documents directory.


Do


Interestingly the standard library actually uses only a fraction of the unstable features because it's almost entirely re-exports of other crates, and re-exporting an unstable feature doesn't require opting into it. These are all the features that code *only* defined in std actually uses.

For reference, there are about 50 features here, but the distribution exposes almost 200.


The SJWs are doing their damnedest to kill off software development.
People do not have the right not to be offended!


To be safe, would use check_file_dir_name() here also.


@ercanozkaya  This needs to be @link not @see. See if for referring to other parts of the docs, not to an external link. See


To be safe, would wrap $eachFile with check_file_dir_name() .


Caracteristicas makefile !!!!


@MarbleFag @afield @tolbiac110 @smeezekitty,
Really? The TOS allow Github to remove content that is deemed offensive.

They are not alone. I know many people who would find this offensive. It isn't like they shadowbanned them or deleted the repo outright. The maintainers were given 24hrs and instructions to fix it.

Try to have productive discussion. This has to be one of the more shameful things I've ever seen on this site, which is generally a bastion of creativity, acceptance, and creation.


To be safe, would wrap $ruleID with check_file_dir_name() .


What does this solve that the new matching rules don't solve (for JSON anyway)?


Don't end a translation with a space (place the space outside the constant).


What next, hackers using their real names, glossing their CV's, calling their SCM repos "a portfolio" and acting all professional and shit? smfh :(


Always pull out of the master. /thatswhatshesaid


attr()


Whoa whoa, it's not enough to simply remove it from master. The email clearly states that you have to completely purge that word from all of human history.


use echo xls()
``````
alert("<?php echo xls('Please select atleast one criteria to download');?>");


Also not you have a typo where your combined atleast into one word


You know, as a species, I'm delighted that we invented Internet comments. The world is a better place for it.


Please change religion to religion_group


[libcollections also has a dedicated test crate](https://github.com/rust-lang/rust/tree/f14a0e2de484c2a61c80128a442827059c05783f/src/libcollectionstest) because of similar lang-item shenanigans. In particular libcollections is the home of many [primitive impls](https://github.com/rust-lang/rust/blob/f14a0e2de484c2a61c80128a442827059c05783f/src/libcollections/str.rs#L427-L430) like `str` and `slice` because this is the point where Vec and String are defined for them to refer to.


@roideuniverse This is now fixed.


As noted before, wedemic group does not exist, it might be just pagan in this event if I understand what is going on correctly


:shipit:


lmao at GitHub's inability to recognize that I renamed SkillsRef


Hi @LEETcc thanks for the feedback. The best way to see a complete view of changes is on the project's github [Releases](https://github.com/aws/aws-sdk-go/releases) page. [v0.7.0](https://github.com/aws/aws-sdk-go/releases/tag/v0.7.0) release links out to the [Release Notes](http://aws.amazon.com/releasenotes/0023860075558122) detailing what changed, and how to migrate existing code bases.


调用阿里云API接口Python脚本


In which case would this happen? And also, why would it happen only for their key ID, and not for our key ID?


As a retard I find the term Git offensive.


Screenshot included


Btw, `elements` is also unsafe - it requires non-empty list otherwise it [throws an error](http://hackage.haskell.org/package/QuickCheck-2.8.1/docs/src/Test-QuickCheck-Gen.html#elements). It is still nicer this way, but without `pre $ ls /= []` it is pretty much equivalent to `!!`


`Uint64` will never be < 0 :rainbow:


yup, happy to add a version of `elements` that takes `NonEmpty a` as an input to disorder


@EduardSergeev if you are going to do something like that though (a "safe" version of `elements`), consider though that rather than `elements [x, y, z]` people would have to write `elements $ x :| [y, z]`....

I dont personally mind, its just characters, but some ppl might understandably find it distasteful, this is the main thing that has held me back from adding something like this to disorder


More clear to use xlt() here, which already does htmlspecialchars.


attr()


Note text would be more clear:
``````
$tempCqmAmcString .= " " . text( xl('PQRI') . ":" . $row['cqm_pqri_code']) . " ";


That for supporting stuff like comma separated ranges.

For example,
`nmatrix[2,5]` as a substitute for `nmatrix[2..5]`


True that. Thanks.


Actually, wouldn't even do the escape here. Do it below. So remove htmlspecialchars.


remove htmlspecialchars


<html>
<head>
<title>
Suma
</title>
</head>
<body>$num1=$_POST["num1"];
	$num1=$_POST["num2"];
	$suma=$num1+$num2;
	</form>
</body>
</html>


Change this to 'Profiler - Nodes' and move it alphabetically above 'Profiler -Variables'


use text()
``````
echo "(".text($tempCqmAmcString).")";


assignTypes when there is no row defined, makes all columns type as string and it will broken number sort function to work properly. rows size should be checked:

if (self.rows.length > 0) {
        self.assignTypes();
    }


Remove commented out code.


I see now.
It's a little bit prone to errors (it was the case before your mod) ...
I suggest:
- this 23 becomes a constant (as the one you created)
- the sbus++ is done between the sanity check and the loop
Would you include this in the review? Thanks!


Picky!


html escape with attr()
``````
echo "<input type=hidden id=text" . attr($counter) . " name=text" . attr($counter) . " value='" . attr($row['cqm_nqf_code']) . "'/>";


attr() around $counter


este es otro commit


DDWD


Updated in c714df6a518af5bdb1f8dd12c99a601e3b88e444.


For posterity:
Composer Asset Plugin has fixed their bug via https://github.com/francoispluchino/composer-asset-plugin/pull/137


Yes, might complicate things.. In this example though some functions based on `pre` could actually return `NonEmpty`.
```haskell
nonEmptyList :: [a] -> PropertyM m (NonEmptyList a)
```
since `pre` otherwise [shortcuts](http://hackage.haskell.org/package/QuickCheck-2.8.1/docs/Test-QuickCheck-Monadic.html#v:pre) `PropertyM` execution.


I don't think this syntax works in R. You need to use `grepl(value, chars)`.


So is "Skxawng" considered as an insult too? :laughing:


Yeah this is basically just running the option parser and then the first thing we do with the parsed options is look for `-V`. It basically just means that alternate implementations have to parse options first (and error out on unknown ones) which doesn't seem too unreasonable to me.


use binding


matrix copied to hander->matrix_table, then copied to kernel. there are twice copy. how about reduce to one.
directly set const XCam3aResultColorMatrix &matrix to _yuv_pipe_kernel.
you can refer to CLBayerPipeImageHandler::set_wb_config/set_gamma_table


Hmm. Maybe documentation (this code or in the reference) is enough. Ack to the other things.


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1776 of 1778 tests passed)

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


+1 for simply using 'binary' and 'utf8' but by string in order to leave open future possibilities in case 'text' would become a need.
+1 for using option argument as a dictionary where to pass this optional settings!

by the end of the week i will try to clean bart branch, test the integration with saucelabs and test the performances in relation to the streaming patch i'm implementing.


``...Also reject the message if the sender keyid is their_keyid-1, but their_y[their_keyid-1] is NULL. ``

Maybe this can happen if something wrong happen in the middle of the way to set the safe channel and we try to send a data msg before receiving their pub key.


This is true hehe


Be careful though, this the increases overhead of your application -> Be careful though, this can decrease performance.


<html>
<head>
<title>
Suma
</title>
</head>
<body>$num1=$_POST["num1"];
	$num1=$_POST["num2"];
	$suma=$num1+$num2;
	</form>
</body>
</html>


This is not a good idea IMO.  Should only use fallback positioning if there's no GPOS at all.


When component events are broadcast, the plugin event handlers get notified the same way. -> When component events are broadcast, the plugin event handlers get triggered in the same way.


97648166-FixPIUInducedNode-4926 (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)


last_used => \'now()', would save us the DateTime object (and the DBIC deprecation warning about that)


This probably should've got into a separate PR


Alright. I just pushed a fix. Thanks.


Just give me a quick reason why this rule is getting special treatment (so I can figure out what is going on here).


Awesome! I would reinforce the fact that it can be set to an actual boolean value, and not just a method that returns a `boolean`. How about adding _` (or a method that returns `true`)`_ after `set to true` in the current version?

Thanks for all your work on this pr!


@ybur-yug if you find it offensive why not just fork and maintain your own product?


remove commented code (unless still under active development)


text() is cleaner


#ok


actually if this is going to xml, then don't need to do any escaping


_Chévere!_ I can only think about preventing an attacker to crash our client by sending invalid sender/receiver key IDs.

That's why it seems to me that this should not be exclusive for the sender key ID. I mean, `pick[Our|Their]Keys()` should return the same `otr: unexpected keyID X` error whenever they intent to return a `nil` key. WDYT? :sleepy:


Yes.


Amended.


I'm fine committing this, but I am curious why github shows every line has changed, when in fact only a few lines have changed.  Is this normal?


requesting info on license:
https://github.com/thibaud-rohmer/PhotoLight/issues/2


p_out, looks like float is better than float4 and save GPU register.
you can consult Ruiling whether float4 is ok.


What is this `period` command?


Can you communicate about this on the forum?
I agree that we shouldn't show this information to non-map-makers, but our map makers are used to any errors popping up right away when they start triplea up.


+	Pranav_whiz
+	Date registered
+	Role
+	Status
+	Actions
+


asdsad


If `index > @size*2`, that condition is handled by `index >= @data.size`.


text() is cleaner:
``````
<title><?php echo text( xl('Export QRDA Report')); ?></title>


<html>
<head>
<title>
Suma
</title>
</head>
<body>$num1=$_POST["num1"];
	$num1=$_POST["num2"];
	$suma=$num1+$num2;
	</form>
</body>
</html>


xlt()


I have another automated rewrite at https://github.com/copumpkin/nixpkgs/commit/28ac395e70306068535b29024ba06ff091556edc but it seems to change the hashes of the packages it touches, so I'm puzzled. If anyone can spot what the issue is, I'd appreciate a pointer!


use:
echo xlt()
(and remove the e parameter)


Probably going to change these fields private??


Good catch!
Thanks!

BTW an advice, can you please **not** make direct commits and make PRs instead. This will allow me to review the patches and changes before committing to the master and thus prevent any potential breakage. This is only till the point that I am also confident that you have got a good hang of the codebase.


xla():
``````

<input type='button' value='<?php echo xla('Close'); ?>' onclick='closeme();' />


Make sure it applies cleanly and then just put the patch file (ideally with a reasonable filename) into the `depends/common/protobuf/` directory.


Good job @Mottie :-)


Wasn't these arrays in above code. Seem like better to have then in one place (ie. library file)


LGTM except this blank line.


We *could* always say "special" instead.

[![Stephen Lynch - Special Ed](http://img.youtube.com/vi/pfFRv_1XdDM/0.jpg)](http://www.youtube.com/watch?v=pfFRv_1XdDM)


LOL just PR it https://github.com/zalora/git-intro/pull/1


i did it


Change to private


since this is xml, html escaping is not needed


@foxnewsnetwork Stephen Lynch FTW :+1:


Good work. Carry on when you are free. :) :+1:


> we are hiring

Seems not so relevant, given where you intend to deliver this.


It was added in a pr.


Collections.shuffle(success, random)

The second parameter of that method is type of java.util.Random.
You're sending a RandomGen, is that normal?


wrap $rule_id with check_file_dir_name()
(should wrap the names also, but this would break for any non standard name characters)


@verydapeng I'm sorry but as a :-1: I find your :+1: offensive.


wrap with check_file_dir_name()
``````
$xmlurl = $qrda_file_path . check_file_dir_name($qrda_fname);


xlt()


Note that the term git in itself is an offensive term: https://en.wikipedia.org/wiki/Git_%28slang%29


>@afield @tolbiac110 @smeezekitty @MarbleFag,
>Really? The TOS allow Github to remove content that is deemed offensive.
>
>They are not alone. I know many people who would find this offensive. It isn't like they shadowbanned them or deleted the repo outright. The maintainers were given 24hrs and instructions to fix it.
>
>Try to have productive discussion. This thread itself has to be one of the more shameful things I've ever seen on this site, which is generally a bastion of creativity, acceptance, and creation.

He says, on a website named after an insulting word for a stupid or annoying person.

You are not the brightest tool in the shed.


Maybe instead of getExtraArguments returning List of objects (which is very ugly), it would be better to call abstract method such us

protected MethodHandle createMethodHandle(baseMethodHandle);

that would be overridden by each operator?


실수함..
mHandler 멤버 변수의 하위 method 에 접근하는 방식으로 수정되야됨...

곧 반영예정


Merger of Development and master


vhsovhsohcsodc


cho nay bi loi


can xem lai noi dng


A typo with "receive".


I agree :bowtie:


This doesn't seem to skip the test on my build.


@ybur-yug
"[other username removed because it is offensive and contains a slur],"

You really need to grow thicker skin on the internet. I don't agree with the choice of the word "retard" (or "fag")
but I am not going to get all butt hurt over it and cry for it's removal. If something offends you, move on and go somewhere else.

Removing projects for such as stupid reason is not helping any rational person.


Why did the interval need to be upped?


```
$ docker images | grep registry
docker-registry-cli                                   latest-alpine       707a278438f1        2 minutes ago       52.98 MB
docker-registry-cli                                   latest-slim         759a03d97737        About an hour ago   207.1 MB
```


X forwarded Proto is just one of the possible mechanisms though. For
example, recently a RFC proposes a Forwaded header. So we currently only
have a single header to care about but we may support others quite soon.

--

*José Valimwww.plataformatec.com.br
<http://www.plataformatec.com.br/>Founder and Director of R&D*


yeah that's perfect adding it now.


    hash = {}
    if(options)
      options.each do |(k,v)|
        hash[k] = v
      end
    end

should probably be

    hash = options.dup


@ybur-yug
" which is generally a bastion of creativity, acceptance, and creation."

Hmmm. Bastion eh?
Alexis Ohanian is that you?


write event는 추후 등록될 예정
예제와 같이 업로드 할 생각


Why is `resolve` being called twice?


'messages' is not an util module


The knowls are fine, but what is the reason for putting the information about the base field in with the parameters of the form instead of by themselves?
Also, it is a good idea to rebase your work on an updated master, since it makes testers more confident that what they are looking at is what it will actually look like.


lgtm


How does this work?  Is there some magic in libtool or automake that includes a filename if that is all that is contained in a .c file?


In CookingData,

Shouldn't name be name() in even if it's groovy? It exceeds it's access rights.


Can a provider config ever change, and if so does the apiserver need to be restarted to pick up the change?


Outdated doc


The problem was that I was switching from the normal TEI rendering of the `<pb>` to the page-image specific view in the wrong function to operate on ajax requests too.


Perhaps this test should be a check to see if it duck-types as Subscription? `instanceof Subscriber` will work, but technically, anything that duck types as both an Observer and a Subscription could make it here. We probably need an `isSubscription` method to use. It should check the minimal interface (`unsubscribe()` and `isUnsubscribed`, IMO).


Oh, you handle that below.  Awesome.


i know the stuff


This is ridiculous. GitHub management is a bunch of gits! (No pun intended.)


This is pretty cool.  Looks good, except I'd prefer to not pull in stdint via bs.h into everything; it's probably better to explicitly include it (or the alternate) where needed.  I can hack together a version that does that no problem.

Could you tell me why (void) is needed in function declarations?  As far as I can tell, modern C compilers would complain if any arguments are passed to a function that is ().


hgfds


Didnt we discuss this and decided against variadics here?


:x:  "You must enter dates without gaps and the dates must not overlap."
:+1: Downgrade to 1 day of StartDate


'open' instead of 'open-browser' here and in other places


Performing the geometric mean by accumulating the product of the values can quickly lead to overflow or underflow for large numbers of values. It'd be better to do it by computing the arithmetic mean of the logarithms of the values:

```
GM(x0...xN) = exp((ln(x0) + ln(x1) + ... + ln(xN))/N)
```


shouldn't here be aPrecision + aScale? I don't get it, why if bScale > aScale you are effectively setting precision to:

precision = aPrecision + bScale + bScale - aScale?


add comment why here is +1 :)


일단은 Volley Extension에서 나온 문서대로 작성을 해본 것임.

아마 Volleyer를 구현한쪽을 보면 나올듯.


nice bror


그렇긴한데 어떤 기능을 개발했냐고 꼬집어 본다면..

애플리케이션을 처음 켤때의 화면과, 그 화면에서 이 애플리케이션에 아무도 로그인을 안했을때 페이스북 로그인 버튼을 이용해서 로그인 하게 하는 기능을 개발했다고 보면됨.


Indeed...


redundant


good work


@bagder I've since noticed the doc for [`CURLOPT_NOSIGNAL`](http://curl.haxx.se/libcurl/c/CURLOPT_NOSIGNAL.html) says use `1` as well instead of what I would expect `1L`. I was under the impression after some investigating I did in #346 that if the value type is `long` it must be explicit because that could be an issue for some platforms with va_args. In Windows long and int are the same width and there's no difference but I'm curious about the details surrounding this behavior for other platforms. I'm now noticing places in the code where integers are used, see the [protocol flags](https://github.com/bagder/curl/blob/172b2be/include/curl/curl.h#L787-L814) for example, and I did something similar myself when I added a flag to ssl options.


@Enzemo : The tutorial function hasn't worked since the beginning of OpenRCT2, but you could still choose a tutorial, after which the game would display an error. As this is bad UX, my commit disabled the button entirely.

We want to have a tutorial function again at some point, but the old implementation was resolution dependent and just a recording of keystrokes and mouse movements. We would need to have a better implementation for that. In the meantime, this should make it clear OpenRCT2 doesn't have a tutorial function yet. (As a side note: if it was to be removed entirely I would have taken out the whole button, strings, etc.)


wifidog 的核心还是依赖于 iptables 防火墙过滤规则来实现的，所以建议对 iptables 有了了解后再去阅读 wifidog 的源码。

在路由器上启动 wifidog 之后，wifidog 在启动时会初始化一堆的防火墙规则，如下：

[cpp] view plaincopy在CODE上查看代码片派生到我的代码片

/** Initialize the firewall rules
*/
int iptables_fw_init(void)
{
    … …
/*
     *
     * Everything in the NAT table
     *
     */
/* Create new chains */
    iptables_do_command("-t nat -N " TABLE_WIFIDOG_OUTGOING);
    iptables_do_command("-t nat -N " TABLE_WIFIDOG_WIFI_TO_ROUTER);
    iptables_do_command("-t nat -N " TABLE_WIFIDOG_WIFI_TO_INTERNET);
    iptables_do_command("-t nat -N " TABLE_WIFIDOG_GLOBAL);
    iptables_do_command("-t nat -N " TABLE_WIFIDOG_UNKNOWN);
    iptables_do_command("-t nat -N " TABLE_WIFIDOG_AUTHSERVERS);
/* Assign links and rules to these new chains */
    iptables_do_command("-t nat -A PREROUTING -i %s -j " TABLE_WIFIDOG_OUTGOING, config->gw_interface);
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_OUTGOING " -d %s -j " TABLE_WIFIDOG_WIFI_TO_ROUTER, config->gw_address);
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_WIFI_TO_ROUTER " -j ACCEPT");
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_OUTGOING " -j " TABLE_WIFIDOG_WIFI_TO_INTERNET);
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -m mark --mark 0x%u -j ACCEPT", FW_MARK_KNOWN);
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -m mark --mark 0x%u -j ACCEPT", FW_MARK_PROBATION);
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -j " TABLE_WIFIDOG_UNKNOWN);
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_UNKNOWN " -j " TABLE_WIFIDOG_AUTHSERVERS);
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_UNKNOWN " -j " TABLE_WIFIDOG_GLOBAL);
// 将 80 端口的访问重定向(REDIRECT)到 (本路由)网关web服务器的监听端口
    iptables_do_command("-t nat -A " TABLE_WIFIDOG_UNKNOWN " -p tcp --dport 80 -j REDIRECT --to-ports %d", gw_port);
/*
     *
     * Everything in the FILTER table
     *
     */
/* Create new chains */
    iptables_do_command("-t filter -N " TABLE_WIFIDOG_WIFI_TO_INTERNET);
    iptables_do_command("-t filter -N " TABLE_WIFIDOG_AUTHSERVERS);
    iptables_do_command("-t filter -N " TABLE_WIFIDOG_LOCKED);
    iptables_do_command("-t filter -N " TABLE_WIFIDOG_GLOBAL);
    iptables_do_command("-t filter -N " TABLE_WIFIDOG_VALIDATE);
    iptables_do_command("-t filter -N " TABLE_WIFIDOG_KNOWN);
    iptables_do_command("-t filter -N " TABLE_WIFIDOG_UNKNOWN);
/* Assign links and rules to these new chains */
/* Insert at the beginning */
    iptables_do_command("-t filter -I FORWARD -i %s -j " TABLE_WIFIDOG_WIFI_TO_INTERNET, config->gw_interface);
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -m state --state INVALID -j DROP");
/* TCPMSS rule for PPPoE */
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -o %s -p tcp --tcp-flags SYN,RST SYN -j TCPMSS --clamp-mss-to-pmtu", ext_interface);
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -j " TABLE_WIFIDOG_AUTHSERVERS);
    iptables_fw_set_authservers();
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -m mark --mark 0x%u -j " TABLE_WIFIDOG_LOCKED, FW_MARK_LOCKED);
    iptables_load_ruleset("filter", "locked-users", TABLE_WIFIDOG_LOCKED);
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -j " TABLE_WIFIDOG_GLOBAL);
    iptables_load_ruleset("filter", "global", TABLE_WIFIDOG_GLOBAL);
    iptables_load_ruleset("nat", "global", TABLE_WIFIDOG_GLOBAL);
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -m mark --mark 0x%u -j " TABLE_WIFIDOG_VALIDATE, FW_MARK_PROBATION);
    iptables_load_ruleset("filter", "validating-users", TABLE_WIFIDOG_VALIDATE);
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -m mark --mark 0x%u -j " TABLE_WIFIDOG_KNOWN, FW_MARK_KNOWN);
    iptables_load_ruleset("filter", "known-users", TABLE_WIFIDOG_KNOWN);
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_WIFI_TO_INTERNET " -j " TABLE_WIFIDOG_UNKNOWN);
    iptables_load_ruleset("filter", "unknown-users", TABLE_WIFIDOG_UNKNOWN);
    iptables_do_command("-t filter -A " TABLE_WIFIDOG_UNKNOWN " -j REJECT --reject-with icmp-port-unreachable");
    UNLOCK_CONFIG();
return 1;
}
在该 防火墙规则的初始化过程中，会首先清除掉已有的防火墙规则，重新创建新的过滤链，另外，除了通过iptables_do_command("-t nat -A "TABLE_WIFIDOG_UNKNOWN " -p tcp --dport 80 -j REDIRECT --to-ports %d",gw_port); 这个命令将 接入设备的 80 端口（HTTP）的访问重定向至网关自身的 HTTP 的端口之外，还通过iptables_fw_set_authservers(); 函数设置了 鉴权服务器(auth-server) 的防火墙规则：

[cpp] view plaincopy在CODE上查看代码片派生到我的代码片

void iptables_fw_set_authservers(void)
{
const s_config *config;
    t_auth_serv *auth_server;
    config = config_get_config();
for (auth_server = config->auth_servers; auth_server != NULL; auth_server = auth_server->next) {
if (auth_server->last_ip && strcmp(auth_server->last_ip, "0.0.0.0") != 0) {
            iptables_do_command("-t filter -A " TABLE_WIFIDOG_AUTHSERVERS " -d %s -j ACCEPT", auth_server->last_ip);
            iptables_do_command("-t nat -A " TABLE_WIFIDOG_AUTHSERVERS " -d %s -j ACCEPT", auth_server->last_ip);
        }
    }
}
首先从上面的代码可以看出 wifidog 支持多个 鉴权服务器，并且针对每一个鉴权服务器 设置了如下两条规则：

1)在filter表中追加一条[任何访问鉴权服务器都被接受]的WiFiDog_$ID$_AuthServers过滤链：

iptables -t filter -A  WiFiDog_$ID$_AuthServers -d auth-server地址 -j ACCEPT

2)在nat表中追加一条[任何访问鉴权服务器都被接受]的WiFiDog_$ID$_AuthServers过滤链：

iptables -t nat -A WiFiDog_$ID$_AuthServers  -d auth-server地址 -j ACCEPT

这样确保可以访问鉴权服务器，而不是拒绝所有的出口访问。


just -moz-border-radius: 9px

And check latest versions of ff and chrome. If they support the 'border-radius' rule we can get rid of '-moz-border-radius' and '-webkit-border...' rules


replace with classes


It will slightly simplify the code (one indentation level less) if you check at the begining

if (b == 0) throw new PrestoException(DIVISION_BY_ZERO);

and it would be more self documenting.


dddd


Hier hat Roman ein Beispiel für eine Änderung hinzugefügt.


According to the example at https://developers.google.com/identity/protocols/OpenIDConnect?hl=en
the `sub` claim is a long random number.  I don't think that is what we want.

People are going to want to write authorization policies using human readable names, and UIs are going to want to show human readable names too.

How about using `sub` as the uid, and then using `email` as the name?  I think you need to check that `email_verified` claim is present too?

@liggitt what do you think about this?  Also, in openshift, aren't you using something like `domain:username` as the user.Name, so does the email need to be split on the at-sign and transformed into that format?


same as above?


what kind of optimization you have done for these images? Just want to know so that I can also keep in mind while putting images into the project.


According to http://users.frii.com/jarvi/rxtx/license.html it is GLPL - is this problematic??


jhjf


lgtm


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (744 of 744 tests passed)

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


@jozefg I think it should be possible to prove that this is the realizer for the induction principle for `NAT`, and thence we will prove the induction principle itself. I am having trouble with the proof, though.

The reason we have to start from the realizer is that the elimination rule for the image type is only valid at a squash-stable motive. So, we can't use induction to prove the induction principle directly (since the conclusion is not squash stable), but wellformedness of the proposed realizer is of course squash-stable, so we may begin there.


The option to call a specific module->action is already covered by the module/action parameters by the route, and the controllers are already located in the Actions.php class. While controllers may very well be better namespaced under modulename\controller\Controllername, I think it's a bad idea to introduce this inconsistency. Is there a reason the runResolve() method cannot live inside the main Actions.php controller?


Looking at salesforce.com, it seems like they also have an opaque id in `sub` and also use the email and email_verified claims.


You don't need to do this anymore, it's the default in Travis now


@glasser This should be `cache.files[cacheKey] = { hash: file.getSourceHash(), errors: errors };` ;-)


```js
var resizeArgs = req.body.paramsType === 'width-height' ?
    [Number(req.body.width), Number(req.body.height)] :
    [req.body.deviceName, req.body.orientation];
```


ref、__mk_zh_CN 和 mkt 是什么？原来没有的东西。


---Introduction---


テスト


Ah I think the subselects are only an issue with mysql when used in UPDATE and DELETE statements.

Concerning the `dateAdded` columns for artists and albums they don't even have a matching enum key like all other fields do so I highly doubt that they are being used anywhere and could probably be removed (which should also work on SQLite since its the last field in the table).


Conexion a la BD


Hi @tomaszdomin-arevocom,

This was tested on an OpenSIPS flatfile. Are you using with Kamailio or? Can you provide me with a sample of CDR which does not work with NewPartialFlatstoreRecord maybe?

Ta,
DanB


`</b>`.


can't we simply add amnesia to mix.exs -> applications?


currentForegroundWindow is defined but never used


So do we know the retard at github who thought this is actually offensive? I mean, are us retards not allowed to use webm? Is it offensive to people at github that we would actually use software and not be a super smart programmer?


Que es esto? Can we delete this file?


A stupid decision from github, that's for sure.


should slack invite be a submodule of https://github.com/unfoldingWord-dev/dokuwiki-slack-invite ? or do we want to drop that repo in favor of maintaining it in door43. With this commit they have different code.


Fixed in https://github.com/apache/stratos/pull/410


おっと。失礼しました。使っていたのですね、気付きませんでした。


Fixed in https://github.com/apache/stratos/pull/410 to check > 0


@Pablete1234 ```args.getJoinedStrings(int);``` trims the string anyawy.


なるほど、そういうことでしたか。


remove comments


Added error details to messages in https://github.com/apache/stratos/pull/410


In the upgrade to rouge 2.0, you should be able to inherit most of this behavior from `Formatters::HTML` rather than reinvent the whole thing.


Shouldn't you also do contains for the phone number.
Most of my phone numbers start with the same country code so I wouldn't want to type it...


i use spark1.5.0 , but pyspark on yarn-client  has a error,it can't startup,
i use    bin/pyspark --master  yarn-client  to run .

ERROR LiveListenerBus: SparkListenerBus has already stopped! Dropping event SparkListenerBlockManagerAdded(1438152010663,BlockManagerId(1, dwsparkslave1.wlt, 45355),556038881)
15/07/29 14:40:10 INFO BlockManagerMaster: BlockManagerMaster stopped
15/07/29 14:40:10 INFO OutputCommitCoordinator$OutputCommitCoordinatorEndpoint: OutputCommitCoordinator stopped!
15/07/29 14:40:10 INFO SparkContext: Successfully stopped SparkContext
Traceback (most recent call last):
  File "/home/www/spark/spark-1.5.0-bin-20150129/python/pyspark/shell.py", line 43, in <module>
    sc = SparkContext(appName="PySparkShell", pyFiles=add_files)
  File "/home/www/spark/spark-1.5.0-bin-20150129/python/pyspark/context.py", line 113, in __init__
    conf, jsc, profiler_cls)
  File "/home/www/spark/spark-1.5.0-bin-20150129/python/pyspark/context.py", line 174, in _do_init
    self._accumulatorServer = accumulators._start_update_server()
  File "/home/www/spark/spark-1.5.0-bin-20150129/python/pyspark/accumulators.py", line 259, in _start_update_server
    server = AccumulatorServer(("localhost", 0), _UpdateRequestHandler)
  File "/usr/lib64/python2.6/SocketServer.py", line 402, in __init__
    self.server_bind()
  File "/usr/lib64/python2.6/SocketServer.py", line 414, in server_bind
    self.socket.bind(self.server_address)
  File "<string>", line 1, in bind
socket.gaierror: [Errno -2] Name or service not known


明確なコーディング規約はありませんが、下記サイトはよく参考にしていますね。
http://qiita.com/jnchito/items/dedb3b889ab226933ccf
https://github.com/satour/rails-style-guide/blob/master/README-jaJA.md


now it's not understandable why we call the ShowWindowAndRestoreForeground function from the ResizeWindow function. Let's explain this here in this way:

```c#
...
int newWidth  = Convert.ToInt32(args[1]),
    newHeight = Convert.ToInt32(args[2]);

//NOTE: if the target window is maximized we should restore down it before resizing. Otherwise we'll have some problems with it (... and explain here what the problems - styling, or something else)
if(IsZoomed(hWnd)) {
    RestoreDownWindow(hWnd);
}

ResizeWindow();
```


Debug logs are printed by the method which is calling this method.


Shouldn't this vector's size be defined by C_S_AXI_**HOST**_ADDR_WIDTH?


Shouldn't this vector's size be defined by C_S_AXI_**HOST**_DATA_WIDTH?


@gkoz That sounds like a nice idea, thanks!

I'll work on getting this implemented soon, thanks again for your help on this! :)


@captain-caveman2k No problem, thanks for the update.


Yes, all the numerical varargs take longs and we should make sure to mention `long` or use L suffixes in the documentation.


This must have taken bloody ages! Appreciated.


Yeah :)


You cannot really say that they aren't pointers if it compiles fine with `nullptr`. I think (but I might be wrong about this) that even MS started using `nullptr` in MSDN for handles in some example codes.

NB: I don't really care if you prefer using `NULL`  though.


Don't remember, will check it again later.


С миграциями ничего не надо делать - удалять и создавать одну?


@Moghedrin At ownCloud 8.0 and 8.1 are two major versions: https://owncloud.org/blog/owncloud-release-model-changes/


our build sys doesn't support platform specific patches, we never had a need for it.
Why don't you want the same gtests on all platforms?
If you really want this only for win, you'd have to make the patch itself do the right thing.
i.e. add a platform check to autogen in the patch


<?php

/**
 * Twitter-API-PHP : Simple PHP wrapper for the v1.1 API
 *
 * PHP version 5.3.10
 *
 * @category Awesomeness
 * @package  Twitter-API-PHP
 * @author   James Mallison <me@j7mbo.co.uk>
 * @license  MIT License
 * @version  1.0.4
 * @link     http://github.com/j7mbo/twitter-api-php
 */
class TwitterAPIExchange
{
    /**
     * @var string
     */
    private $oauth_access_token;

    /**
     * @var string
     */
    private $oauth_access_token_secret;

    /**
     * @var string
     */
    private $consumer_key;

    /**
     * @var string
     */
    private $consumer_secret;

    /**
     * @var array
     */
    private $postfields;

    /**
     * @var string
     */
    private $getfield;

    /**
     * @var mixed
     */
    protected $oauth;

    /**
     * @var string
     */
    public $url;

    /**
     * @var string
     */
    public $requestMethod;

    /**
     * Create the API access object. Requires an array of settings::
     * oauth access token, oauth access token secret, consumer key, consumer secret
     * These are all available by creating your own application on dev.twitter.com
     * Requires the cURL library
     *
     * @throws \Exception When cURL isn't installed or incorrect settings parameters are provided
     *
     * @param array $settings
     */
    public function __construct(array $settings)
    {
        if (!in_array('curl', get_loaded_extensions()))
        {
            throw new Exception('You need to install cURL, see: http://curl.haxx.se/docs/install.html');
        }

        if (!isset($settings['oauth_access_token'])
            || !isset($settings['oauth_access_token_secret'])
            || !isset($settings['consumer_key'])
            || !isset($settings['consumer_secret']))
        {
            throw new Exception('Make sure you are passing in the correct parameters');
        }

        $this->oauth_access_token = $settings['oauth_access_token'];
        $this->oauth_access_token_secret = $settings['oauth_access_token_secret'];
        $this->consumer_key = $settings['consumer_key'];
        $this->consumer_secret = $settings['consumer_secret'];
    }

    /**
     * Set postfields array, example: array('screen_name' => 'J7mbo')
     *
     * @param array $array Array of parameters to send to API
     *
     * @throws \Exception When you are trying to set both get and post fields
     *
     * @return TwitterAPIExchange Instance of self for method chaining
     */
    public function setPostfields(array $array)
    {
        if (!is_null($this->getGetfield()))
        {
            throw new Exception('You can only choose get OR post fields.');
        }

        if (isset($array['status']) && substr($array['status'], 0, 1) === '@')
        {
            $array['status'] = sprintf("\0%s", $array['status']);
        }

        $this->postfields = $array;

        // rebuild oAuth
        if (isset($this->oauth['oauth_signature'])) {
            $this->buildOauth($this->url, $this->requestMethod);
        }

        return $this;
    }

    /**
     * Set getfield string, example: '?screen_name=J7mbo'
     *
     * @param string $string Get key and value pairs as string
     *
     * @throws \Exception
     *
     * @return \TwitterAPIExchange Instance of self for method chaining
     */
    public function setGetfield($string)
    {
        if (!is_null($this->getPostfields()))
        {
            throw new Exception('You can only choose get OR post fields.');
        }

        $getfields = preg_replace('/^\?/', '', explode('&', $string));
        $params = array();

        foreach ($getfields as $field)
        {
            if ($field !== '')
            {
                list($key, $value) = explode('=', $field);
                $params[$key] = $value;
            }
        }

        $this->getfield = '?' . http_build_query($params);

        return $this;
    }

    /**
     * Get getfield string (simple getter)
     *
     * @return string $this->getfields
     */
    public function getGetfield()
    {
        return $this->getfield;
    }

    /**
     * Get postfields array (simple getter)
     *
     * @return array $this->postfields
     */
    public function getPostfields()
    {
        return $this->postfields;
    }

    /**
     * Build the Oauth object using params set in construct and additionals
     * passed to this method. For v1.1, see: https://dev.twitter.com/docs/api/1.1
     *
     * @param string $url           The API url to use. Example: https://api.twitter.com/1.1/search/tweets.json
     * @param string $requestMethod Either POST or GET
     *
     * @throws \Exception
     *
     * @return \TwitterAPIExchange Instance of self for method chaining
     */
    public function buildOauth($url, $requestMethod)
    {
        if (!in_array(strtolower($requestMethod), array('post', 'get')))
        {
            throw new Exception('Request method must be either POST or GET');
        }

        $consumer_key              = $this->consumer_key;
        $consumer_secret           = $this->consumer_secret;
        $oauth_access_token        = $this->oauth_access_token;
        $oauth_access_token_secret = $this->oauth_access_token_secret;

        $oauth = array(
            'oauth_consumer_key' => $consumer_key,
            'oauth_nonce' => time(),
            'oauth_signature_method' => 'HMAC-SHA1',
            'oauth_token' => $oauth_access_token,
            'oauth_timestamp' => time(),
            'oauth_version' => '1.0'
        );

        $getfield = $this->getGetfield();

        if (!is_null($getfield))
        {
            $getfields = str_replace('?', '', explode('&', $getfield));

            foreach ($getfields as $g)
            {
                $split = explode('=', $g);

                /** In case a null is passed through **/
                if (isset($split[1]))
                {
                    $oauth[$split[0]] = urldecode($split[1]);
                }
            }
        }

        $postfields = $this->getPostfields();

        if (!is_null($postfields)) {
            foreach ($postfields as $key => $value) {
                $oauth[$key] = $value;
            }
        }

        $base_info = $this->buildBaseString($url, $requestMethod, $oauth);
        $composite_key = rawurlencode($consumer_secret) . '&' . rawurlencode($oauth_access_token_secret);
        $oauth_signature = base64_encode(hash_hmac('sha1', $base_info, $composite_key, true));
        $oauth['oauth_signature'] = $oauth_signature;

        $this->url = $url;
        $this->requestMethod = $requestMethod;
        $this->oauth = $oauth;

        return $this;
    }

    /**
     * Perform the actual data retrieval from the API
     *
     * @param boolean $return      If true, returns data. This is left in for backward compatibility reasons
     * @param array   $curlOptions Additional Curl options for this request
     *
     * @throws \Exception
     *
     * @return string json If $return param is true, returns json data.
     */
    public function performRequest($return = true, $curlOptions = array())
    {
        if (!is_bool($return))
        {
            throw new Exception('performRequest parameter must be true or false');
        }

        $header =  array($this->buildAuthorizationHeader($this->oauth), 'Expect:');

        $getfield = $this->getGetfield();
        $postfields = $this->getPostfields();

        $options = array(
            CURLOPT_HTTPHEADER => $header,
            CURLOPT_HEADER => false,
            CURLOPT_URL => $this->url,
            CURLOPT_RETURNTRANSFER => true,
            CURLOPT_TIMEOUT => 100,
			CURLOPT_SSL_VERIFYPEER => false,//para correrlo en localhost
        ) + $curlOptions;

        if (!is_null($postfields))
        {
            $options[CURLOPT_POSTFIELDS] = http_build_query($postfields);

        }
        else
        {
            if ($getfield !== '')
            {
                $options[CURLOPT_URL] .= $getfield;
            }
        }

        $feed = curl_init();
        curl_setopt_array($feed, $options);
        $json = curl_exec($feed);

        if (($error = curl_error($feed)) !== '')
        {
            curl_close($feed);

            throw new \Exception($error);
        }

        curl_close($feed);

        return $json;
    }

    /**
     * Private method to generate the base string used by cURL
     *
     * @param string $baseURI
     * @param string $method
     * @param array  $params
     *
     * @return string Built base string
     */
    private function buildBaseString($baseURI, $method, $params)
    {
        $return = array();
        ksort($params);

        foreach($params as $key => $value)
        {
            $return[] = rawurlencode($key) . '=' . rawurlencode($value);
        }

        return $method . "&" . rawurlencode($baseURI) . '&' . rawurlencode(implode('&', $return));
    }

    /**
     * Private method to generate authorization header used by cURL
     *
     * @param array $oauth Array of oauth data generated by buildOauth()
     *
     * @return string $return Header used by cURL for request
     */
    private function buildAuthorizationHeader(array $oauth)
    {
        $return = 'Authorization: OAuth ';
        $values = array();

        foreach($oauth as $key => $value)
        {
            if (in_array($key, array('oauth_consumer_key', 'oauth_nonce', 'oauth_signature',
                'oauth_signature_method', 'oauth_timestamp', 'oauth_token', 'oauth_version'))) {
                $values[] = "$key=\"" . rawurlencode($value) . "\"";
            }
        }

        $return .= implode(', ', $values);
        return $return;
    }

    /**
     * Helper method to perform our request
     *
     * @param string $url
     * @param string $method
     * @param string $data
     * @param array  $curlOptions
     *
     * @throws \Exception
     *
     * @return string The json response from the server
     */
    public function request($url, $method = 'get', $data = null, $curlOptions = array())
    {
        if (strtolower($method) === 'get')
        {
            $this->setGetfield($data);
        }
        else
        {
            $this->setPostfields($data);
        }

        return $this->buildOauth($url, $method)->performRequest(true, $curlOptions);
    }
}
?>


Why are we skipping javadocs on the api modules? Is there a way to not skip using the javadoc plugin? The javadoc jar files are no longer being created which the ddf distribution attempts to pull in.


blibioteca para realizar conexion api tweeter


Nice one!


we can call this function `RestoreDownWindow` and use SW_RESTORE instead of showCommand.
The 'AndRestoreForeground' suffix is redundant because we change foreground window inside this function and then restore it. So, it doesn't matter outside the function


Thanks for merging my changes ! :)

Could you perform a release now ? I think there's no API changes (only deprecated methods) so it can be a minor release.


你这改的什么玩意


Transparent

rashigoel <notifications@github.com> wrote:

Instead of purple color should i make it transprent or try some other color?

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


At some places, Update Info still called Update Request. Is it intentional?


[trending on HN now](https://news.ycombinator.com/item?id=9966118)


Thanks, fixed it.

DanB


both dirty 不是total dirty
你这个trace可以吧, 主端/备端/both 的dirty都打印出来


I think this was broken by #12327 - are you wanting to sort by keys or values?
```
ERROR: MethodError: `isless` has no method matching isless(::Array{ASCIIString,1}, ::Array{ASCIIString,1})
 in isless at operators.jl:438
 in sort! at ./sort.jl:216
 in sort! at ./sort.jl:305
 in sort! at ./sort.jl:310 (repeats 8 times)
 in sort! at ./sort.jl:393
```


find_package(kodi) is probably using the wrong kodi-config.cmake for this use case. You can override it by passing -Dkodi_CONFIG=$HOME/workspace/xbmc/project/cmake


Doesn't this make clang complain about empty body of the if statement?


а это зачем?


Yeah the point is that when you type '49' you then get +49..., 049..., ...49..., and ...49. I do not know if it is not too much results then. I'll maybe do an option for that


`Function::derivative` is deprecated, please use `Function::derForward`, `Function::derReverse` instead.


I'm having to do my own diff here to see what was changed because I don't know what changes were made to this file. For example, I'm troubleshooting a bug in the map's max zoom level. Please, next time you copy/paste then edit a large file, commit after the copy/paste event so I can more easily visualize the edits you made to the file.


gracias compañero


Pre-analysis feedback - String.format() would be better.


This commented-out line of code can be dropped.


@vilchik-elena indeed good question. We want to be depending on the "smallest LTS" so a wider range of user will be able to upgrade without having to upgrade the platform (which is a long and painful process in some company). Here we depend on 4.5.**2** and not 4.5.**1** because there've been a API change between those 2 versions that we want to take advantage of.


это вообще нужно, чтобы при uri.to_s ерез урл параметры передавались, но это только для get запроса. здесь это только запутывает и совершенно бессмысленно


It was needed because h264_nal.o is part of two targets, and there was some conflict about this.


What is the deal with `n_repreated`? Looks like it just complicates things to me.


This is the bug! I'm not sure why you changed this to 25, but that's fine, because that's the point of code review. But I have no easy way of knowing you made this change because you copy/pasted then edited the file in the same commit.


por nada amigo


Yep, the next commit solved it.


嗯啊


Method has too many lines. [11/10]


Great bugfixs/improves :)


I didn't look for more instances since I forgot that there were more. Looking at the files now there seem to be some instances where it's hard-coded into the storage (I clearly did not think this through a year ago), but I'll fix the ones where it doesn't involve the actual submissions json file.


I likes it. +1


I seen it done both ways. I like the inline way because it's more succinct.


Note: the highlight of the changes to `GNUInstallDirs.cmake`:

> This patch fixes this issue by using "lib/\<arch\>" only if the user is installing the software in /usr, and uses "lib" in all the other cases.

Unless I missed something, this would solve the problem here, except if people on multi-arch distributions want to install the libraries to `/usr/lib` specifically for some reason.


Hi Toke,

silly question (I am most likely confused) but if we somehow have a directory in $run_path won't the stat return that as well? Or can we check whether the result is a file? Other than that I like your new code better, it is more elegant and concise.

Best Regards
        Sebastian


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


So would you enable CONFIG_SCHED_HMP_LITTLE_PACKING?
Seems like it's disabled by default.


I haven't worked through the whole bootstrapping flow yet, but yes, we need to gracefully handle the first time through.


Oh, just a little nitpick, the `aexpect` is system package now, so there is no reason in separating it by an extra line... (sorry for not noticing this earlier).


Maybe we should put this as a reminder somewhere. I fear this commit annotation will get lost in the wild.


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/18430/testReport/) for this commit**


3.0.2 Introduces new feature. Thus fails compilation. The dynamically added target revision tag does not work any more.


Hi liorshin

thanks for the translation.
One question: Have you considered all the special parameters and placeholders used by the software and have you checked the resulting functionality in TDL?

I'm asking because I saw
+"%s %.2f %c"		"אחוז"	"text"
+"%s %d"			"אחוז"	"text"
+"%s %s (%s)"	"אחוז"	"text"
+"%s %s"			"אחוז"	"text"

I don't know the Hebrew language, but it seems that 4 different (dynamic) English strings have the same static translation. Everything with %s, %d and so on is a placeholder for dynamic values and must not be translated. Here are examples from French and German:

French:
"%0.1f Kb downloaded so far (%0.1f Kb remaining)"	"%0.1f Kb téléchargés (%0.1f Kb restants)"	"text"
"%d / %d tasks"	"%d / %d tâches"	"text"
"%d task(s) selected"	"%d tâche(s) sélectionnée(s)"	"text"
"%d"	"%d"	"text"
"%s (Custom)"	"%s (Personnalisé)"	"text"
"%s (Custom, Relative)"	"%s (Personnalisé, Relatif)"	"text"
"%s [%s]"	"%s [%s]"	"text"

German:
"%0.1f Kb downloaded so far (%0.1f Kb remaining)"	"%0.1f kB bisher heruntergeladen (%0.1f kB verbleibend)"	"text"
"%d / %d tasks"	"%d / %d Aufgaben"	"text"
"%d task(s) selected"	"%d Aufgabe(n) ausgewählt"	"text"
"%d"	"%d"	"text"
"%s %.2f %c"	"%s %.2f %c"	"text"
"%s %d"	"%s %d"	"text"
"%s %s (%s)"	"%s %s (%s)"	"text"
"%s %s"	"%s %s"	"text"
"%s (Custom)"	"%s (Benutzerdefiniert)"	"text"
"%s (Custom, Relative)"	"%s (Benutzerdefiniert, Relativ)"	"text"
"%s [%s]"	"%s [%s]"	"text"

Regards

Peter


@Underground78 Handles are abstraction, they just happen to be implemented on top of pointer types, and I'm not even sure this is always the case. You can't say INVALID_HANDLE_VALUE is a valid pointer for example, because it doesn't really point anywhere. Ideally they need NULL_HANDLE value, but we don't have that. But we still have NULL and can clearly separate them from pointers now.


Keys


Currently there is no way to abstract away certain features, the only option is a really thick controller. I'm sure performance will have a profit with the separation, but I'll profile the performance difference later today :+1:


You could use `reverse_each` if you want to iterate the collection in reverse.


Hello GearCM. I wonder the OS version of these blobs came from. Are they came from official lollipop firmware?


> Could you tell me why (void) is needed in function declarations? As far as I can tell, modern C compilers would complain if any arguments are passed to a function that is ().

"Declaration is not a signature", a fatal warning like that.
Anyway, i also had to add -std=gnu99 to CFLAGS to allow such things like variable declarations in for() etc.

If you are interested in my usecase with kernel, it is tw5864 media grabber driver: http://github.com/krieger-od/linux/ , drivers/staging/media/tw5864 . If you feel interested, you are welcome to check correctness of my usage. The resulting bitstream is not perfect. Example of resulting file is http://whdd.org/out.h264 . The generation of file is processed in tw5864-video.c in tw5864_prepare_frame_headers(), tw5864_handle_frame().
Also important missing part of API of your project is handling of last incomplete byte. See my forked repo.


@intervigilium


No one gives a shit that you know a bunch of thin skinned walking vaginas.

On Wed, Jul 29, 2015 at 12:58 AM, yburyug <notifications@github.com> wrote:

> @MarbleFag <https://github.com/MarbleFag> @afield
> <https://github.com/afield> @tolbiac110 <https://github.com/tolbiac110>
> @smeezekitty <https://github.com/smeezekitty>,
> Really? The TOS allow Github to remove content that is deemed offensive.
>
> They are not alone. I know many people who would find this offensive. It
> isn't like they shadowbanned them or deleted the repo outright. The
> maintainers were given 24hrs and instructions to fix it.
>
> Try to have productive discussion. This has to be one of the more shameful
> things I've ever seen on this site, which is generally a bastion of
> creativity, acceptance, and creation.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12412209>
> .
>


Sì in realtà volevo un delegate System.Action ( https://msdn.microsoft.com/en-us/library/system.action(v=vs.110).aspx )


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/18431/testReport/) for this commit**


Couldn't find any changes. DId you look into it? Rename the repo to something which tells what it is about.


@0a-
It's disturbing how many people justified it with the "private company" defense.
Yes, github CAN do what they want as a private company. But being legal doesn't make it right


i fixed it, the problem is my /etc/hosts file,  in the /etc/hosts file, 127.0.0.1 localhost was annotation.


I have changed it a bit. What do you think?


DDS is not working because of this casting issue.. Return type should be Integer..?


Can you check if the service account e2e test(s) still work when the newAuthenticatorFromOpenIDEndpoint is active?  I'm worried that the OpenID authenticator will return not OK and then we won't get around to calling the service account authenticator, and so then the service accounts can't auth.


Looks like adding `by=first` should fix it then?


![not popcorn](http://tclhost.com/NMYhAiw.gif "not popcorn")


In realtà stiamo solo aspettando di upgradare a VS2015 ( https://msdn.microsoft.com/en-us/library/dn986595.aspx ) :santa:


Well IMO it should be there but not commented out. It raises `NotImplemented` thus it's a valid implementation...


@wachterjohannes Also removed that line in another PR already, we don't really have another choice, if styleci says this is an error :wink:


@jwatzman I don't want to increase further desk-flipping, but this should also be included here https://github.com/facebook/hhvm/blob/master/hphp/test/tools/import_zend_test.py#L93


If I'm not wrong, that does not exists in Windows 7 (https://stackoverflow.com/questions/19529442/gather-ip-address-information).


Changed in 9a7f44ae639b41be9b309398de0aac2595ca6271.


можно комментировать коммиты


Yes, that worked. I'll commit the fix.


@tunnelshade Is that debug print?


убрал вот это, потому что адок получался. зачем он был нужен?


@tunnelshade And debug break point?


Makes sense, views are very fragile against update and delete statements. They can't know what to do when you try to set or get a subselect value or an isnull or anything thats more complex.


@tunnelshade And debug print?


wrap $begin_date and implode(",", $patArr) with the add_escape_custom() function


черт
давай настроим одинаково порядок импортов. у меня стоит "сначала все из пакета java.*, потом пустая строка, потом все остальное"


If evt.lengthComputable === false evt.total is always 0


It doesn't really do much considering it keeps saying it's dead


wrap implode(",", $patArr) with the add_escape_custom() function


Due to the white space differences, dos line endings vs none. Interestingly enough there is a secret hide whitespace flag, makes this diff much cleaner : )

https://github.com/blog/967-github-secrets


This is silly that it had to have its description updated now - it has been working like this since wotlk (adding T7 sets would give you both 10/25 man versions)


Well my comments from last version probably disappeared, so how about using `re.escape`? It should behave the same with the exception of escaping ` ` too. (which is IMO correct ,because when you supply filename with ` ` in it, it needs to be escaped as `\ `.


иначе ругался на getInfluences() и getActivities(), что они не дженерализованы


test not on GH


use binding


97648166-FixPIUInducedNode-4926 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1262 of 1262 tests passed)

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


Hi Justin,
Thanks for answering.
I think equation 5 is correct,but I'm a little confused .In another progress to writing state's matrix there is no non linearity .Here it is, we first write state equations in robot's reference.
Then we use Xg,d(desired global state) to generate Xb,d(desired bot state),that's it: ``Xb,d = Xg,d . R'  ``. And then generating du and u* from Xb,d:
``0 = A.Xb,d + B.u*``
``du = k.(X - Xb,d)``
``u=du+u*``

Maybe there is a mistake in my progress to obtaining state's matrix,or maybe this is a difference because of different references?!


Yes, not sure how that happened ... it's definitely wrong.


нет смысле


@clebergnu I had the exact same requirement last time, Lucas assured me, that's only in github PR, the real order is correct. But yes, I wonder why it keeps reordering the patches...


Thanks! Maybe we should define a lexical ordering for Pairs too?


Couldn't find any changes. DId you look into it? Rename the repo to something which tells what it is about.


Please add some comments about what this is doing.


gkt gu ghk gk


You've probably figured this out already, but for the browse pages you should use  `filter_mainstream_browse_pages`, _not_ `filter_sections`, as rummager uses the correct terminology there.


Did this project need what the commit name says?


The program SFXR <http://www.drpetter.se/project_sfxr.html>can be used to
make exactly such sounds completely for free.

On Tue, Jul 28, 2015 at 7:41 PM, LudwikJaniuk <notifications@github.com>
wrote:

> Bwahaha, just heard the sound. One can of course just change the file, but
> you should be thinking about changing the default one. This sound is way
> too loud and distracting to work long-term. Some kind of toned down
> digital-ish "blip" would be what I'd imagine a good sound, but all I can
> say with certainty is that this one is bad.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/babin101/DevKeyboard/commit/3d5c6cf46ac1df400d3e273422542ba79984151d#commitcomment-12402503>
> .
>


Git is run but a bunch of retards


Really don't understand what's going on here.


We are only using super tenant registry to persist. Added comments to explain why we had to do this in
https://github.com/apache/stratos/pull/410


The note should be part of the same t(), even if it gets very long. You can move the arguments to $args to make the line shorter.


First Header | Second Header
------------ | -------------
Content from cell 1 | Content from cell 2
Content in the first column | Content in the second column


Important for this file to have proper spacing since it's complicated. Don't use tabs in your spacing to ensure it lines up with the current code.


added comment from github website


Fixed formatting issues in https://github.com/apache/stratos/pull/410


Yes, this breaks apps. Why can't it just be like below?

Instead of:
```javascript
  // Omit the path parameter `id` from values, unless it was explicitly defined
  // elsewhere (body/query):
  var idParamExplicitlyIncluded = ((req.body && req.body.id) || req.query.id);
  if (!idParamExplicitlyIncluded) delete values.id;

  // No matter what, don't allow changing the PK via the update blueprint
  // (you should just drop and re-add the record if that's what you really want)
  if (typeof values[Model.primaryKey] !== 'undefined') {
    sails.log.warn('Cannot change primary key via update blueprint; ignoring value sent for `' + Model.primaryKey + '`');
  }
  delete values[Model.primaryKey];
```

Do:
```javascript
values[Model.primaryKey] = pk;
```


//AA


Was going over failing tests in Ren/C and noticed the loss of this line as the cause of a regression.  The reason is because the PROTECT status was being checked inside of Copy_Stack_Values:

https://github.com/rebol/rebol/blob/25033f897b2bd466068d7663563cd3ff64740b94/src/core/f-blocks.c#L282

So losing that means that if you use REDUCE/INTO, you can wind up writing into protected series.

Not high priority, and this code is all likely to change in Ren/C.  So don't feel like you have to do something about it, just keeping you in the loop.  :-)


Added log if deleted is false in https://github.com/apache/stratos/pull/410


Added support for attribute respondTo
from slick documentation:
respondTo
string
'window'
Width that responsive object responds to. Can be 'window', 'slider' or 'min' (the smaller of the two)


```json
{
  "version": "0.6.20",
  "approved": true,
  "approver": "mschaaf",
  "approvalDate": 1438155300670
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


Couldn't find any changes. DId you look into it?


All done with these!


You can't use %payment_keys. Named arguments need a corresponding _load function. But that's not needed here. So just %.


This is either wrong or really confusing. The docstring of this method says: "*Replace the first instance of x with y, bypassing callbacks.*" and was intentionally not implemented as ``self.remove()`` followed by ``self.add()`` to ensure exactly that. So either the docstring and the implementation needs to change or any problematic call of this method should be replaced with ``remove(); add()``.


Reverted.

Possible solutions: turn off optimization of knxd.c, move the daemon code code to a .c file, remove that daemon code and tell people to use "nohup" and/or systemd, ask Travis to update their broken installation.


Commented codes are removed in later commit.


Would this need to be `* display: inline` to make it only apply to IE?


Sure. Let's do this though, I don't want to take away that feature from map makers. Let's make it a preference option with default behavior to suppress.I'll then post on the forum that we're making that change and ask for feedback.


Just by searching in various repos on Github, there are worse offenders than this repo...

Like those 1853 repos: https://github.com/search?q=fuck

This whole thing is silly...